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