Skip to content

ProcState Deduplication#2518

Merged
wixoaGit merged 4 commits intoOpenDreamProject:masterfrom
ike709:onestatesolution
Feb 3, 2026
Merged

ProcState Deduplication#2518
wixoaGit merged 4 commits intoOpenDreamProject:masterfrom
ike709:onestatesolution

Conversation

@ike709
Copy link
Collaborator

@ike709 ike709 commented Jan 31, 2026

Moves some code around to make it less painful for PRs like #2517 that depend on a ProcState being either a DMProcState or a AsyncNativeProc.State. Also renamed the latter to AsyncDMProcState for added clarity over just being named State.

@ike709 ike709 requested a review from wixoaGit January 31, 2026 10:37
@boring-cyborg boring-cyborg bot added the Runtime Involves the OpenDream server/runtime label Jan 31, 2026
@wixoaGit
Copy link
Member

wixoaGit commented Feb 3, 2026

Also renamed the latter to AsyncDMProcState for added clarity over just being named State.

I think AsyncNativeProcState would be better, since it only runs native procs and not DM procs.

@wixoaGit wixoaGit merged commit 9760211 into OpenDreamProject:master Feb 3, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Runtime Involves the OpenDream server/runtime size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants