PMD Results

The following document contains the results of PMD.

Summary

FilesErrors
1017

Files

FilesViolations
org/apache/commons/jelly/tags/junit/AssertFileContainsTag.java 1
org/apache/commons/jelly/tags/junit/AssertFileExistsTag.java 1
org/apache/commons/jelly/tags/junit/AssertFileNotFoundTag.java 1
org/apache/commons/jelly/tags/junit/AssertTag.java 2
org/apache/commons/jelly/tags/junit/AssertTagSupport.java 3
org/apache/commons/jelly/tags/junit/AssertThrowsTag.java 2
org/apache/commons/jelly/tags/junit/JUnitTagLibrary.java 3
org/apache/commons/jelly/tags/junit/JellyTestSuite.java 1
org/apache/commons/jelly/tags/junit/RunTag.java 2
org/apache/commons/jelly/tags/junit/TestJUnit.java 1

org/apache/commons/jelly/tags/junit/AssertFileContainsTag.java

ViolationLine
Avoid unused imports such as 'java.io.FileNotFoundException' 20

org/apache/commons/jelly/tags/junit/AssertFileExistsTag.java

ViolationLine
Avoid unused imports such as 'org.apache.commons.jelly.expression.Expression' 23

org/apache/commons/jelly/tags/junit/AssertFileNotFoundTag.java

ViolationLine
Avoid unused imports such as 'org.apache.commons.jelly.expression.Expression' 23

org/apache/commons/jelly/tags/junit/AssertTag.java

ViolationLine
Avoid unused private fields such as 'log' 39
Avoid if (x != y) ..; else ..; 56

org/apache/commons/jelly/tags/junit/AssertTagSupport.java

ViolationLine
This abstract class does not have any abstract methods 27
Use equals() to compare object references. 326
Use equals() to compare object references. 348

org/apache/commons/jelly/tags/junit/AssertThrowsTag.java

ViolationLine
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 69
Avoid if (x != y) ..; else ..; 79

org/apache/commons/jelly/tags/junit/JUnitTagLibrary.java

ViolationLine
Avoid duplicate imports such as 'org.apache.commons.jelly.TagLibrary' 21
Avoid unused imports such as 'org.jaxen.JaxenException' 30
Private field could be made final. It is only initialized in the declaration or constructor. 40

org/apache/commons/jelly/tags/junit/JellyTestSuite.java

ViolationLine
This abstract class does not have any abstract methods 33

org/apache/commons/jelly/tags/junit/RunTag.java

ViolationLine
The String literal "message" appears 4 times in this file; the first occurrence is on line 148 148
The String literal "stack" appears 4 times in this file; the first occurrence is on line 152 152

org/apache/commons/jelly/tags/junit/TestJUnit.java

ViolationLine
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 36