skylot / jadx

Dex to Java decompiler
Apache License 2.0
41.03k stars 4.81k forks source link

[core]Rename method to prohibit jadx errors #1876

Closed bhwxha closed 1 year ago

bhwxha commented 1 year ago

link: https://drive.google.com/file/d/1EYP3KNNmSez3SNjFjX7Qe4kdl8kCmi4V/view?usp=sharing

As in the picture, jadx wrongly changes the method name to showVersionInfo, how to prohibit it from renaming incorrectly?

image

image

skylot commented 1 year ago

This is a feature to remove synthetic bridge methods by "merging" with wrapped method. This can be disabled by Inline methods option (if checkbox missing, please use latest unstable build :sweat_smile:) I also commit an additional rename reason: /* merged with bridge method [inline-methods] */ And if debug code comments are enabled, merged methods signatures will be shown:

    /* JADX DEBUG: Method merged with bridge method: showVersionInfo(Lcom/nomagic/b/ag;)Z */
    /* JADX DEBUG: Method merged with bridge method: showVersionInfo(Lcom/nomagic/rcpf/product/lic/ProductVersion;)Z */

Hope this makes it easier to understand why method was renamed and what option control it.

Related issues #1109 and #1170.