PMD Results

The following document contains the results of PMD.

Summary

FilesErrors
1015

Files

FilesViolations
org/apache/commons/jelly/tags/define/AttributeTag.java 2
org/apache/commons/jelly/tags/define/BeanTag.java 2
org/apache/commons/jelly/tags/define/DefineTagSupport.java 1
org/apache/commons/jelly/tags/define/DynaBeanTag.java 2
org/apache/commons/jelly/tags/define/InvokeBodyTag.java 2
org/apache/commons/jelly/tags/define/InvokeTag.java 1
org/apache/commons/jelly/tags/define/JellyBeanTag.java 2
org/apache/commons/jelly/tags/define/MyRunnable.java 1
org/apache/commons/jelly/tags/define/TestJelly.java 1
org/apache/commons/jelly/tags/mock/TestJUnit.java 1

org/apache/commons/jelly/tags/define/AttributeTag.java

ViolationLine
Avoid unused private fields such as 'log' 40
Private field could be made final. It is only initialized in the declaration or constructor. 43

org/apache/commons/jelly/tags/define/BeanTag.java

ViolationLine
Avoid unused local variables such as 'classLoader' 91
Avoid if (x != y) ..; else ..; 105

org/apache/commons/jelly/tags/define/DefineTagSupport.java

ViolationLine
This abstract class does not have any abstract methods 29

org/apache/commons/jelly/tags/define/DynaBeanTag.java

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

org/apache/commons/jelly/tags/define/InvokeBodyTag.java

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

org/apache/commons/jelly/tags/define/InvokeTag.java

ViolationLine
Avoid unused private fields such as 'log' 37

org/apache/commons/jelly/tags/define/JellyBeanTag.java

ViolationLine
Avoid unused private fields such as 'log' 38
Avoid empty 'if' statements 81

org/apache/commons/jelly/tags/define/MyRunnable.java

ViolationLine
Private field could be made final. It is only initialized in the declaration or constructor. 40

org/apache/commons/jelly/tags/define/TestJelly.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. 38

org/apache/commons/jelly/tags/mock/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. 38