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

Add Show Imports decompiler setting #344

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

YanisBft
Copy link
Contributor

Nothing major, just a new "Show Imports" setting when decompiling classes.

Closes #136

@Gegy
Copy link
Collaborator

Gegy commented Jan 28, 2021

Does this work for imports of obfuscated classes? I'm not sure how it would get around the issue with displaying imports mentioned in #136.

@YanisBft
Copy link
Contributor Author

This PR only adds the option, but yes the imports are a bit limited, as Procyon only shows library imports, and CFR shows all but with obfuscated names

@Gegy
Copy link
Collaborator

Gegy commented Jan 29, 2021

The option might be a bit misleading and potentially have very limited scope for use.

@YanisBft YanisBft marked this pull request as draft January 29, 2021 18:41
@liach
Copy link
Contributor

liach commented Mar 15, 2021

These imports will be confusing as well, as they include more imports than what's assumed by javadoc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Suggestion] Option to show declared imports
3 participants