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

Testpr #2

Closed
wants to merge 17 commits into from
Closed

Testpr #2

wants to merge 17 commits into from

Conversation

Matyrobbrt
Copy link
Member

No description provided.

Matyrobbrt and others added 17 commits December 25, 2023 13:38
…rged#425)

This reimplements the ID offset previously applied to modded
`EntityDataSerializer`s which was lost during the registry rework.

Previously this was achieved by setting a minimum ID on the registry.
Due to the modified vanilla registries not supporting that, the IDs of
modded serializers now also start at 0 which leads to the client using
the vanilla serializer corresponding to the ID of the modded one and
potentially blowing up while deserializing the data item, i.e. by
reading past the buffer's length.
Update CoreMods version to 6.0.4 which uses the new SPI artifact.
This fixes the duplicate SPI dependencies which caused a
`NoClassDefFoundError` in certain situations.
@Matyrobbrt Matyrobbrt closed this Dec 26, 2023
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.

4 participants