PMD Results
The following document contains the results of PMD 4.2.5.
Files
org/apache/maven/artifact/deployer/DefaultArtifactDeployer.java
Violation | Line |
---|
Avoid empty if statements | 112 - 114 |
org/apache/maven/artifact/installer/DefaultArtifactInstaller.java
Violation | Line |
---|
Avoid empty if statements | 96 - 98 |
org/apache/maven/artifact/repository/DefaultArtifactRepository.java
Violation | Line |
---|
Avoid unused constructor parameters such as 'uniqueVersion'. | 77 - 81 |
org/apache/maven/artifact/repository/metadata/MetadataBridge.java
Violation | Line |
---|
No need to import a type that lives in the same package | 27 |
org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
Violation | Line |
---|
Avoid using ThreadGroup; it is not thread safe | 536 |
org/apache/maven/artifact/resolver/WarningResolutionListener.java
Violation | Line |
---|
Avoid unused private fields such as 'logger'. | 34 |
org/apache/maven/profiles/DefaultProfileManager.java
Violation | Line |
---|
Avoid unused imports such as 'org.apache.maven.model.profile.ProfileActivationContext' | 29 |
org/apache/maven/project/inheritance/DefaultModelInheritanceAssembler.java
Violation | Line |
---|
These nested if statements could be combined | 209 - 212 |
These nested if statements could be combined | 602 - 619 |
These nested if statements could be combined | 624 - 628 |
These nested if statements could be combined | 633 - 638 |
Avoid empty if statements | 716 - 718 |
Avoid empty if statements | 722 - 726 |
org/apache/maven/project/interpolation/RegexBasedModelInterpolator.java
Violation | Line |
---|
Avoid unused constructor parameters such as 'envars'. | 50 - 52 |
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 IPv4 or IPv6 addresses, even 127.0.0.1 ! | 135 |
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'. | 730 - 741 |
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'. | 152 - 160 |
org/apache/maven/repository/metadata/DefaultClasspathTransformation.java
Violation | Line |
---|
Avoid unused local variables such as 'md'. | 73 |
org/apache/maven/repository/metadata/DefaultGraphConflictResolver.java
Violation | Line |
---|
Avoid empty if statements | 110 - 116 |
Avoid unused method parameters such as 'v'. | 207 - 246 |
org/apache/maven/repository/metadata/MetadataGraph.java
Violation | Line |
---|
Avoid unused method parameters such as 'nVertices'. | 243 - 249 |
org/apache/maven/repository/metadata/MetadataGraphEdge.java
Violation | Line |
---|
Ensure you override both equals() and hashCode() | 84 |