PMD Results

The following document contains the results of PMD 4.2.5.

Files

net/sf/sshapi/vfs/SftpFileObject.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method63
A method should have only one exit point, and that should be the last statement in the method67
Assigning an Object to null is a code smell. Consider refactoring.78
Found 'DU'-anomaly for variable 'sftp' (lines '126'-'138').126 - 138
Avoid using final local variables, turn them into fields142
Use explicit scoping instead of the default package private level196 - 210
Assigning an Object to null is a code smell. Consider refactoring.228

net/sf/sshapi/vfs/SftpFileProvider.java

ViolationLine
Found 'DU'-anomaly for variable 'ssh' (lines '69'-'76').69 - 76

net/sf/sshapi/vfs/SftpFileSystem.java

ViolationLine
Use explicit scoping instead of the default package private level47
Assigning an Object to null is a code smell. Consider refactoring.63
Assigning an Object to null is a code smell. Consider refactoring.68
Found 'DU'-anomaly for variable 'ssh' (lines '83'-'112').83 - 112
Avoid using final local variables, turn them into fields99
Assigning an Object to null is a code smell. Consider refactoring.102
Avoid empty catch blocks120 - 121

net/sf/sshapi/vfs/SftpFileSystemConfigBuilder.java

ViolationLine
It is a good practice to call super() in a constructor51 - 52

net/sf/sshapi/vfs/SftpRandomAccessContent.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.113
Assigning an Object to null is a code smell. Consider refactoring.115
Do not use the short type171
Do not use the short type173

net/sf/sshapi/vfs/SshClientFactory.java

ViolationLine
Use explicit scoping instead of the default package private level47
A method should have only one exit point, and that should be the last statement in the method75
Found 'DU'-anomaly for variable 'con' (lines '92'-'115').92 - 115