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

Update ASM to 9.5, drop AsmBridge #591

Merged
merged 2 commits into from
Aug 30, 2023
Merged

Update ASM to 9.5, drop AsmBridge #591

merged 2 commits into from
Aug 30, 2023

Conversation

pxb1988
Copy link
Owner

@pxb1988 pxb1988 commented Aug 30, 2023

asm may introduce incompatible between versions, the api pass to ClassVisitor() is for compatible. i believe keep the api separately is better than global constant. as we can upgrade asm without touching/review all dex2jar codes.

AsmBridge is a tricky to 'Method Code too large', drop it.

cherry-pick 600f04a from https://github.com/ThexXTURBOXx/dex2jar

ThexXTURBOXx and others added 2 commits August 30, 2023 20:42
asm may introduce incompatible between versions, the `api` pass to ClassVisitor() is for compatible.
i believe keep the `api` separately is better than global constant. as we can upgrade asm without touching/review all dex2jar codes.

AsmBridge is a tricky to 'Method Code too large', drop it.

cherry-pick 600f04a from https://github.com/ThexXTURBOXx/dex2jar
@pxb1988 pxb1988 merged commit 17ae98f into 2.x Aug 30, 2023
1 check passed
@pxb1988 pxb1988 deleted the asm9 branch August 30, 2023 13:26
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