-
-
Notifications
You must be signed in to change notification settings - Fork 74
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix NullPointerException when server ticking #581
Conversation
Signed-off-by: Kevin Z <[email protected]>
What bug is this trying to fix? I don't see any usages of ServerWorker.add that enqueue |
That's not very helpful!. I suspect the issue here is actually due to the non-thread-safe usage of |
Ok then: |
You are right, they are running concurrently |
IntelligenceModding#581 Critical fix back to 1.18 Signed-off-by: Kevin Z <[email protected]>
merge please : ) |
@19PHOBOSS98 can you build your own version based on this PR and test if that error never happen again? Cause it's not really easy for me to reproduce |
sure thing! I'll be back with what I find |
it doesn't crash anymore when I tried setting the threads to 4 with all of my drones flying with VS2. Thanks again! Hope this releases soon |
PLEASE READ THE GUIDELINES BEFORE MAKING A CONTRIBUTION
Some mod may call
ServerWorker.tick
with null value, this PR ensure null value never go to the queue, and if it did somehow, the null will never be executedWhat is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change? (What changes might users need to make in their scripts due to this PR?)
No
Other information: