PMD Results

The following document contains the results of PMD 5.0.2.

Files

org/andromda/maven/plugin/bootstrap/AndroMDAMojo.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 42
Found non-transient, non-static member. Please mark as transient or provide accessors. 63
Found non-transient, non-static member. Please mark as transient or provide accessors. 71
Found non-transient, non-static member. Please mark as transient or provide accessors. 79
Found non-transient, non-static member. Please mark as transient or provide accessors. 96
Found non-transient, non-static member. Please mark as transient or provide accessors. 105
Found non-transient, non-static member. Please mark as transient or provide accessors. 112
Line too long 137
Local variable 'repositoryCount' could be declared final 139
Deeply nested if..then statements are hard to read 143 - 158
A catch statement should never catch throwable since it includes errors. 178
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 182
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 182
Parameter 'configurationUri' is not assigned and could be declared final 193
Parameter 'project' is not assigned and could be declared final 201
Parameter 'settings' is not assigned and could be declared final 211
Parameter 'propertyFiles' is not assigned and could be declared final 221

org/andromda/maven/plugin/bootstrap/TranslationLibraryTestFormatter.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 25
Private field 'report' could be made final; it is only initialized in the declaration or constructor. 30
Found non-transient, non-static member. Please mark as transient or provide accessors. 30
Found non-transient, non-static member. Please mark as transient or provide accessors. 35
Private field 'reportWriter' could be made final; it is only initialized in the declaration or constructor. 35
Avoid using redundant field initializer for 'numberOfErrors' 60
Found non-transient, non-static member. Please mark as transient or provide accessors. 60
Parameter 'test' is not assigned and could be declared final 66
Parameter 'throwable' is not assigned and could be declared final 67
Avoid using redundant field initializer for 'numberOfFailures' 78
Found non-transient, non-static member. Please mark as transient or provide accessors. 78
Parameter 'test' is not assigned and could be declared final 84
Parameter 'failure' is not assigned and could be declared final 85
Parameter 'test' is not assigned and could be declared final 96
Document empty method 97 - 98
Found non-transient, non-static member. Please mark as transient or provide accessors. 103
Avoid using redundant field initializer for 'numberOfTests' 103
Parameter 'test' is not assigned and could be declared final 108
Found non-transient, non-static member. Please mark as transient or provide accessors. 116
Avoid using redundant field initializer for 'startTime' 116
Variables that are final and static should be all capitals, 'newLine' is not all capitals. 121
Parameter 'test' is not assigned and could be declared final 145
Parameter 'throwable' is not assigned and could be declared final 146
Found non-transient, non-static member. Please mark as transient or provide accessors. 153
Private field 'failures' could be made final; it is only initialized in the declaration or constructor. 153
Use explicit scoping instead of the default package private level 161 - 207
Useless parentheses. 163
No need to call String.valueOf to append to a string. 164
StringBuffer constructor is initialized with size 16, but has at least 40 characters appended. 164
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 165
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 166
Avoid throwing raw exception types. 203
Use explicit scoping instead of the default package private level 214 - 215
Found non-transient, non-static member. Please mark as transient or provide accessors. 215
Use explicit scoping instead of the default package private level 216
Found non-transient, non-static member. Please mark as transient or provide accessors. 216

org/andromda/maven/plugin/bootstrap/TranslationLibraryTestMojo.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 36
Line too long 39
Found non-transient, non-static member. Please mark as transient or provide accessors. 44
Found non-transient, non-static member. Please mark as transient or provide accessors. 51
Found non-transient, non-static member. Please mark as transient or provide accessors. 60
Found non-transient, non-static member. Please mark as transient or provide accessors. 68
Found non-transient, non-static member. Please mark as transient or provide accessors. 75
Found non-transient, non-static member. Please mark as transient or provide accessors. 80
Found non-transient, non-static member. Please mark as transient or provide accessors. 90
Found non-transient, non-static member. Please mark as transient or provide accessors. 97
Found non-transient, non-static member. Please mark as transient or provide accessors. 106
Found non-transient, non-static member. Please mark as transient or provide accessors. 113
Found non-transient, non-static member. Please mark as transient or provide accessors. 120
Line too long 123
Found non-transient, non-static member. Please mark as transient or provide accessors. 127
Found non-transient, non-static member. Please mark as transient or provide accessors. 134
Avoid if (x != y) ..; else ..; 142 - 206
A catch statement should never catch throwable since it includes errors. 183
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 185