PMD Results
The following document contains the results of PMD 5.6.1.
 
Files
org/apache/maven/artifact/deployer/DefaultArtifactDeployer.java
| Violation | Line | 
| Avoid empty if statements | 115–117 | 
 
org/apache/maven/artifact/installer/DefaultArtifactInstaller.java
| Violation | Line | 
| Avoid empty if statements | 96–98 | 
 
org/apache/maven/artifact/repository/DefaultArtifactRepository.java
| Violation | Line | 
| Avoid unused constructor parameters such as 'uniqueVersion'. | 79 | 
 
org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
| Violation | Line | 
| These nested if statements could be combined | 141–153 | 
| Useless parentheses. | 252 | 
| Useless parentheses. | 252 | 
| Useless parentheses. | 296 | 
| These nested if statements could be combined | 441–453 | 
 
org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
| Violation | Line | 
| Avoid using java.lang.ThreadGroup; it is not thread safe | 578 | 
 
org/apache/maven/artifact/resolver/WarningResolutionListener.java
| Violation | Line | 
| Avoid unused private fields such as 'logger'. | 34 | 
 
org/apache/maven/project/ModelUtils.java
| Violation | Line | 
| Useless parentheses. | 57 | 
| Useless parentheses. | 57 | 
| Useless parentheses. | 65 | 
| Useless parentheses. | 79 | 
| Useless parentheses. | 99 | 
| Useless parentheses. | 104 | 
| Useless parentheses. | 117 | 
| Useless parentheses. | 192 | 
| Useless parentheses. | 192 | 
| Useless parentheses. | 203 | 
| Useless parentheses. | 203 | 
| Useless parentheses. | 220 | 
| Useless parentheses. | 224 | 
| Useless parentheses. | 237 | 
| Useless parentheses. | 251 | 
| Useless parentheses. | 288 | 
 
org/apache/maven/project/inheritance/DefaultModelInheritanceAssembler.java
| Violation | Line | 
| Useless parentheses. | 109 | 
| Useless parentheses. | 115 | 
| Useless parentheses. | 127 | 
| Useless parentheses. | 127 | 
| These nested if statements could be combined | 210–213 | 
| Useless parentheses. | 369 | 
| Useless parentheses. | 369 | 
| Useless parentheses. | 377 | 
| Useless parentheses. | 387 | 
| Useless parentheses. | 401 | 
| Useless parentheses. | 431 | 
| Useless parentheses. | 461 | 
| Useless parentheses. | 461 | 
| Useless parentheses. | 467 | 
| Useless parentheses. | 467 | 
| Useless parentheses. | 475 | 
| Useless parentheses. | 479 | 
| Useless parentheses. | 501 | 
| These nested if statements could be combined | 590–607 | 
| These nested if statements could be combined | 612–616 | 
| These nested if statements could be combined | 621–626 | 
| Avoid empty if statements | 710–714 | 
 
org/apache/maven/project/interpolation/RegexBasedModelInterpolator.java
| Violation | Line | 
| Avoid unused constructor parameters such as 'envars'. | 50 | 
 
org/apache/maven/project/path/DefaultPathTranslator.java
| Violation | Line | 
| These nested if statements could be combined | 168–171 | 
 
org/apache/maven/repository/DefaultMirrorSelector.java
| Violation | Line | 
| Do not hard code the IP address | 163 | 
 
org/apache/maven/repository/MetadataResolutionResult.java
| Violation | Line | 
| Avoid unused private fields such as 'dirtyTree'. | 66 | 
| Avoid unused private fields such as 'resolvedGraph'. | 70 | 
 
org/apache/maven/repository/legacy/DefaultUpdateCheckManager.java
| Violation | Line | 
| Useless parentheses. | 110 | 
| Useless parentheses. | 150 | 
 
org/apache/maven/repository/legacy/DefaultWagonManager.java
| Violation | Line | 
| Avoid unused method parameters such as 'protocol'. | 803 | 
 
org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
| Violation | Line | 
| These nested if statements could be combined | 183–191 | 
| Useless parentheses. | 217 | 
| Useless parentheses. | 217 | 
| Useless parentheses. | 276 | 
| Useless parentheses. | 276 | 
| Useless parentheses. | 379 | 
| Useless parentheses. | 379 | 
| Useless parentheses. | 635 | 
| Useless parentheses. | 642 | 
| Useless parentheses. | 648 | 
| Useless parentheses. | 649 | 
| Useless parentheses. | 686 | 
 
org/apache/maven/repository/legacy/resolver/transform/SnapshotTransformation.java
| Violation | Line | 
| Useless parentheses. | 158 | 
| Useless parentheses. | 159 | 
 
org/apache/maven/repository/metadata/ArtifactMetadata.java
| Violation | Line | 
| Avoid unused constructor parameters such as 'af'. | 149 | 
 
org/apache/maven/repository/metadata/DefaultGraphConflictResolver.java
| Violation | Line | 
| Avoid empty if statements | 110–116 | 
| Avoid unused method parameters such as 'v'. | 207 | 
 
org/apache/maven/repository/metadata/MetadataGraph.java
| Violation | Line | 
| Avoid unused method parameters such as 'nVertices'. | 244 | 
 
org/apache/maven/repository/metadata/MetadataGraphEdge.java
| Violation | Line | 
| Ensure you override both equals() and hashCode() | 85 | 
| Useless parentheses. | 184 | 
| Useless parentheses. | 185 |