-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Fix PMD violations for ShortVariable rule #985
Comments
I am not sure that we need to change variables like GT, ... |
Of course short-named public variables from Maybe |
I am agree on SR. |
@mkordas , Please share your thought on naming of following problematic variables: ex, sb, it, id Should we stay with minimum size 2 or 3 ?
|
http://checkstyle.sourceforge.net/config_naming.html#LocalVariableName should be reconfigured to catch variables who are less then min size (in default form it allow one-char variables). |
I mark issue as resolved, for now we have limit 2 for names , discussion for #985 (comment) could be continued in this issue. |
Follow-up to #982.
Violations:
The text was updated successfully, but these errors were encountered: