PMD Results
The following document contains the results of PMD 6.38.0.
Violations By Priority
Priority 3
org/apache/maven/DefaultMaven.java
| Rule | Violation | Line |
|---|---|---|
| ReturnFromFinallyBlock | Avoid returning from a finally block | 311 |
| UnusedFormalParameter | Avoid unused method parameters such as 'result'. | 532 |
org/apache/maven/artifact/repository/metadata/io/DefaultMetadataReader.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'in': resource specifications are implicitly final | 60 |
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'in': resource specifications are implicitly final | 75 |
org/apache/maven/bridge/MavenRepositorySystem.java
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingHardCodedIP | Do not hard code the IP address | 836 |
org/apache/maven/configuration/BasedirBeanConfigurationPathTranslator.java
| Rule | Violation | Line |
|---|---|---|
| EmptyIfStmt | Avoid empty if statements | 53–55 |
org/apache/maven/execution/MavenSession.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'eventDispatcher'. | 337 |
org/apache/maven/execution/SettingsAdapter.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'runtimeInfo'. | 48 |
org/apache/maven/lifecycle/LifecycleExecutionException.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'execution'. | 57 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'execution'. | 63 |
org/apache/maven/lifecycle/internal/DefaultLifecycleExecutionPlanCalculator.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'pluginVersionResolver'. | 75 |
org/apache/maven/lifecycle/internal/LifecycleModuleBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'projectExecutionListeners'. | 67 |
org/apache/maven/lifecycle/internal/LifecycleStarter.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'sessionScope'. | 70 |
org/apache/maven/lifecycle/internal/builder/BuilderCommon.java
| Rule | Violation | Line |
|---|---|---|
| EmptyIfStmt | Avoid empty if statements | 163–165 |
org/apache/maven/lifecycle/internal/builder/multithreaded/MultiThreadedBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'muxer'. | 191 |
org/apache/maven/plugin/CycleDetectedInPluginGraphException.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'e'. | 35 |
org/apache/maven/plugin/DefaultBuildPluginManager.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'mojoExecutionListeners'. | 64 |
org/apache/maven/plugin/PluginConfigurationException.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'pluginDescriptor'. | 33 |
| UnusedPrivateField | Avoid unused private fields such as 'originalMessage'. | 35 |
org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'project'. | 90 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'pathTranslator'. | 90 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'logger'. | 90 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'properties'. | 91 |
| EmptyIfStmt | Avoid empty if statements | 468–470 |
org/apache/maven/plugin/internal/DefaultPluginDependenciesResolver.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 113–120 |
org/apache/maven/project/DefaultProjectBuildingHelper.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'container'. | 72 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 366–370 |
| CollapsibleIfStatements | These nested if statements could be combined | 1574–1577 |
| CollapsibleIfStatements | These nested if statements could be combined | 1599–1603 |
| CollapsibleIfStatements | These nested if statements could be combined | 1753–1756 |
| CollapsibleIfStatements | These nested if statements could be combined | 1773–1776 |
org/apache/maven/toolchain/building/DefaultToolchainsBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'request'. | 158 |
org/apache/maven/toolchain/io/DefaultToolchainsReader.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'in': resource specifications are implicitly final | 64 |
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'in': resource specifications are implicitly final | 80 |
org/apache/maven/toolchain/io/DefaultToolchainsWriter.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'out': resource specifications are implicitly final | 49 |
org/apache/maven/toolchain/java/DefaultJavaToolChain.java
| Rule | Violation | Line |
|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 44–47 |
| UselessOverridingMethod | Overriding method merely calls super | 50–53 |
Priority 4
org/apache/maven/ReactorReader.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 253 |
| UselessParentheses | Useless parentheses. | 254 |
org/apache/maven/RepositoryUtils.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 65 |
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.*' | 30 |
org/apache/maven/artifact/repository/LegacyLocalRepositoryManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 111 |
| UselessParentheses | Useless parentheses. | 232 |
org/apache/maven/execution/SettingsAdapter.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 60 |
org/apache/maven/lifecycle/internal/DependencyContext.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 96–97 |
org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 171 |
| UselessParentheses | Useless parentheses. | 171 |
| UselessParentheses | Useless parentheses. | 409 |
| UselessParentheses | Useless parentheses. | 409 |
| UselessParentheses | Useless parentheses. | 409 |
| UselessParentheses | Useless parentheses. | 409 |
| UselessParentheses | Useless parentheses. | 447–448 |
| UselessParentheses | Useless parentheses. | 453 |
org/apache/maven/plugin/internal/DefaultMavenPluginManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 531 |
| UselessParentheses | Useless parentheses. | 531 |
org/apache/maven/plugin/internal/DefaultPluginDependenciesResolver.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.eclipse.aether.artifact.Artifact' due to existing import 'org.eclipse.aether.artifact.Artifact' | 254 |
org/apache/maven/project/DefaultProjectBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 786 |
| UselessParentheses | Useless parentheses. | 786 |
| UselessParentheses | Useless parentheses. | 786 |
| UselessParentheses | Useless parentheses. | 787 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 458 |
| UselessParentheses | Useless parentheses. | 458 |
| UselessParentheses | Useless parentheses. | 503 |
| UselessParentheses | Useless parentheses. | 503 |
| UselessParentheses | Useless parentheses. | 995 |
| UselessParentheses | Useless parentheses. | 1093 |
| UselessParentheses | Useless parentheses. | 1093 |
| UselessParentheses | Useless parentheses. | 1262 |
| UselessParentheses | Useless parentheses. | 1276 |
| UselessParentheses | Useless parentheses. | 1288 |
| UselessParentheses | Useless parentheses. | 1293 |
| UselessParentheses | Useless parentheses. | 1298 |
| UselessParentheses | Useless parentheses. | 1614 |
| UselessParentheses | Useless parentheses. | 1663 |
| UselessParentheses | Useless parentheses. | 1691 |
| UselessParentheses | Useless parentheses. | 1787 |
org/apache/maven/project/ProjectSorter.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 244 |
org/apache/maven/project/artifact/MavenMetadataSource.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 487 |
| UselessParentheses | Useless parentheses. | 487 |
org/apache/maven/repository/ArtifactTransferEvent.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 309 |
| UselessParentheses | Useless parentheses. | 309 |
Files
org/apache/maven/DefaultMaven.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ReturnFromFinallyBlock | Avoid returning from a finally block | 3 | 311 |
| UnusedFormalParameter | Avoid unused method parameters such as 'result'. | 3 | 532 |
org/apache/maven/ReactorReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 253 |
| UselessParentheses | Useless parentheses. | 4 | 254 |
org/apache/maven/RepositoryUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 65 |
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 | 30 |
org/apache/maven/artifact/repository/LegacyLocalRepositoryManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 111 |
| UselessParentheses | Useless parentheses. | 4 | 232 |
org/apache/maven/artifact/repository/metadata/io/DefaultMetadataReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'in': resource specifications are implicitly final | 3 | 60 |
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'in': resource specifications are implicitly final | 3 | 75 |
org/apache/maven/bridge/MavenRepositorySystem.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidUsingHardCodedIP | Do not hard code the IP address | 3 | 836 |
org/apache/maven/configuration/BasedirBeanConfigurationPathTranslator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyIfStmt | Avoid empty if statements | 3 | 53–55 |
org/apache/maven/execution/MavenSession.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'eventDispatcher'. | 3 | 337 |
org/apache/maven/execution/SettingsAdapter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'runtimeInfo'. | 3 | 48 |
| UselessParentheses | Useless parentheses. | 4 | 60 |
org/apache/maven/lifecycle/LifecycleExecutionException.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'execution'. | 3 | 57 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'execution'. | 3 | 63 |
org/apache/maven/lifecycle/internal/DefaultLifecycleExecutionPlanCalculator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'pluginVersionResolver'. | 3 | 75 |
org/apache/maven/lifecycle/internal/DependencyContext.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 96–97 |
org/apache/maven/lifecycle/internal/LifecycleModuleBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'projectExecutionListeners'. | 3 | 67 |
org/apache/maven/lifecycle/internal/LifecycleStarter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'sessionScope'. | 3 | 70 |
org/apache/maven/lifecycle/internal/builder/BuilderCommon.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyIfStmt | Avoid empty if statements | 3 | 163–165 |
org/apache/maven/lifecycle/internal/builder/multithreaded/MultiThreadedBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'muxer'. | 3 | 191 |
org/apache/maven/plugin/CycleDetectedInPluginGraphException.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'e'. | 3 | 35 |
org/apache/maven/plugin/DefaultBuildPluginManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'mojoExecutionListeners'. | 3 | 64 |
org/apache/maven/plugin/PluginConfigurationException.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'pluginDescriptor'. | 3 | 33 |
| UnusedPrivateField | Avoid unused private fields such as 'originalMessage'. | 3 | 35 |
org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'project'. | 3 | 90 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'pathTranslator'. | 3 | 90 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'logger'. | 3 | 90 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'properties'. | 3 | 91 |
| UselessParentheses | Useless parentheses. | 4 | 171 |
| UselessParentheses | Useless parentheses. | 4 | 171 |
| UselessParentheses | Useless parentheses. | 4 | 409 |
| UselessParentheses | Useless parentheses. | 4 | 409 |
| UselessParentheses | Useless parentheses. | 4 | 409 |
| UselessParentheses | Useless parentheses. | 4 | 409 |
| UselessParentheses | Useless parentheses. | 4 | 447–448 |
| UselessParentheses | Useless parentheses. | 4 | 453 |
| EmptyIfStmt | Avoid empty if statements | 3 | 468–470 |
org/apache/maven/plugin/internal/DefaultMavenPluginManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 531 |
| UselessParentheses | Useless parentheses. | 4 | 531 |
org/apache/maven/plugin/internal/DefaultPluginDependenciesResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 113–120 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.eclipse.aether.artifact.Artifact' due to existing import 'org.eclipse.aether.artifact.Artifact' | 4 | 254 |
org/apache/maven/project/DefaultProjectBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 786 |
| UselessParentheses | Useless parentheses. | 4 | 786 |
| UselessParentheses | Useless parentheses. | 4 | 786 |
| UselessParentheses | Useless parentheses. | 4 | 787 |
org/apache/maven/project/DefaultProjectBuildingHelper.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'container'. | 3 | 72 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 366–370 |
| UselessParentheses | Useless parentheses. | 4 | 458 |
| UselessParentheses | Useless parentheses. | 4 | 458 |
| UselessParentheses | Useless parentheses. | 4 | 503 |
| UselessParentheses | Useless parentheses. | 4 | 503 |
| UselessParentheses | Useless parentheses. | 4 | 995 |
| UselessParentheses | Useless parentheses. | 4 | 1093 |
| UselessParentheses | Useless parentheses. | 4 | 1093 |
| UselessParentheses | Useless parentheses. | 4 | 1262 |
| UselessParentheses | Useless parentheses. | 4 | 1276 |
| UselessParentheses | Useless parentheses. | 4 | 1288 |
| UselessParentheses | Useless parentheses. | 4 | 1293 |
| UselessParentheses | Useless parentheses. | 4 | 1298 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 1574–1577 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 1599–1603 |
| UselessParentheses | Useless parentheses. | 4 | 1614 |
| UselessParentheses | Useless parentheses. | 4 | 1663 |
| UselessParentheses | Useless parentheses. | 4 | 1691 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 1753–1756 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 1773–1776 |
| UselessParentheses | Useless parentheses. | 4 | 1787 |
org/apache/maven/project/ProjectSorter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 244 |
org/apache/maven/project/artifact/MavenMetadataSource.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 487 |
| UselessParentheses | Useless parentheses. | 4 | 487 |
org/apache/maven/repository/ArtifactTransferEvent.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 309 |
| UselessParentheses | Useless parentheses. | 4 | 309 |
org/apache/maven/toolchain/building/DefaultToolchainsBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'request'. | 3 | 158 |
org/apache/maven/toolchain/io/DefaultToolchainsReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'in': resource specifications are implicitly final | 3 | 64 |
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'in': resource specifications are implicitly final | 3 | 80 |
org/apache/maven/toolchain/io/DefaultToolchainsWriter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'out': resource specifications are implicitly final | 3 | 49 |
org/apache/maven/toolchain/java/DefaultJavaToolChain.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 3 | 44–47 |
| UselessOverridingMethod | Overriding method merely calls super | 3 | 50–53 |