PMD Results
The following document contains the results of PMD 6.55.0.
Violations By Priority
Priority 2
org/apache/maven/buildcache/CacheUtils.java
| Rule | Violation | Line |
|---|---|---|
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 151 |
Priority 3
org/apache/maven/buildcache/CacheControllerImpl.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–924 |
org/apache/maven/buildcache/CacheDiff.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'config'. | 46 |
org/apache/maven/buildcache/LocalCacheRepositoryImpl.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–419 |
| CollapsibleIfStatements | These nested if statements could be combined | 315–317 |
org/apache/maven/buildcache/checksum/DigestUtils.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–163 |
org/apache/maven/buildcache/checksum/MavenProjectInput.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–723 |
| EmptyControlStatement | Empty if statement | 537–565 |
org/apache/maven/buildcache/hash/CloseableBuffer.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 126–127 |
| EmptyCatchBlock | Avoid empty catch blocks | 153–154 |
Priority 4
org/apache/maven/buildcache/BuildCacheMojosExecutionStrategy.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 354 |
org/apache/maven/buildcache/CacheControllerImpl.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 191 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'StringUtils.isNotBlank' due to existing static import 'org.apache.commons.lang3.StringUtils.isNotBlank' | 758 |
org/apache/maven/buildcache/DefaultNormalizedModelProvider.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.apache.maven.model.Build' due to existing import 'org.apache.maven.model.Build' | 87 |
org/apache/maven/buildcache/checksum/MavenProjectInput.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 541 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'CacheUtils.isPom' due to existing static import 'org.apache.maven.buildcache.CacheUtils.isPom' | 604 |
Files
org/apache/maven/buildcache/BuildCacheMojosExecutionStrategy.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 354 |
org/apache/maven/buildcache/CacheControllerImpl.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–924 |
| UselessParentheses | Useless parentheses. | 4 | 191 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'StringUtils.isNotBlank' due to existing static import 'org.apache.commons.lang3.StringUtils.isNotBlank' | 4 | 758 |
org/apache/maven/buildcache/CacheDiff.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'config'. | 3 | 46 |
org/apache/maven/buildcache/CacheUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 151 |
org/apache/maven/buildcache/DefaultNormalizedModelProvider.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.apache.maven.model.Build' due to existing import 'org.apache.maven.model.Build' | 4 | 87 |
org/apache/maven/buildcache/LocalCacheRepositoryImpl.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–419 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 315–317 |
org/apache/maven/buildcache/checksum/DigestUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–163 |
org/apache/maven/buildcache/checksum/MavenProjectInput.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–723 |
| EmptyControlStatement | Empty if statement | 3 | 537–565 |
| UselessParentheses | Useless parentheses. | 4 | 541 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'CacheUtils.isPom' due to existing static import 'org.apache.maven.buildcache.CacheUtils.isPom' | 4 | 604 |
org/apache/maven/buildcache/hash/CloseableBuffer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 126–127 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 153–154 |
Suppressed Violations
| Filename | Rule message | Suppression type | Reason |
|---|---|---|---|
| org/apache/maven/buildcache/BuildCacheMojosExecutionStrategy.java | Avoid unused method parameters such as 'mojoExecutionRunner'. | annotation | |
| org/apache/maven/buildcache/CacheControllerImpl.java | Avoid unused constructor parameters such as 'repoSystem'. | annotation | |
| org/apache/maven/buildcache/CacheControllerImpl.java | Avoid unused constructor parameters such as 'session'. | annotation | |
| org/apache/maven/buildcache/CacheControllerImpl.java | Avoid unused method parameters such as 'project'. | annotation |