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.

Sign up for GitHub

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

Jump to bottom

Improvements to DynamicMethodBodyReader #470

Merged
merged 2 commits into from
Jun 17, 2022
Merged

Improvements to DynamicMethodBodyReader #470

merged 2 commits into from
Jun 17, 2022

Conversation

ElektroKill
Copy link
Contributor

  • Added support for reading InitLocals property and use the value in GetMethod and RestoreMethod.
  • Set MaxStack property in RestoreMethod.

@ElektroKill
Improved RestoreMethod behavior for DynamicMethodBodyReader
50e5767 
* Added support for reading and restoring InitLocals property
* Restore max stack
@ElektroKill
make initLocals field not nullable.
a4e735e 
`DynamicResolver`'s `m_method` field is always a `DynamicMethod` so a type check was unnecessary.
@ElektroKill ElektroKill requested a review from wtfsck June 16, 2022 19:07
@wtfsck wtfsck merged commit 6f4dc6f into 0xd4d:master Jun 17, 2022
@wtfsck
Copy link
Contributor

wtfsck commented Jun 17, 2022

Thanks, merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wtfsck wtfsck wtfsck approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ElektroKill @wtfsck