update to support 1.20.4 #163
Conversation
This is an early stab at updating to 1.20.4. There are several deficiencies: - carpetfixes.mixins.advanced.MinecraftServer_autosaveDelayMixin needs to be rewritten to match the changes in the game - I do not understand what carpetfixes.mixins.reIntroduced.PlayerManager_LlamaRidingDupeMixin is trying to do with the "vmp" and "c2me" prefixes. - The method call intercept to com.google.commons.collect.Maps.newHashMap() fail in carpetfixes.mixins.blockFixes.PistonBlock_pushOrderMixin and I do not understand why. - Update suppression fixes should be removed as they are handled by TIS. The mod does compile and work for the few fixes I have tried.
|
Will be taking a look at this today |
There was a problem hiding this comment.
Ill be removing it, as minecraft has done significant work on it to fix lag issues.
I'll be removing it.
I'll also be removing that rule. Those prefixes are for the vmp (very many people) & c2me (Concurrent Chunk Management Engine) mods. It's a compatibility fix.
Ya mixin always does this for google commons, not sure why. However it will still inject correctly. |
I thought I was going crazy. :-) |
This is an early stab at updating to 1.20.4. There are several deficiencies:
carpetfixes.mixins.advanced.MinecraftServer_autosaveDelayMixin needs to be rewritten to match the changes in the game
I do not understand what carpetfixes.mixins.reIntroduced.PlayerManager_LlamaRidingDupeMixin is trying to do with the "vmp" and "c2me" prefixes.
The method call intercept to com.google.commons.collect.Maps.newHashMap() fail in carpetfixes.mixins.blockFixes.PistonBlock_pushOrderMixin and I do not understand why.
Update suppression fixes should be removed as they are handled by TIS.
The mod does compile and work for the few fixes I have tried.
This should eventually resolve #159.