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

finding reference error #1591

Closed
bhwxha opened this issue Jul 29, 2022 · 2 comments
Closed

finding reference error #1591

bhwxha opened this issue Jul 29, 2022 · 2 comments
Labels
bug Core Issues in jadx-core module

Comments

@bhwxha
Copy link

bhwxha commented Jul 29, 2022

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

Find a reference to class com.yuanrenxue.match2022.fragment.challenge.ChallengeOneFragment.OooO00o, the result is empty

image

Find reference to the constructor of class com.yuanrenxue.match2022.fragment.challenge.ChallengeOneFragment.OooO00o, the result is not empty

image

Also, there is a bug, I'm not sure if it's an optimization or a bug?
Constructor for class com.yuanrenxue.match2022.fragment.challenge.ChallengeOneFragment.OooO00o:
The constructor has 2 arguments, but the pseudocode has only 1 argument.

image

GDA:

image

@bhwxha bhwxha added bug Core Issues in jadx-core module labels Jul 29, 2022
@jpstotz
Copy link
Collaborator

jpstotz commented Jul 30, 2022

The constructor has 2 arguments, but the pseudocode has only 1 argument.

That is by design because you have activated to print inner classes inlined like you develop it in Java. Non-static inner classes always have the parent class as first parameter in constructor. In Java code you don't have to write the parent class as parameter, as it is always there, so Jadx does the same if the inner class is shown inlined.

GDA seems to show the class non-inlined as separate class so the first implicit parameter has to be printed.

@skylot
Copy link
Owner

skylot commented Jul 30, 2022

@bhwxha I fixed the first case (add constructor usages into class usage) and second case is a feature as @jpstotz already mention.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Core Issues in jadx-core module
Projects
None yet
Development

No branches or pull requests

3 participants