TouchInstinct
/
checkstyle
306
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Rename non-boolean method starting with question word. #1555 

Fixes `NonBooleanMethodNameMayNotStartWithQuestion` inspection violations.

Description:
>Reports non-boolean methods whose names start with a question word. Non-boolean methods that override library methods are ignored by this inspection.
This commit is contained in:
Michal Kordas 2015-08-24 22:13:05 +02:00 committed by Roman Ivanov
parent d345f3beb2
commit c4a7d06206
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/main/java/com/puppycrawl/tools/checkstyle/checks/design/VisibilityModifierCheck.java
View File

@ -449,7 +449,7 @@ public class VisibilityModifierCheck
*/
private boolean hasIgnoreAnnotation(DetailAST variableDef) {
final DetailAST firstIgnoreAnnotation =
containsMatchingAnnotation(variableDef);
findMatchingAnnotation(variableDef);
return firstIgnoreAnnotation != null;
}
@ -751,7 +751,7 @@ public class VisibilityModifierCheck
* @return the AST representing the first such annotation or null if
* no such annotation was found
*/
private DetailAST containsMatchingAnnotation(DetailAST variableDef) {
private DetailAST findMatchingAnnotation(DetailAST variableDef) {
DetailAST matchingAnnotation = null;
final DetailAST holder = AnnotationUtility.getAnnotationHolder(variableDef);