PMD Results
The following document contains the results of PMD 7.0.0.
Violations By Priority
Priority 3
org/apache/maven/cling/invoker/BaseParser.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–483 |
org/apache/maven/cling/invoker/LookupInvoker.java
| Rule | Violation | Line |
|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 333 |
org/apache/maven/cling/invoker/mvnsh/builtin/BuiltinShellCommandRegistryFactory.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 177 |
| UnusedFormalParameter | Avoid unused method parameters such as 'input'. | 181 |
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 202 |
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 240 |
org/apache/maven/cling/logging/impl/UnsupportedSlf4jBindingConfiguration.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'slf4jBinding'. | 38 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'supported'. | 38 |
org/apache/maven/cling/props/MavenProperties.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 358–360 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 371–373 |
org/apache/maven/cling/transfer/ConsoleMavenTransferListener.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'event'. | 139 |
org/apache/maven/cling/transfer/FileSizeFormat.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 73 |
org/apache/maven/cling/utils/CLIReportingUtils.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'ONE_DAY'. | 47 |
Priority 4
org/apache/maven/cling/invoker/mvnenc/CommonsCliEncryptOptions.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Options': 'SOURCE_CLI' is already in scope because it is inherited by an enclosing type | 45 |
org/apache/maven/cling/invoker/mvnsh/CommonsCliShellOptions.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Options': 'SOURCE_CLI' is already in scope because it is inherited by an enclosing type | 42 |
org/apache/maven/cling/logging/Slf4jLoggerManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 53–55 |
org/apache/maven/cling/props/MavenPropertiesLoader.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 124 |
| UselessParentheses | Useless parentheses. | 155 |
org/apache/maven/cling/transfer/AbstractMavenTransferListener.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 72 |
org/apache/maven/cling/transfer/Slf4jMavenTransferListener.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 75 |
org/apache/maven/cling/utils/CLIReportingUtils.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 99 |
| UselessParentheses | Useless parentheses. | 119 |
Files
org/apache/maven/cling/invoker/BaseParser.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–483 |
org/apache/maven/cling/invoker/LookupInvoker.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 333 |
org/apache/maven/cling/invoker/mvnenc/CommonsCliEncryptOptions.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Options': 'SOURCE_CLI' is already in scope because it is inherited by an enclosing type | 4 | 45 |
org/apache/maven/cling/invoker/mvnsh/CommonsCliShellOptions.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Options': 'SOURCE_CLI' is already in scope because it is inherited by an enclosing type | 4 | 42 |
org/apache/maven/cling/invoker/mvnsh/builtin/BuiltinShellCommandRegistryFactory.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 3 | 177 |
| UnusedFormalParameter | Avoid unused method parameters such as 'input'. | 3 | 181 |
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 3 | 202 |
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 3 | 240 |
org/apache/maven/cling/logging/Slf4jLoggerManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 53–55 |
org/apache/maven/cling/logging/impl/UnsupportedSlf4jBindingConfiguration.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'slf4jBinding'. | 3 | 38 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'supported'. | 3 | 38 |
org/apache/maven/cling/props/MavenProperties.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 358–360 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 371–373 |
org/apache/maven/cling/props/MavenPropertiesLoader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 124 |
| UselessParentheses | Useless parentheses. | 4 | 155 |
org/apache/maven/cling/transfer/AbstractMavenTransferListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 72 |
org/apache/maven/cling/transfer/ConsoleMavenTransferListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'event'. | 3 | 139 |
org/apache/maven/cling/transfer/FileSizeFormat.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 73 |
org/apache/maven/cling/transfer/Slf4jMavenTransferListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 75 |
org/apache/maven/cling/utils/CLIReportingUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'ONE_DAY'. | 3 | 47 |
| UselessParentheses | Useless parentheses. | 4 | 99 |
| UselessParentheses | Useless parentheses. | 4 | 119 |