-
-
Notifications
You must be signed in to change notification settings - Fork 54
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
Inspector: remove redundant Java package prefixes #242
Comments
@alexander-yakushev you may want to give this one a shot while you're with other things Inspector |
I see what's going on. First, the method gets printed honoring the |
I don't mind dancing around truncation limits a bit (serialize into a separate printer, shorten, then truncate). But I'm not sure this is needed – method signature that is 100+ characters long is not very useful. |
However it would seem simple enough to disable all trimming for Method/Field/... |
When inspecting classes, often all fields/methods/... are prefixed with the fully-qualified class name.
Example:
This is particularly ugly in
com.corp.Dog.isTopStylist
- there isn't an actual thing namedcom.corp.Dog.isTopStylist
- it'sisTopStylist
so removing the prefix would be both more readable and more correct.There may be cases where there's a legit use case for rendering a prefix - I'm thinking interface implementation or method inheritance.
Even then, I believe the right thing would be to render the short name, and provide any extra info when going
down
that particular method.The text was updated successfully, but these errors were encountered: