Skip to content

Redo my old improper fix for #3580 (undoes 655f36e) - #99

Merged
CaitlynMainer merged 1 commit into
PC-Logix:main-MC1.21.1from
Ocawesome101:fix-process-for-real-this-time
Sep 14, 2026
Merged

CaitlynMainer merged 1 commit into
PC-Logix:main-MC1.21.1from
Ocawesome101:fix-process-for-real-this-time

Conversation

@Ocawesome101

Copy link
Copy Markdown

process.removeHandle(mt.process, t) was incorrect argument order and causing weird issues with the thread API. The new fix fixes a check that was the actual root cause of the issue, by acknowledging the fact that process.info() returns a new table every time--but for a given process, the data field of this table will always be the same.

`process.removeHandle(mt.process, t)` was incorrect argument order and
causing weird issues with the `thread` API.  The new fix fixes a check
that was the actual root cause of the issue, by acknowledging the fact
that `process.info()` returns a new table every time--but for a given
process, the `data` field of this table will always be the same.
@CaitlynMainer
CaitlynMainer merged commit fa8adda into PC-Logix:main-MC1.21.1 Sep 14, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants