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 chuckpacker y bug #4639

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

fix chuckpacker y bug #4639

wants to merge 1 commit into from

Conversation

babbaj
Copy link
Collaborator

@babbaj babbaj commented Feb 9, 2025

y in pack is already normalized to 0 - 384 and getPathingBlockType was trying to normalize it again, causing adjY to be offset 64 blocks, which caused an oobe in the call to BlockStateInterface.getFromChunk.
The exception got caught but the result seems to be that special blocks close to the build limit didn't get cached.
This probably also broke the caching of water.

@babbaj
Copy link
Collaborator Author

babbaj commented Feb 9, 2025

d22a529

everyone point and laugh at this user for introducing this bug

@ZacSharp
Copy link
Collaborator

Did GitHub miss a usage of getPathingBlockType or are we now denormalizing the y coordinate just to renormalize it immediately in the called method?

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