Skip to content

Commit

Permalink
fix for Findbugs violations NM_CONFUSING. Issue #912
Browse files Browse the repository at this point in the history
  • Loading branch information
romani committed Apr 11, 2015
1 parent e1d0e7f commit 7dd24c8
Show file tree
Hide file tree
Showing 6 changed files with 21 additions and 15 deletions.
6 changes: 6 additions & 0 deletions config/findbugs-exclude.xml
Original file line number Diff line number Diff line change
Expand Up @@ -34,4 +34,10 @@
<Class name="com.puppycrawl.tools.checkstyle.checks.indentation.LineWrappingHandler" />
<Bug pattern="RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE" />
</Match>
<Match>
<!-- till #922 -->
<Class name="com.puppycrawl.tools.checkstyle.Checker" />
<Method name="setClassloader" />
<Bug pattern="NM_CONFUSING" />
</Match>
</FindBugsFilter>
4 changes: 2 additions & 2 deletions src/main/java/com/puppycrawl/tools/checkstyle/TreeWalker.java
Original file line number Diff line number Diff line change
Expand Up @@ -467,7 +467,7 @@ public static DetailAST parse(FileContents contents)
final String fullText = contents.getText().getFullText().toString();
final Reader sr = new StringReader(fullText);
final GeneratedJavaLexer lexer = new GeneratedJavaLexer(sr);
lexer.setFilename(contents.getFilename());
lexer.setFilename(contents.getFileName());
lexer.setCommentListener(contents);
lexer.setTreatAssertAsKeyword(true);
lexer.setTreatEnumAsKeyword(true);
Expand All @@ -480,7 +480,7 @@ public static DetailAST parse(FileContents contents)

final GeneratedJavaRecognizer parser =
new GeneratedJavaRecognizer(filter);
parser.setFilename(contents.getFilename());
parser.setFilename(contents.getFileName());
parser.setASTNodeClass(DetailAST.class.getName());
parser.compilationUnit();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ public final class FileContents implements CommentListener
.compile(MATCH_SINGLELINE_COMMENT_PAT);

/** the file name */
private final String filename;
private final String fileName;

/** the text */
private final FileText text;
Expand Down Expand Up @@ -76,7 +76,7 @@ public final class FileContents implements CommentListener
*/
@Deprecated public FileContents(String filename, String[] lines)
{
this.filename = filename;
this.fileName = filename;
text = FileText.fromLines(new File(filename), Arrays.asList(lines));
}

Expand All @@ -87,7 +87,7 @@ public final class FileContents implements CommentListener
*/
public FileContents(FileText text)
{
filename = text.getFile().toString();
fileName = text.getFile().toString();
this.text = text;
}

Expand Down Expand Up @@ -261,9 +261,9 @@ public String getLine(int index)
}

/** @return the name of the file */
public String getFilename()
public String getFileName()
{
return filename;
return fileName;
}

/**
Expand Down Expand Up @@ -332,6 +332,6 @@ public boolean hasIntersectionWithComment(int startLineNo,
*/
public boolean inPackageInfo()
{
return this.getFilename().endsWith("package-info.java");
return this.getFileName().endsWith("package-info.java");
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ public void finishTree(DetailAST rootAST)
*/
private String getFileName()
{
String fname = getFileContents().getFilename();
String fname = getFileContents().getFileName();
fname = fname.substring(fname.lastIndexOf(File.separatorChar) + 1);
fname = fname.replaceAll("\\.[^\\.]*$", "");
return fname;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ public boolean isIncompleteTag()
* Used for displaying a Checkstyle error.
* @return an int line number.
*/
public int getLineno()
public int getLineNo()
{
return lineNo;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -371,8 +371,8 @@ private void checkHtml(final DetailAST ast, final TextBlock comment)
final HtmlTag tag = parser.nextTag();

if (tag.isIncompleteTag()) {
log(tag.getLineno(), INCOMPLETE_TAG,
text[tag.getLineno() - lineno]);
log(tag.getLineNo(), INCOMPLETE_TAG,
text[tag.getLineNo() - lineno]);
return;
}
if (tag.isClosedTag()) {
Expand All @@ -389,7 +389,7 @@ private void checkHtml(final DetailAST ast, final TextBlock comment)
// We have found a close tag.
if (isExtraHtml(tag.getId(), htmlStack)) {
// No corresponding open tag was found on the stack.
log(tag.getLineno(),
log(tag.getLineNo(),
tag.getPosition(),
EXTRA_HTML,
tag);
Expand All @@ -409,7 +409,7 @@ private void checkHtml(final DetailAST ast, final TextBlock comment)
&& !htag.getId().equals(lastFound)
&& !typeParameters.contains(htag.getId()))
{
log(htag.getLineno(), htag.getPosition(), UNCLOSED_HTML, htag);
log(htag.getLineNo(), htag.getPosition(), UNCLOSED_HTML, htag);
lastFound = htag.getId();
}
}
Expand Down Expand Up @@ -448,7 +448,7 @@ private void checkUnclosedTags(Deque<HtmlTag> htmlStack, String token)
continue;
}
lastFound = lastOpenTag.getId();
log(lastOpenTag.getLineno(),
log(lastOpenTag.getLineNo(),
lastOpenTag.getPosition(),
UNCLOSED_HTML,
lastOpenTag);
Expand Down

0 comments on commit 7dd24c8

Please sign in to comment.