-
Notifications
You must be signed in to change notification settings - Fork 32
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
Added support to automatically detect read order #16
base: main
Are you sure you want to change the base?
Conversation
Read Order Analyzer has been added for VM Method Reader and Operand Reader
SwitchAnalyzer is a class for reading the instructions of each statement in a switch statement
Also support 2024 version. |
Been a while, thanks for the PR! I've marked it as a draft as it's missing support for exception handlers and a few other constants like |
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.
a
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.
a
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.
a
As mentioned in issue #3, this pull request solved that problem. The issues were somewhere around VM Method Info, Type Resolver, and VM Operand Reader.