PMD Results

The following document contains the results of PMD 4.3.

Files

org/apache/maven/artifact/deployer/DefaultArtifactDeployer.java

ViolationLine
Avoid empty if statements112 - 114

org/apache/maven/artifact/installer/DefaultArtifactInstaller.java

ViolationLine
Avoid empty if statements96 - 98

org/apache/maven/artifact/repository/DefaultArtifactRepository.java

ViolationLine
Avoid unused constructor parameters such as 'uniqueVersion'.77 - 81

org/apache/maven/artifact/repository/metadata/MetadataBridge.java

ViolationLine
No need to import a type that lives in the same package27

org/apache/maven/artifact/resolver/DefaultArtifactResolver.java

ViolationLine
Avoid using java.lang.ThreadGroup; it is not thread safe536

org/apache/maven/artifact/resolver/WarningResolutionListener.java

ViolationLine
Avoid unused private fields such as 'logger'.34

org/apache/maven/profiles/DefaultProfileManager.java

ViolationLine
Avoid unused imports such as 'org.apache.maven.model.profile.ProfileActivationContext'29

org/apache/maven/profiles/ProfilesConversionUtils.java

ViolationLine
Avoid using java.lang.ThreadGroup; it is not thread safe73
Avoid using java.lang.ThreadGroup; it is not thread safe147

org/apache/maven/project/inheritance/DefaultModelInheritanceAssembler.java

ViolationLine
These nested if statements could be combined209 - 212
These nested if statements could be combined602 - 619
These nested if statements could be combined624 - 628
These nested if statements could be combined633 - 638
Avoid empty if statements716 - 718
Avoid empty if statements722 - 726

org/apache/maven/project/interpolation/RegexBasedModelInterpolator.java

ViolationLine
Avoid unused constructor parameters such as 'envars'.50 - 52

org/apache/maven/project/path/DefaultPathTranslator.java

ViolationLine
These nested if statements could be combined169 - 172

org/apache/maven/repository/DefaultMirrorSelector.java

ViolationLine
Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 !135

org/apache/maven/repository/MetadataResolutionResult.java

ViolationLine
Avoid unused private fields such as 'dirtyTree'.66
Avoid unused private fields such as 'resolvedGraph'.70

org/apache/maven/repository/legacy/DefaultWagonManager.java

ViolationLine
Avoid unused method parameters such as 'protocol'.730 - 741

org/apache/maven/repository/legacy/LegacyRepositorySystem.java

ViolationLine
Avoid using java.lang.ThreadGroup; it is not thread safe626

org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java

ViolationLine
These nested if statements could be combined179 - 187

org/apache/maven/repository/metadata/ArtifactMetadata.java

ViolationLine
Avoid unused constructor parameters such as 'af'.152 - 160

org/apache/maven/repository/metadata/DefaultClasspathTransformation.java

ViolationLine
Avoid unused local variables such as 'md'.73

org/apache/maven/repository/metadata/DefaultGraphConflictResolver.java

ViolationLine
Avoid empty if statements110 - 116
Avoid unused method parameters such as 'v'.207 - 246

org/apache/maven/repository/metadata/MetadataGraph.java

ViolationLine
Avoid unused method parameters such as 'nVertices'.243 - 249

org/apache/maven/repository/metadata/MetadataGraphEdge.java

ViolationLine
Ensure you override both equals() and hashCode()84