PMD Results
The following document contains the results of PMD 5.0.5.
Files
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 |
144–156 |
These nested if statements could be combined |
463–475 |
org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
Violation |
Line |
Avoid using java.lang.ThreadGroup; it is not thread safe |
556 |
org/apache/maven/artifact/resolver/WarningResolutionListener.java
Violation |
Line |
Avoid unused private fields such as 'logger'. |
34 |
org/apache/maven/profiles/activation/ProfileActivator.java
Violation |
Line |
Avoid modifiers which are implied by the context |
27 |
org/apache/maven/project/inheritance/DefaultModelInheritanceAssembler.java
Violation |
Line |
Avoid unused imports such as 'java.util.Collection' |
23 |
Avoid unused imports such as 'java.util.Iterator' |
24 |
These nested if statements could be combined |
210–213 |
These nested if statements could be combined |
592–609 |
These nested if statements could be combined |
614–618 |
These nested if statements could be combined |
623–628 |
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 |
169–172 |
org/apache/maven/repository/DefaultMirrorSelector.java
Violation |
Line |
Do not hard code the IP address |
136 |
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/DefaultWagonManager.java
Violation |
Line |
Avoid unused method parameters such as 'protocol'. |
764 |
org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
Violation |
Line |
These nested if statements could be combined |
179–187 |
org/apache/maven/repository/metadata/ArtifactMetadata.java
Violation |
Line |
Avoid unused constructor parameters such as 'af'. |
151 |
org/apache/maven/repository/metadata/DefaultGraphConflictResolver.java
Violation |
Line |
Avoid unused method parameters such as 'v'. |
207 |
org/apache/maven/repository/metadata/MetadataGraph.java
Violation |
Line |
Avoid unused method parameters such as 'nVertices'. |
243 |
org/apache/maven/repository/metadata/MetadataGraphEdge.java
Violation |
Line |
Ensure you override both equals() and hashCode() |
84 |