intellij-community icon indicating copy to clipboard operation
intellij-community copied to clipboard

[Fernflower] Renaming support for .class field access and assertions

Open droid666 opened this issue 9 years ago • 2 comments

If you use renaming, .class field access and assertions don't decompile correctly in some cases (output contains synthetics).

Examples: https://github.com/droid666/intellij-community/tree/398801bec078efc14529078b22e4b109f7efcadf/plugins/java-decompiler/engine/testData/src-stdout/javadefault/renaming

The examples are part of this pull request: https://github.com/JetBrains/intellij-community/pull/390 They are deactivate in the pull by default because they fail without this fix.

Example B needs the first commit of this pull to work (access to .class on renamed class). Example A also needs the 2nd commit of this pull to work (Assertion location is not recognized as Assertion without adding renaming there).

The question is if this is the correct fix or if these 2 problems mean something does not work as it should?

droid666 avatar Apr 26 '16 12:04 droid666

Is it possible to extract tests specific to this pr and attach here?

gorrus avatar May 02 '17 12:05 gorrus

Ouch, commits from another project.

trespasserw avatar Oct 09 '20 18:10 trespasserw