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'. | 77 | 
 
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. | 110 | 
| Useless parentheses. | 116 | 
| Useless parentheses. | 128 | 
| Useless parentheses. | 128 | 
| These nested if statements could be combined | 211–214 | 
| Useless parentheses. | 371 | 
| Useless parentheses. | 371 | 
| Useless parentheses. | 379 | 
| Useless parentheses. | 389 | 
| Useless parentheses. | 403 | 
| Useless parentheses. | 433 | 
| Useless parentheses. | 463 | 
| Useless parentheses. | 463 | 
| Useless parentheses. | 469 | 
| Useless parentheses. | 469 | 
| Useless parentheses. | 477 | 
| Useless parentheses. | 481 | 
| Useless parentheses. | 503 | 
| These nested if statements could be combined | 593–610 | 
| These nested if statements could be combined | 615–619 | 
| These nested if statements could be combined | 624–629 | 
| Avoid empty if statements | 713–717 | 
 
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 | 139 | 
 
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. | 157 | 
| Useless parentheses. | 158 | 
 
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 |