Skip to content
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 crash with betterchunkloading mod #568

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

lugui1998
Copy link

There is a issue involving Betterchunkloading mod that causes the server to crash when a player moves enough to unload chunks.
this.positionWithExpiringTicket.get(pos) are presumed to never be null but rather a list that could be empty. More debugging is needed in order to find the reason why this would ever return a null value, which is probably Betterchunkloading's fault, regardless, this check will prevent the server from crashing on a situation like this.

@2No2Name
Copy link
Member

2No2Name commented Oct 7, 2024

I interpret the given NullPointerException issue at a hint at incompatible changes in the two mods. Adding a simple null check may completely resolve the issue, but it is also very likely that there are other incompatibilities which are hidden by the additional null check. I think investigating why the NullPointerException happens in the first place is needed.

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.

None yet

2 participants