PMD Results

The following document contains the results of PMD.

Summary

FilesErrors
1437

Files

FilesViolations
org/apache/commons/jelly/tags/jface/ActionTag.java 1
org/apache/commons/jelly/tags/jface/ContributionItemTag.java 1
org/apache/commons/jelly/tags/jface/JFaceImageTag.java 2
org/apache/commons/jelly/tags/jface/JFaceLayoutDataTag.java 1
org/apache/commons/jelly/tags/jface/JFaceLayoutTag.java 1
org/apache/commons/jelly/tags/jface/JFaceTagLibrary.java 19
org/apache/commons/jelly/tags/jface/JFaceWidgetTag.java 1
org/apache/commons/jelly/tags/jface/ViewerTag.java 2
org/apache/commons/jelly/tags/jface/preference/PreferenceDialogTag.java 1
org/apache/commons/jelly/tags/jface/window/ApplicationWindowTag.java 2
org/apache/commons/jelly/tags/jface/wizard/WizardDialogTag.java 2
org/apache/commons/jelly/tags/jface/JFaceDemo.java 2
org/apache/commons/jelly/tags/jface/preference/PreferenceDemo.java 1
org/apache/commons/jelly/tags/jface/wizard/WizardDemo.java 1

org/apache/commons/jelly/tags/jface/ActionTag.java

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

org/apache/commons/jelly/tags/jface/ContributionItemTag.java

ViolationLine
No need to check for null before an instanceof 45

org/apache/commons/jelly/tags/jface/JFaceImageTag.java

ViolationLine
No need to check for null before an instanceof 67
Avoid if (x != y) ..; else ..; 77

org/apache/commons/jelly/tags/jface/JFaceLayoutDataTag.java

ViolationLine
No need to check for null before an instanceof 48

org/apache/commons/jelly/tags/jface/JFaceLayoutTag.java

ViolationLine
No need to check for null before an instanceof 49

org/apache/commons/jelly/tags/jface/JFaceTagLibrary.java

ViolationLine
Overridable method 'registerViewerTag' called during construction 55
Overridable method 'registerViewerTag' called during construction 56
Overridable method 'registerViewerTag' called during construction 57
Overridable method 'registerViewerTag' called during construction 58
Overridable method 'registerWindowTag' called during construction 65
Overridable method 'registerActionTag' called during construction 71
Overridable method 'registerContributionItemTag' called during construction 74
Overridable method 'registerWizardDialogTag' called during construction 77
Overridable method 'registerWizardPageTag' called during construction 78
Overridable method 'registerPreferenceDialogTag' called during construction 81
Overridable method 'registerFieldEditorTag' called during construction 83
Overridable method 'registerFieldEditorTag' called during construction 84
Overridable method 'registerFieldEditorTag' called during construction 85
Overridable method 'registerFieldEditorTag' called during construction 86
Overridable method 'registerFieldEditorTag' called during construction 87
Overridable method 'registerFieldEditorTag' called during construction 88
Overridable method 'registerFieldEditorTag' called during construction 91
Avoid unused formal parameters such as 'theClass' 99
A value of 23 may denote a high amount of coupling within the class 276

org/apache/commons/jelly/tags/jface/JFaceWidgetTag.java

ViolationLine
No need to check for null before an instanceof 88

org/apache/commons/jelly/tags/jface/ViewerTag.java

ViolationLine
Avoid unused private fields such as 'parent' 35
Avoid unused private fields such as 'style' 36

org/apache/commons/jelly/tags/jface/preference/PreferenceDialogTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 58

org/apache/commons/jelly/tags/jface/window/ApplicationWindowTag.java

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

org/apache/commons/jelly/tags/jface/wizard/WizardDialogTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 63
Avoid if (x != y) ..; else ..; 77

org/apache/commons/jelly/tags/jface/JFaceDemo.java

ViolationLine
No need to import a type that's in the same package 22
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 50

org/apache/commons/jelly/tags/jface/preference/PreferenceDemo.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. 50

org/apache/commons/jelly/tags/jface/wizard/WizardDemo.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. 50