PMD Results
The following document contains the results of PMD 7.17.0.
Violations By Priority
Priority 3
org/apache/maven/surefire/group/parse/GroupMatcherParser.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 82 |
| UnnecessarySemicolon | Unnecessary semicolon | 102 |
| UnnecessarySemicolon | Unnecessary semicolon | 110 |
| UnnecessarySemicolon | Unnecessary semicolon | 129 |
| UnusedPrivateField | Avoid unused private fields such as 'jj_ntk'. | 436 |
| UnusedPrivateMethod | Avoid unused private methods such as 'jj_ntk_f()'. | 600 |
| EmptyCatchBlock | Avoid empty catch blocks | 728 |
org/apache/maven/surefire/group/parse/GroupMatcherParserTokenManager.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'jjStopStringLiteralDfa_0': private methods cannot be overridden | 15 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'jjStartNfa_0': private methods cannot be overridden | 63 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 233 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 280 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 373 |
org/apache/maven/surefire/group/parse/ParseException.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 131–140 |
org/apache/maven/surefire/group/parse/SimpleCharStream.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryReturn | Unnecessary return statement | 123 |
Priority 4
org/apache/maven/surefire/group/parse/GroupMatcherParser.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `new StringReader(expression)`. | 32 |
| UselessParentheses | Useless parentheses around `!jj_3_1()`. | 175 |
| UselessParentheses | Useless parentheses around `!jj_3_2()`. | 183 |
| UselessParentheses | Useless parentheses around `!jj_3_3()`. | 191 |
| UselessParentheses | Useless parentheses around `!jj_3_4()`. | 199 |
| UselessParentheses | Useless parentheses around `!jj_3_5()`. | 207 |
| UselessParentheses | Useless parentheses around `!jj_3_6()`. | 215 |
| UselessParentheses | Useless parentheses around `!jj_3_7()`. | 223 |
| UselessParentheses | Useless parentheses around `!jj_3_8()`. | 231 |
| UselessParentheses | Useless parentheses around `!jj_3_9()`. | 239 |
| UselessParentheses | Useless parentheses around `!jj_3_10()`. | 247 |
| UselessParentheses | Useless parentheses around `!jj_3_11()`. | 255 |
| UselessParentheses | Useless parentheses around `!jj_3_12()`. | 263 |
| UselessParentheses | Useless parentheses around `!jj_3_13()`. | 271 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 453 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 457 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'UnsupportedEncodingException' is already in scope because it is imported in this file | 458 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 467 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 471 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'UnsupportedEncodingException' is already in scope because it is imported in this file | 472 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'Reader' is already in scope because it is imported in this file | 482 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'Reader' is already in scope because it is imported in this file | 492 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.lang': 'Error' is already in scope because it is declared in java.lang | 552 |
| UselessParentheses | Useless parentheses around `jj_ntk = (token.next = token_source.getNextToke...`. | 602 |
| UselessParentheses | Useless parentheses around `jj_ntk = jj_nt.kind`. | 604 |
org/apache/maven/surefire/group/parse/GroupMatcherParserTokenManager.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unused import 'org.apache.maven.surefire.group.match.*' | 4 |
| UnnecessaryImport | Unused import 'java.io.*' | 5 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'PrintStream' is already in scope because it is imported in this file | 12 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'PrintStream' is already in scope because it is imported in this file | 14 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 97 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 121 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 124 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 142 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 145 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 159 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 162 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 176 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 179 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 199 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 299 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 365 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 388 |
| UselessParentheses | Useless parentheses around `lengthOfMatch = jjmatchedPos + 1`. | 416 |
org/apache/maven/surefire/group/parse/SimpleCharStream.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `bufpos = bufsize - tokenBegin`. | 63 |
| UselessParentheses | Useless parentheses around `bufpos = tokenBegin`. | 76 |
| UselessParentheses | Useless parentheses around `column = 1`. | 151 |
| UselessParentheses | Useless parentheses around `column = 1`. | 161 |
Files
org/apache/maven/surefire/group/parse/GroupMatcherParser.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `new StringReader(expression)`. | 4 | 32 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 82 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 102 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 110 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 129 |
| UselessParentheses | Useless parentheses around `!jj_3_1()`. | 4 | 175 |
| UselessParentheses | Useless parentheses around `!jj_3_2()`. | 4 | 183 |
| UselessParentheses | Useless parentheses around `!jj_3_3()`. | 4 | 191 |
| UselessParentheses | Useless parentheses around `!jj_3_4()`. | 4 | 199 |
| UselessParentheses | Useless parentheses around `!jj_3_5()`. | 4 | 207 |
| UselessParentheses | Useless parentheses around `!jj_3_6()`. | 4 | 215 |
| UselessParentheses | Useless parentheses around `!jj_3_7()`. | 4 | 223 |
| UselessParentheses | Useless parentheses around `!jj_3_8()`. | 4 | 231 |
| UselessParentheses | Useless parentheses around `!jj_3_9()`. | 4 | 239 |
| UselessParentheses | Useless parentheses around `!jj_3_10()`. | 4 | 247 |
| UselessParentheses | Useless parentheses around `!jj_3_11()`. | 4 | 255 |
| UselessParentheses | Useless parentheses around `!jj_3_12()`. | 4 | 263 |
| UselessParentheses | Useless parentheses around `!jj_3_13()`. | 4 | 271 |
| UnusedPrivateField | Avoid unused private fields such as 'jj_ntk'. | 3 | 436 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 4 | 453 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 4 | 457 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'UnsupportedEncodingException' is already in scope because it is imported in this file | 4 | 458 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 4 | 467 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'InputStream' is already in scope because it is imported in this file | 4 | 471 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'UnsupportedEncodingException' is already in scope because it is imported in this file | 4 | 472 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'Reader' is already in scope because it is imported in this file | 4 | 482 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'Reader' is already in scope because it is imported in this file | 4 | 492 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.lang': 'Error' is already in scope because it is declared in java.lang | 4 | 552 |
| UnusedPrivateMethod | Avoid unused private methods such as 'jj_ntk_f()'. | 3 | 600 |
| UselessParentheses | Useless parentheses around `jj_ntk = (token.next = token_source.getNextToke...`. | 4 | 602 |
| UselessParentheses | Useless parentheses around `jj_ntk = jj_nt.kind`. | 4 | 604 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 728 |
org/apache/maven/surefire/group/parse/GroupMatcherParserTokenManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unused import 'org.apache.maven.surefire.group.match.*' | 4 | 4 |
| UnnecessaryImport | Unused import 'java.io.*' | 4 | 5 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'PrintStream' is already in scope because it is imported in this file | 4 | 12 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'PrintStream' is already in scope because it is imported in this file | 4 | 14 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'jjStopStringLiteralDfa_0': private methods cannot be overridden | 3 | 15 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'jjStartNfa_0': private methods cannot be overridden | 3 | 63 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 97 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 4 | 121 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 124 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 4 | 142 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 145 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 4 | 159 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 162 |
| UselessParentheses | Duplicate parentheses around `active0 = old0`. | 4 | 176 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 179 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 199 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 3 | 233 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 3 | 280 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 299 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 365 |
| AvoidUsingOctalValues | Avoid integer literals that start with zero (interpreted as octal), remove the leading 0 to get a decimal literal (or use explicit 0x, 0b prefixes) | 3 | 373 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.io': 'IOException' is already in scope because it is imported in this file | 4 | 388 |
| UselessParentheses | Useless parentheses around `lengthOfMatch = jjmatchedPos + 1`. | 4 | 416 |
org/apache/maven/surefire/group/parse/ParseException.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 131–140 |
org/apache/maven/surefire/group/parse/SimpleCharStream.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `bufpos = bufsize - tokenBegin`. | 4 | 63 |
| UselessParentheses | Useless parentheses around `bufpos = tokenBegin`. | 4 | 76 |
| UnnecessaryReturn | Unnecessary return statement | 3 | 123 |
| UselessParentheses | Useless parentheses around `column = 1`. | 4 | 151 |
| UselessParentheses | Useless parentheses around `column = 1`. | 4 | 161 |
Suppressed Violations
| Filename | Rule message | Suppression type | Reason |
|---|---|---|---|
| org/apache/maven/surefire/group/parse/GroupMatcherParserTokenManager.java | Avoid unused local variables such as 'i2'. | @suppresswarnings | |
| org/apache/maven/surefire/group/parse/GroupMatcherParserTokenManager.java | Avoid unused local variables such as 'l2'. | @suppresswarnings | |
| org/apache/maven/surefire/group/parse/GroupMatcherParserTokenManager.java | Avoid unused private fields such as 'image'. | @suppresswarnings | |
| org/apache/maven/surefire/group/parse/GroupMatcherParserTokenManager.java | Avoid unused private fields such as 'jjimageLen'. | @suppresswarnings | |
| org/apache/maven/surefire/group/parse/GroupMatcherParserTokenManager.java | Avoid unused private fields such as 'lengthOfMatch'. | @suppresswarnings | |
| org/apache/maven/surefire/group/parse/GroupMatcherParserTokenManager.java | Avoid unused private methods such as 'jjCheckNAddTwoStates(int, int)'. | @suppresswarnings | |
| org/apache/maven/surefire/group/parse/GroupMatcherParserTokenManager.java | Avoid unused private methods such as 'jjAddStates(int, int)'. | @suppresswarnings | |
| org/apache/maven/surefire/group/parse/TokenMgrError.java | Useless parentheses around `"Lexical error at line " + errorLine + ", colum...`. | @suppresswarnings | |
| org/apache/maven/surefire/group/parse/TokenMgrError.java | Overriding method merely calls super | @suppresswarnings |