PMD Results
The following document contains the results of PMD 6.55.0.
Violations By Priority
Priority 3
org/apache/maven/artifact/deployer/DefaultArtifactDeployer.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 109–114 |
org/apache/maven/artifact/installer/DefaultArtifactInstaller.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 88–93 |
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 | 124–132 |
| CollapsibleIfStatements | These nested if statements could be combined | 363–371 |
org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
| Rule | Violation | Line |
|---|---|---|
| AvoidThreadGroup | Avoid using java.lang.ThreadGroup; it is not thread safe | 538 |
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 | 142–144 |
org/apache/maven/repository/DefaultMirrorSelector.java
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingHardCodedIP | Do not hard code the IP address | 139 |
org/apache/maven/repository/legacy/DefaultWagonManager.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'protocol'. | 675 |
org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 176–183 |
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 | 97–103 |
| UnusedFormalParameter | Avoid unused method parameters such as 'v'. | 189 |
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/metadata/AbstractArtifactMetadata.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.apache.maven.artifact.metadata.ArtifactMetadata' due to existing same package import 'org.apache.maven.artifact.metadata.*' | 29 |
org/apache/maven/artifact/repository/LegacyLocalRepositoryManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 105 |
| UselessParentheses | Useless parentheses. | 202 |
org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 215 |
| UselessParentheses | Useless parentheses. | 216 |
| UselessParentheses | Useless parentheses. | 254 |
org/apache/maven/project/artifact/MavenMetadataSource.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 468 |
| UselessParentheses | Useless parentheses. | 468 |
org/apache/maven/project/interpolation/AbstractStringBasedModelInterpolator.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 280 |
| UselessParentheses | Useless parentheses. | 282 |
org/apache/maven/repository/ArtifactTransferEvent.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 279 |
| UselessParentheses | Useless parentheses. | 279 |
org/apache/maven/repository/DefaultMirrorSelector.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 178 |
org/apache/maven/repository/legacy/DefaultUpdateCheckManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 98 |
| UselessParentheses | Useless parentheses. | 133 |
org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 260 |
| UselessParentheses | Useless parentheses. | 260 |
| UselessParentheses | Useless parentheses. | 351 |
| UselessParentheses | Useless parentheses. | 351 |
| UselessParentheses | Useless parentheses. | 575 |
| UselessParentheses | Useless parentheses. | 581 |
| UselessParentheses | Useless parentheses. | 588 |
| UselessParentheses | Useless parentheses. | 589 |
| UselessParentheses | Useless parentheses. | 622 |
org/apache/maven/repository/legacy/resolver/transform/SnapshotTransformation.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 137 |
| UselessParentheses | Useless parentheses. | 138–139 |
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 | 109–114 |
org/apache/maven/artifact/installer/DefaultArtifactInstaller.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 88–93 |
org/apache/maven/artifact/metadata/AbstractArtifactMetadata.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.apache.maven.artifact.metadata.ArtifactMetadata' due to existing same package import 'org.apache.maven.artifact.metadata.*' | 4 | 29 |
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/LegacyLocalRepositoryManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 105 |
| UselessParentheses | Useless parentheses. | 4 | 202 |
org/apache/maven/artifact/repository/metadata/DefaultRepositoryMetadataManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 124–132 |
| UselessParentheses | Useless parentheses. | 4 | 215 |
| UselessParentheses | Useless parentheses. | 4 | 216 |
| UselessParentheses | Useless parentheses. | 4 | 254 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 363–371 |
org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidThreadGroup | Avoid using java.lang.ThreadGroup; it is not thread safe | 3 | 538 |
org/apache/maven/project/artifact/MavenMetadataSource.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 468 |
| UselessParentheses | Useless parentheses. | 4 | 468 |
org/apache/maven/project/interpolation/AbstractStringBasedModelInterpolator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 280 |
| UselessParentheses | Useless parentheses. | 4 | 282 |
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 | 142–144 |
org/apache/maven/repository/ArtifactTransferEvent.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 279 |
| UselessParentheses | Useless parentheses. | 4 | 279 |
org/apache/maven/repository/DefaultMirrorSelector.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidUsingHardCodedIP | Do not hard code the IP address | 3 | 139 |
| UselessParentheses | Useless parentheses. | 4 | 178 |
org/apache/maven/repository/legacy/DefaultUpdateCheckManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 98 |
| UselessParentheses | Useless parentheses. | 4 | 133 |
org/apache/maven/repository/legacy/DefaultWagonManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'protocol'. | 3 | 675 |
org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 176–183 |
| UselessParentheses | Useless parentheses. | 4 | 260 |
| UselessParentheses | Useless parentheses. | 4 | 260 |
| UselessParentheses | Useless parentheses. | 4 | 351 |
| UselessParentheses | Useless parentheses. | 4 | 351 |
| UselessParentheses | Useless parentheses. | 4 | 575 |
| UselessParentheses | Useless parentheses. | 4 | 581 |
| UselessParentheses | Useless parentheses. | 4 | 588 |
| UselessParentheses | Useless parentheses. | 4 | 589 |
| UselessParentheses | Useless parentheses. | 4 | 622 |
org/apache/maven/repository/legacy/resolver/transform/SnapshotTransformation.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 137 |
| UselessParentheses | Useless parentheses. | 4 | 138–139 |
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 | 97–103 |
| UnusedFormalParameter | Avoid unused method parameters such as 'v'. | 3 | 189 |
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 |