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 | 106–111 |
org/apache/maven/artifact/installer/DefaultArtifactInstaller.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 84–89 |
org/apache/maven/artifact/repository/DefaultArtifactRepository.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'uniqueVersion'. | 73 |
org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 121–129 |
| CollapsibleIfStatements | These nested if statements could be combined | 364–372 |
org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
| Rule | Violation | Line |
|---|---|---|
| AvoidThreadGroup | Avoid using java.lang.ThreadGroup; it is not thread safe | 534 |
org/apache/maven/artifact/resolver/WarningResolutionListener.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 31 |
org/apache/maven/project/interpolation/RegexBasedModelInterpolator.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'envars'. | 42 |
org/apache/maven/project/path/DefaultPathTranslator.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 138–140 |
org/apache/maven/repository/DefaultMirrorSelector.java
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingHardCodedIP | Do not hard code the IP address | 136 |
org/apache/maven/repository/MetadataResolutionResult.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'dirtyTree'. | 63 |
| UnusedPrivateField | Avoid unused private fields such as 'resolvedGraph'. | 67 |
org/apache/maven/repository/legacy/DefaultWagonManager.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'protocol'. | 655 |
org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 172–179 |
org/apache/maven/repository/metadata/ArtifactMetadata.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'af'. | 171 |
org/apache/maven/repository/metadata/DefaultGraphConflictResolver.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty else statement | 93–99 |
| UnusedFormalParameter | Avoid unused method parameters such as 'v'. | 185 |
org/apache/maven/repository/metadata/MetadataGraph.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'nVertices'. | 217 |
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. | 212 |
| UselessParentheses | Useless parentheses. | 213 |
| UselessParentheses | Useless parentheses. | 251 |
org/apache/maven/project/interpolation/AbstractStringBasedModelInterpolator.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 278 |
| UselessParentheses | Useless parentheses. | 280 |
org/apache/maven/repository/legacy/DefaultUpdateCheckManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 93 |
| UselessParentheses | Useless parentheses. | 128 |
org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 256 |
| UselessParentheses | Useless parentheses. | 256 |
| UselessParentheses | Useless parentheses. | 347 |
| UselessParentheses | Useless parentheses. | 347 |
| UselessParentheses | Useless parentheses. | 571 |
| UselessParentheses | Useless parentheses. | 577 |
| UselessParentheses | Useless parentheses. | 584 |
| UselessParentheses | Useless parentheses. | 585 |
| UselessParentheses | Useless parentheses. | 618 |
org/apache/maven/repository/legacy/resolver/transform/SnapshotTransformation.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 135 |
| UselessParentheses | Useless parentheses. | 136–137 |
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 | 106–111 |
org/apache/maven/artifact/installer/DefaultArtifactInstaller.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 84–89 |
org/apache/maven/artifact/repository/DefaultArtifactRepository.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'uniqueVersion'. | 3 | 73 |
org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 121–129 |
| UselessParentheses | Useless parentheses. | 4 | 212 |
| UselessParentheses | Useless parentheses. | 4 | 213 |
| UselessParentheses | Useless parentheses. | 4 | 251 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 364–372 |
org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidThreadGroup | Avoid using java.lang.ThreadGroup; it is not thread safe | 3 | 534 |
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/interpolation/AbstractStringBasedModelInterpolator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 278 |
| UselessParentheses | Useless parentheses. | 4 | 280 |
org/apache/maven/project/interpolation/RegexBasedModelInterpolator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'envars'. | 3 | 42 |
org/apache/maven/project/path/DefaultPathTranslator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 138–140 |
org/apache/maven/repository/DefaultMirrorSelector.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidUsingHardCodedIP | Do not hard code the IP address | 3 | 136 |
org/apache/maven/repository/MetadataResolutionResult.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'dirtyTree'. | 3 | 63 |
| UnusedPrivateField | Avoid unused private fields such as 'resolvedGraph'. | 3 | 67 |
org/apache/maven/repository/legacy/DefaultUpdateCheckManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 93 |
| UselessParentheses | Useless parentheses. | 4 | 128 |
org/apache/maven/repository/legacy/DefaultWagonManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'protocol'. | 3 | 655 |
org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 172–179 |
| UselessParentheses | Useless parentheses. | 4 | 256 |
| UselessParentheses | Useless parentheses. | 4 | 256 |
| UselessParentheses | Useless parentheses. | 4 | 347 |
| UselessParentheses | Useless parentheses. | 4 | 347 |
| UselessParentheses | Useless parentheses. | 4 | 571 |
| UselessParentheses | Useless parentheses. | 4 | 577 |
| UselessParentheses | Useless parentheses. | 4 | 584 |
| UselessParentheses | Useless parentheses. | 4 | 585 |
| UselessParentheses | Useless parentheses. | 4 | 618 |
org/apache/maven/repository/legacy/resolver/transform/SnapshotTransformation.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 135 |
| UselessParentheses | Useless parentheses. | 4 | 136–137 |
org/apache/maven/repository/metadata/ArtifactMetadata.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'af'. | 3 | 171 |
org/apache/maven/repository/metadata/DefaultGraphConflictResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty else statement | 3 | 93–99 |
| UnusedFormalParameter | Avoid unused method parameters such as 'v'. | 3 | 185 |
org/apache/maven/repository/metadata/MetadataGraph.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'nVertices'. | 3 | 217 |
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 |