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–660 |
org/apache/maven/surefire/api/testset/ResolvedTest.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–413 |
org/apache/maven/surefire/api/testset/TestListResolver.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–425 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 158 |
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 | 97 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 157–160 |
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()`. | 181 |
| UselessParentheses | Useless parentheses around `memento.getByteBuffer()`. | 185 |
| UselessParentheses | Useless parentheses around `memento.getByteBuffer()`. | 279 |
| UselessParentheses | Useless parentheses around `output`. | 287 |
| UselessParentheses | Useless parentheses around `output`. | 311 |
| UselessParentheses | Useless parentheses around `output`. | 312 |
| UselessParentheses | Useless parentheses around `output`. | 316 |
| UselessParentheses | Useless parentheses around `input`. | 329 |
| UselessParentheses | Useless parentheses around `input`. | 330 |
| UselessParentheses | Useless parentheses around `input`. | 330 |
| UselessParentheses | Useless parentheses around `input`. | 334 |
| UselessParentheses | Useless parentheses around `input`. | 338 |
| UselessParentheses | Useless parentheses around `buffer.compact()`. | 424 |
org/apache/maven/surefire/api/util/TempFileManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `new File(prefix)`. | 142 |
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–660 |
| UselessParentheses | Useless parentheses around `memento.getCharBuffer()`. | 4 | 181 |
| UselessParentheses | Useless parentheses around `memento.getByteBuffer()`. | 4 | 185 |
| UselessParentheses | Useless parentheses around `memento.getByteBuffer()`. | 4 | 279 |
| UselessParentheses | Useless parentheses around `output`. | 4 | 287 |
| UselessParentheses | Useless parentheses around `output`. | 4 | 311 |
| UselessParentheses | Useless parentheses around `output`. | 4 | 312 |
| UselessParentheses | Useless parentheses around `output`. | 4 | 316 |
| UselessParentheses | Useless parentheses around `input`. | 4 | 329 |
| UselessParentheses | Useless parentheses around `input`. | 4 | 330 |
| UselessParentheses | Useless parentheses around `input`. | 4 | 330 |
| UselessParentheses | Useless parentheses around `input`. | 4 | 334 |
| UselessParentheses | Useless parentheses around `input`. | 4 | 338 |
| UselessParentheses | Useless parentheses around `buffer.compact()`. | 4 | 424 |
org/apache/maven/surefire/api/testset/ResolvedTest.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–413 |
org/apache/maven/surefire/api/testset/TestListResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–425 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 158 |
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 | 97 |
| UselessParentheses | Useless parentheses around `new File(prefix)`. | 4 | 142 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 157–160 |