PMD Results

The following document contains the results of PMD 4.3.

Files

com/swtdesigner/FocusTraversalOnArray.java

ViolationLine
Consider simply returning the value vs storing it in local variable 'next'33
Use equals() to compare object references.40

openr66/r66gui/ProgressDirectTransfer.java

ViolationLine
Private field 'progressBar' could be made final; it is only initialized in the declaration or constructor.35
Private field 'textFieldStatus' could be made final; it is only initialized in the declaration or constructor.36

openr66/r66gui/R66ClientGui.java

ViolationLine
Private field 'environnement' could be made final; it is only initialized in the declaration or constructor.75
Perhaps 'scrollPane' could be replaced by a local variable.87
Perhaps 'scrollPane_1' could be replaced by a local variable.88
Overridable method 'showDialog' called during object construction113
Avoid if (x != y) ..; else ..;405 - 409
These nested if statements could be combined532 - 540

openr66/r66gui/R66Environment.java

ViolationLine
Avoid if (x != y) ..; else ..;154 - 155
Avoid if (x != y) ..; else ..;162 - 163
Avoid empty catch blocks241 - 242
Avoid empty catch blocks264 - 265