PMD Results

The following document contains the results of PMD 4.3.

Files

net/sourceforge/pmd/RuleSet.java

ViolationLine
A catch statement that catches an exception only to rethrow it should be avoided.221 - 223
A catch statement should never catch throwable since it includes errors.223

net/sourceforge/pmd/cli/PMDCommandLineInterface.java

ViolationLine
Avoid unused imports such as 'net.sourceforge.pmd.lang.LanguageVersion'11

net/sourceforge/pmd/cli/PMDParameters.java

ViolationLine
Avoid unused imports such as 'com.beust.jcommander.IParameterValidator'11

net/sourceforge/pmd/cpd/CsTokenizer.java

ViolationLine
No need to call String.valueOf to append to a string.54
No need to call String.valueOf to append to a string.62
No need to call String.valueOf to append to a string.62
No need to call String.valueOf to append to a string.67
No need to call String.valueOf to append to a string.67
No need to call String.valueOf to append to a string.85
No need to call String.valueOf to append to a string.85
No need to call String.valueOf to append to a string.103

net/sourceforge/pmd/cpd/JavaTokenizer.java

ViolationLine
Avoid unused local variables such as 'popped'.129
Avoid empty while statements130

net/sourceforge/pmd/dcd/graph/MemberNode.java

ViolationLine
Avoid unused private fields such as 'decoration'.26

net/sourceforge/pmd/lang/ast/AbstractNode.java

ViolationLine
JUnit 4 tests that execute tests should use the @Test annotation111 - 113
JUnit 4 tests that execute tests should use the @Test annotation127 - 129
JUnit 4 tests that execute tests should use the @Test annotation135 - 137
JUnit 4 tests that execute tests should use the @Test annotation143 - 145

net/sourceforge/pmd/lang/ast/xpath/saxon/AbstractNodeInfo.java

ViolationLine
Ensure you override both equals() and hashCode()105

net/sourceforge/pmd/lang/dfa/SequenceChecker.java

ViolationLine
Avoid unused private fields such as 'type'.33

net/sourceforge/pmd/lang/ecmascript/ast/EcmascriptTreeBuilder.java

ViolationLine
Avoid unused local variables such as 'problemStart'.213

net/sourceforge/pmd/lang/java/rule/comments/CommentContentRule.java

ViolationLine
Perhaps 'wordsAreRegex' could be replaced by a local variable.30
Avoid unused private fields such as 'wordsAreRegex'.30

net/sourceforge/pmd/lang/java/rule/comments/CommentRequiredRule.java

ViolationLine
Perhaps 'label' could be replaced by a local variable.17

net/sourceforge/pmd/lang/java/rule/comments/HeaderCommentsRule.java

ViolationLine
Perhaps 'label' could be replaced by a local variable.30

net/sourceforge/pmd/lang/java/rule/logging/MoreThanOneLoggerRule.java

ViolationLine
A catch statement should never catch throwable since it includes errors.27
A catch statement should never catch throwable since it includes errors.33

net/sourceforge/pmd/lang/java/typeresolution/ClassTypeResolver.java

ViolationLine
A catch statement should never catch throwable since it includes errors.663

net/sourceforge/pmd/lang/java/typeresolution/PMDASMClassLoader.java

ViolationLine
Avoid unused imports such as 'java.util.Collections'8