PMD Results

The following document contains the results of PMD 4.3.

Files

goldengate/snmp/GgMOFactory.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.47 - 205

goldengate/snmp/GgSnmpAgent.java

ViolationLine
Private field 'address' could be made final; it is only initialized in the declaration or constructor.79
Private field 'nbThread' could be made final; it is only initialized in the declaration or constructor.82
Perhaps 'nbThread' could be replaced by a local variable.82
Private field 'isFilterAccessEnabled' could be made final; it is only initialized in the declaration or constructor.84
Private field 'useTrap' could be made final; it is only initialized in the declaration or constructor.86
Private field 'listUsmUser' could be made final; it is only initialized in the declaration or constructor.90
Private field 'listTargetElements' could be made final; it is only initialized in the declaration or constructor.92
Private field 'hasV2' could be made final; it is only initialized in the declaration or constructor.94
Private field 'hasV3' could be made final; it is only initialized in the declaration or constructor.96
Private field 'workerPool' could be made final; it is only initialized in the declaration or constructor.100

goldengate/snmp/interf/GgCounter32.java

ViolationLine
An empty method in an abstract class should be abstract instead80 - 82

goldengate/snmp/interf/GgGauge32.java

ViolationLine
An empty method in an abstract class should be abstract instead80 - 82

goldengate/snmp/interf/GgTimeTicks.java

ViolationLine
An empty method in an abstract class should be abstract instead80 - 82

goldengate/snmp/SnmpConfiguration.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.60 - 648
Switch statements should have a default label550 - 563
Avoid instantiation through private constructors from outside of the constructor's class.597 - 601

goldengate/snmp/test/GgSimpleSnmpClient.java

ViolationLine
Private field 'address' could be made final; it is only initialized in the declaration or constructor.53
Private field 'port' could be made final; it is only initialized in the declaration or constructor.55

goldengate/snmp/test/GgTestSnmpClientAgent.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.43 - 142

goldengate/snmp/utils/GgUptime.java

ViolationLine
Overridable method 'setInternalValue' called during object construction49

goldengate/snmp/utils/MemoryGauge32.java

ViolationLine
Switch statements should have a default label54 - 67
Overridable method 'setInternalValue' called during object construction81