PMD Results
The following document contains the results of PMD 6.49.0.
Violations By Priority
Priority 3
org/apache/maven/artifact/deployer/DefaultArtifactDeployer.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 107–112 |
org/apache/maven/artifact/installer/DefaultArtifactInstaller.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 85–90 |
org/apache/maven/artifact/repository/DefaultArtifactRepository.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'uniqueVersion'. | 74 |
org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 122–130 |
| CollapsibleIfStatements | These nested if statements could be combined | 365–373 |
org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
| Rule | Violation | Line |
|---|---|---|
| AvoidThreadGroup | Avoid using java.lang.ThreadGroup; it is not thread safe | 544 |
org/apache/maven/artifact/resolver/WarningResolutionListener.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 31 |
org/apache/maven/project/inheritance/DefaultModelInheritanceAssembler.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 179–181 |
| CollapsibleIfStatements | These nested if statements could be combined | 495–509 |
| CollapsibleIfStatements | These nested if statements could be combined | 513–516 |
| CollapsibleIfStatements | These nested if statements could be combined | 520–524 |
| EmptyControlStatement | Empty if statement | 596–602 |
org/apache/maven/project/interpolation/RegexBasedModelInterpolator.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'envars'. | 43 |
org/apache/maven/project/path/DefaultPathTranslator.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 139–141 |
org/apache/maven/repository/DefaultMirrorSelector.java
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingHardCodedIP | Do not hard code the IP address | 137 |
org/apache/maven/repository/MetadataResolutionResult.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'dirtyTree'. | 64 |
| UnusedPrivateField | Avoid unused private fields such as 'resolvedGraph'. | 68 |
org/apache/maven/repository/legacy/DefaultWagonManager.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'protocol'. | 656 |
org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 173–180 |
org/apache/maven/repository/metadata/ArtifactMetadata.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'af'. | 172 |
org/apache/maven/repository/metadata/DefaultGraphConflictResolver.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty else statement | 94–100 |
| UnusedFormalParameter | Avoid unused method parameters such as 'v'. | 187 |
org/apache/maven/repository/metadata/MetadataGraph.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'nVertices'. | 218 |
org/apache/maven/repository/metadata/MetadataGraphEdge.java
| Rule | Violation | Line |
|---|---|---|
| OverrideBothEqualsAndHashcode | Ensure you override both equals() and hashCode() | 79 |
Priority 4
org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 213 |
| UselessParentheses | Useless parentheses. | 214 |
| UselessParentheses | Useless parentheses. | 252 |
org/apache/maven/project/ModelUtils.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 54 |
| UselessParentheses | Useless parentheses. | 54 |
| UselessParentheses | Useless parentheses. | 61 |
| UselessParentheses | Useless parentheses. | 72 |
| UselessParentheses | Useless parentheses. | 90 |
| UselessParentheses | Useless parentheses. | 93 |
| UselessParentheses | Useless parentheses. | 105 |
| UselessParentheses | Useless parentheses. | 167 |
| UselessParentheses | Useless parentheses. | 167 |
| UselessParentheses | Useless parentheses. | 176 |
| UselessParentheses | Useless parentheses. | 176 |
| UselessParentheses | Useless parentheses. | 192 |
| UselessParentheses | Useless parentheses. | 196 |
| UselessParentheses | Useless parentheses. | 207 |
| UselessParentheses | Useless parentheses. | 218 |
| UselessParentheses | Useless parentheses. | 249 |
org/apache/maven/project/inheritance/DefaultModelInheritanceAssembler.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 97 |
| UselessParentheses | Useless parentheses. | 102 |
| UselessParentheses | Useless parentheses. | 113 |
| UselessParentheses | Useless parentheses. | 113 |
| UselessParentheses | Useless parentheses. | 312 |
| UselessParentheses | Useless parentheses. | 312 |
| UselessParentheses | Useless parentheses. | 319 |
| UselessParentheses | Useless parentheses. | 327 |
| UselessParentheses | Useless parentheses. | 339 |
| UselessParentheses | Useless parentheses. | 365 |
| UselessParentheses | Useless parentheses. | 389 |
| UselessParentheses | Useless parentheses. | 389 |
| UselessParentheses | Useless parentheses. | 394 |
| UselessParentheses | Useless parentheses. | 394 |
| UselessParentheses | Useless parentheses. | 401 |
| UselessParentheses | Useless parentheses. | 405 |
| UselessParentheses | Useless parentheses. | 422 |
org/apache/maven/project/interpolation/AbstractStringBasedModelInterpolator.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 279 |
| UselessParentheses | Useless parentheses. | 281 |
org/apache/maven/repository/legacy/DefaultUpdateCheckManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 94 |
| UselessParentheses | Useless parentheses. | 129 |
org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 203 |
| UselessParentheses | Useless parentheses. | 203 |
| UselessParentheses | Useless parentheses. | 257 |
| UselessParentheses | Useless parentheses. | 257 |
| UselessParentheses | Useless parentheses. | 348 |
| UselessParentheses | Useless parentheses. | 348 |
| UselessParentheses | Useless parentheses. | 572 |
| UselessParentheses | Useless parentheses. | 578 |
| UselessParentheses | Useless parentheses. | 585 |
| UselessParentheses | Useless parentheses. | 586 |
| UselessParentheses | Useless parentheses. | 619 |
org/apache/maven/repository/legacy/resolver/transform/SnapshotTransformation.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 136 |
| UselessParentheses | Useless parentheses. | 137–138 |
org/apache/maven/repository/metadata/MetadataGraphEdge.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 160 |
| UselessParentheses | Useless parentheses. | 161 |
Files
org/apache/maven/artifact/deployer/DefaultArtifactDeployer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 107–112 |
org/apache/maven/artifact/installer/DefaultArtifactInstaller.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 85–90 |
org/apache/maven/artifact/repository/DefaultArtifactRepository.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'uniqueVersion'. | 3 | 74 |
org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 122–130 |
| UselessParentheses | Useless parentheses. | 4 | 213 |
| UselessParentheses | Useless parentheses. | 4 | 214 |
| UselessParentheses | Useless parentheses. | 4 | 252 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 365–373 |
org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidThreadGroup | Avoid using java.lang.ThreadGroup; it is not thread safe | 3 | 544 |
org/apache/maven/artifact/resolver/WarningResolutionListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 3 | 31 |
org/apache/maven/project/ModelUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 54 |
| UselessParentheses | Useless parentheses. | 4 | 54 |
| UselessParentheses | Useless parentheses. | 4 | 61 |
| UselessParentheses | Useless parentheses. | 4 | 72 |
| UselessParentheses | Useless parentheses. | 4 | 90 |
| UselessParentheses | Useless parentheses. | 4 | 93 |
| UselessParentheses | Useless parentheses. | 4 | 105 |
| UselessParentheses | Useless parentheses. | 4 | 167 |
| UselessParentheses | Useless parentheses. | 4 | 167 |
| UselessParentheses | Useless parentheses. | 4 | 176 |
| UselessParentheses | Useless parentheses. | 4 | 176 |
| UselessParentheses | Useless parentheses. | 4 | 192 |
| UselessParentheses | Useless parentheses. | 4 | 196 |
| UselessParentheses | Useless parentheses. | 4 | 207 |
| UselessParentheses | Useless parentheses. | 4 | 218 |
| UselessParentheses | Useless parentheses. | 4 | 249 |
org/apache/maven/project/inheritance/DefaultModelInheritanceAssembler.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 97 |
| UselessParentheses | Useless parentheses. | 4 | 102 |
| UselessParentheses | Useless parentheses. | 4 | 113 |
| UselessParentheses | Useless parentheses. | 4 | 113 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 179–181 |
| UselessParentheses | Useless parentheses. | 4 | 312 |
| UselessParentheses | Useless parentheses. | 4 | 312 |
| UselessParentheses | Useless parentheses. | 4 | 319 |
| UselessParentheses | Useless parentheses. | 4 | 327 |
| UselessParentheses | Useless parentheses. | 4 | 339 |
| UselessParentheses | Useless parentheses. | 4 | 365 |
| UselessParentheses | Useless parentheses. | 4 | 389 |
| UselessParentheses | Useless parentheses. | 4 | 389 |
| UselessParentheses | Useless parentheses. | 4 | 394 |
| UselessParentheses | Useless parentheses. | 4 | 394 |
| UselessParentheses | Useless parentheses. | 4 | 401 |
| UselessParentheses | Useless parentheses. | 4 | 405 |
| UselessParentheses | Useless parentheses. | 4 | 422 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 495–509 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 513–516 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 520–524 |
| EmptyControlStatement | Empty if statement | 3 | 596–602 |
org/apache/maven/project/interpolation/AbstractStringBasedModelInterpolator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 279 |
| UselessParentheses | Useless parentheses. | 4 | 281 |
org/apache/maven/project/interpolation/RegexBasedModelInterpolator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'envars'. | 3 | 43 |
org/apache/maven/project/path/DefaultPathTranslator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 139–141 |
org/apache/maven/repository/DefaultMirrorSelector.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidUsingHardCodedIP | Do not hard code the IP address | 3 | 137 |
org/apache/maven/repository/MetadataResolutionResult.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'dirtyTree'. | 3 | 64 |
| UnusedPrivateField | Avoid unused private fields such as 'resolvedGraph'. | 3 | 68 |
org/apache/maven/repository/legacy/DefaultUpdateCheckManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 94 |
| UselessParentheses | Useless parentheses. | 4 | 129 |
org/apache/maven/repository/legacy/DefaultWagonManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'protocol'. | 3 | 656 |
org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 173–180 |
| UselessParentheses | Useless parentheses. | 4 | 203 |
| UselessParentheses | Useless parentheses. | 4 | 203 |
| UselessParentheses | Useless parentheses. | 4 | 257 |
| UselessParentheses | Useless parentheses. | 4 | 257 |
| UselessParentheses | Useless parentheses. | 4 | 348 |
| UselessParentheses | Useless parentheses. | 4 | 348 |
| UselessParentheses | Useless parentheses. | 4 | 572 |
| UselessParentheses | Useless parentheses. | 4 | 578 |
| UselessParentheses | Useless parentheses. | 4 | 585 |
| UselessParentheses | Useless parentheses. | 4 | 586 |
| UselessParentheses | Useless parentheses. | 4 | 619 |
org/apache/maven/repository/legacy/resolver/transform/SnapshotTransformation.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 136 |
| UselessParentheses | Useless parentheses. | 4 | 137–138 |
org/apache/maven/repository/metadata/ArtifactMetadata.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'af'. | 3 | 172 |
org/apache/maven/repository/metadata/DefaultGraphConflictResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty else statement | 3 | 94–100 |
| UnusedFormalParameter | Avoid unused method parameters such as 'v'. | 3 | 187 |
org/apache/maven/repository/metadata/MetadataGraph.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'nVertices'. | 3 | 218 |
org/apache/maven/repository/metadata/MetadataGraphEdge.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| OverrideBothEqualsAndHashcode | Ensure you override both equals() and hashCode() | 3 | 79 |
| UselessParentheses | Useless parentheses. | 4 | 160 |
| UselessParentheses | Useless parentheses. | 4 | 161 |