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

Add support for new bytecode weirdness #1751

Merged

Conversation

colinator27
Copy link
Member

Description

  • Some version between 2024.2 and before 2024.4 can include pushref.i in the function reference chain
  • WeirdLocalsFlag appears to be on for many code entries in 2024.2+, so this ensures locals are still disassembled when that happens (if any locals are present)

Caveats

There's a small chance this causes some existing games to have their code locals appear, but for the ones where it should matter (e.g., bytecode 14 and below), this doesn't seem to cause any problems.

Also, this does not have compiler support for the new pushref.i function stuff - only disassembler and decompiler. Not sure if it's worth doing for such a short-lived version, or if it has any tangible effect on the game or not.

- Some version between 2024.2 and before 2024.4 can include pushref.i in the function reference chain
- WeirdLocalsFlag appears to be on for many code entries in 2024.2+, so this ensures locals are still disassembled when that happens (if any locals are present)
@colinator27 colinator27 merged commit 21ecfe5 into UnderminersTeam:master May 3, 2024
5 checks passed
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