-
-
Notifications
You must be signed in to change notification settings - Fork 214
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
remap u32 to varint in protocol.json #457
Conversation
Needs to be double checked with Burger that these snapshots actually don't use u32 here |
I don't think this is correct just add u32 as a native |
So all the versions before and after which are varint for that packet, are wrong? |
No, likely it only changed for these versions |
looks like this packet was created in the versions in this PR. Can you find any previous version that contains this packet ? |
Looking at the decompile for 17w18a, this was an i32. The correct answer is to just switch to i32 which will fix the issue |
Updated with i32 @nickelpro @rom1504 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, should be squashed+rebased not merged
#456
remove u32 and replace with varint, per every other protocol version.