Skip to content

Conversation

Yoshy2002
Copy link

as title says. Tested in SP and it works as intended again.
should fix GTNewHorizons/GT-New-Horizons-Modpack#21028

issue is, that AE2FC had its CPacketInventoryAction method args changed from
(action, int, int) to (action, int, long) which gave the NoSuchMethodError

not 100% sure that this is fully correct, but from my testing it fixes the issue

@sisyphussy
Copy link

have you checked if any other mod calls that constructor?

@Dream-Master Dream-Master requested a review from a team September 1, 2025 17:37
@Yoshy2002
Copy link
Author

Yoshy2002 commented Sep 1, 2025

only AE2FC itself, but i couldnt find any crash with that
https://github.com/search?q=org%3AGTNewHorizons%20CPacketInventoryAction&type=code

Copy link

@sisyphussy sisyphussy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i don't actually think the "L" is necessary here (the reason it crashed was because the descriptor changed, so updating the dependency would've been enough) but it doesn't hurt

@sisyphussy sisyphussy added the safe for stable Bugfix with low break chance for patch releases label Sep 1, 2025
@Dream-Master Dream-Master merged commit 3211a4d into GTNewHorizons:master Sep 1, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe for stable Bugfix with low break chance for patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Client crash when autocrafting via NEI
3 participants