search

judovana / java-runtime-decompiler

GNU General Public License v3.0
68 stars 14 forks source link

Jasm messes up lambdas #169

Closed judovana closed 1 year ago

judovana commented 2 years ago

Generally speaking, if the original source code had lambdas, it can be disassembled, but not assembled back.

This has to be discussed and fixed in https://github.com/openjdk/asmtools.

AurumTheEnd commented 2 years ago

Is https://github.com/openjdk/asmtools just a mirror? I don't see an Issues tab to discuss this issue in.

EDIT: The link on the wiki which might be the upstream appears to be dead.

judovana commented 2 years ago

It is no longer just mirror, but bugs to openjdk are here: https://bugs.openjdk.java.net/ and it is pain to fill one (fro good reasons)

-- Mgr. Jiri Vanek @.***

---------- Původní e-mail ---------- Od: Prokop Tunel @.> Komu: pmikova/java-runtime-decompiler @. github.com> Datum: 7. 10. 2021 23:02:40 Předmět: Re: [pmikova/java-runtime-decompiler] Jasm messes up lambdas (#169)

"

Is https://github.com/openjdk/asmtools(https://github.com/openjdk/asmtools) just a mirror? I don't see an Issues tab to discuss this issue in.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub (https://github.com/pmikova/java-runtime-decompiler/issues/169#issuecomment-938153273) , or unsubscribe (https://github.com/notifications/unsubscribe-auth/AAWFCS3KAOEMHBUZVZDVQHDUFYDGRANCNFSM5FOK5EDQ) . Triage notifications on the go with GitHub Mobile for iOS (https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675) or Android (https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub) . "

judovana commented 1 year ago

jasm8 works fine with lambda. But otheriwse it is bugy. Work in progress to fnish it

judovana commented 1 year ago

In addition both asm7 and asm8 with bot g and without g variants are now provided