PMD Results
The following document contains the results of PMD 7.17.0.
Violations By Priority
Priority 3
org/apache/maven/surefire/api/report/ConsoleOutputCapture.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–200 |
org/apache/maven/surefire/api/stream/AbstractStreamDecoder.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–650 |
org/apache/maven/surefire/api/testset/ResolvedTest.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–412 |
org/apache/maven/surefire/api/testset/TestListResolver.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–426 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 159 |
org/apache/maven/surefire/api/util/ReflectionUtils.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 153–154 |
org/apache/maven/surefire/api/util/TempFileManager.java
| Rule | Violation | Line |
|---|---|---|
| AvoidThreadGroup | Avoid using java.lang.ThreadGroup; it is not thread safe | 96 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 156–159 |
Priority 4
org/apache/maven/surefire/api/booter/MasterProcessCommand.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MasterProcessCommand': 'values' is already in scope | 81 |
org/apache/maven/surefire/api/stream/AbstractStreamDecoder.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `memento.getCharBuffer()`. | 162 |
| UselessParentheses | Useless parentheses around `memento.getByteBuffer()`. | 166 |
| UselessParentheses | Useless parentheses around `memento.getByteBuffer()`. | 260 |
| UselessParentheses | Useless parentheses around `output`. | 268 |
| UselessParentheses | Useless parentheses around `output`. | 292 |
| UselessParentheses | Useless parentheses around `output`. | 293 |
| UselessParentheses | Useless parentheses around `output`. | 297 |
| UselessParentheses | Useless parentheses around `input`. | 310 |
| UselessParentheses | Useless parentheses around `input`. | 311 |
| UselessParentheses | Useless parentheses around `input`. | 311 |
| UselessParentheses | Useless parentheses around `input`. | 315 |
| UselessParentheses | Useless parentheses around `input`. | 319 |
| UselessParentheses | Useless parentheses around `buffer.compact()`. | 408 |
org/apache/maven/surefire/api/util/TempFileManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `new File(prefix)`. | 141 |
Files
org/apache/maven/surefire/api/booter/MasterProcessCommand.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MasterProcessCommand': 'values' is already in scope | 4 | 81 |
org/apache/maven/surefire/api/report/ConsoleOutputCapture.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–200 |
org/apache/maven/surefire/api/stream/AbstractStreamDecoder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–650 |
| UselessParentheses | Useless parentheses around `memento.getCharBuffer()`. | 4 | 162 |
| UselessParentheses | Useless parentheses around `memento.getByteBuffer()`. | 4 | 166 |
| UselessParentheses | Useless parentheses around `memento.getByteBuffer()`. | 4 | 260 |
| UselessParentheses | Useless parentheses around `output`. | 4 | 268 |
| UselessParentheses | Useless parentheses around `output`. | 4 | 292 |
| UselessParentheses | Useless parentheses around `output`. | 4 | 293 |
| UselessParentheses | Useless parentheses around `output`. | 4 | 297 |
| UselessParentheses | Useless parentheses around `input`. | 4 | 310 |
| UselessParentheses | Useless parentheses around `input`. | 4 | 311 |
| UselessParentheses | Useless parentheses around `input`. | 4 | 311 |
| UselessParentheses | Useless parentheses around `input`. | 4 | 315 |
| UselessParentheses | Useless parentheses around `input`. | 4 | 319 |
| UselessParentheses | Useless parentheses around `buffer.compact()`. | 4 | 408 |
org/apache/maven/surefire/api/testset/ResolvedTest.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–412 |
org/apache/maven/surefire/api/testset/TestListResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–426 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 159 |
org/apache/maven/surefire/api/util/ReflectionUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 153–154 |
org/apache/maven/surefire/api/util/TempFileManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidThreadGroup | Avoid using java.lang.ThreadGroup; it is not thread safe | 3 | 96 |
| UselessParentheses | Useless parentheses around `new File(prefix)`. | 4 | 141 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 156–159 |