PMD Results
The following document contains the results of PMD 7.17.0.
Violations By Priority
Priority 2
org/codehaus/plexus/util/SelectorUtils.java
| Rule | Violation | Line |
|---|---|---|
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 371 |
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 475 |
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 629 |
org/codehaus/plexus/util/xml/pull/MXSerializer.java
| Rule | Violation | Line |
|---|---|---|
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 526 |
Priority 3
org/codehaus/plexus/util/Base64.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 296–301 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 334–341 |
org/codehaus/plexus/util/CollectionUtils.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 207–208 |
org/codehaus/plexus/util/Expand.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty for statement | 144–147 |
org/codehaus/plexus/util/FileUtils.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 1296–1300 |
org/codehaus/plexus/util/NioFiles.java
| Rule | Violation | Line |
|---|---|---|
| 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) | 49 |
| 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) | 52 |
| 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) | 55 |
| 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) | 59 |
| 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) | 62 |
| 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) | 65 |
org/codehaus/plexus/util/PathTool.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'determineRelativePath': private methods cannot be overridden | 329 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'determineSeparator': private methods cannot be overridden | 365 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'buildRelativePath': private methods cannot be overridden | 388 |
org/codehaus/plexus/util/SelectorUtils.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 659–662 |
org/codehaus/plexus/util/StringUtils.java
| Rule | Violation | Line |
|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'sz'. | 283 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1003–1005 |
org/codehaus/plexus/util/SweeperPool.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 253–254 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'debug': private methods cannot be overridden | 285 |
org/codehaus/plexus/util/TypeFormat.java
| Rule | Violation | Line |
|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 124 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 172 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 220 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 273 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 310 |
| EmptyControlStatement | Empty for statement | 408 |
| EmptyControlStatement | Empty for statement | 488 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'multE': private methods cannot be overridden | 757 |
org/codehaus/plexus/util/cli/ShutdownHookUtils.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 32–33 |
| EmptyCatchBlock | Avoid empty catch blocks | 33–34 |
| EmptyCatchBlock | Avoid empty catch blocks | 40–41 |
| EmptyCatchBlock | Avoid empty catch blocks | 41–42 |
org/codehaus/plexus/util/cli/StreamConsumer.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'consumeLine': the method is declared in an interface type | 73 |
org/codehaus/plexus/util/cli/StreamPumper.java
| Rule | Violation | Line |
|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 119 |
org/codehaus/plexus/util/introspection/ClassMap.java
| Rule | Violation | Line |
|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'methodKey'. | 118 |
org/codehaus/plexus/util/io/CachingOutputStream.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'path'. | 34 |
org/codehaus/plexus/util/xml/XmlReader.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 332–341 |
| UnusedFormalParameter | Avoid unused method parameters such as 'lenient'. | 381 |
org/codehaus/plexus/util/xml/Xpp3Dom.java
| Rule | Violation | Line |
|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 178 |
org/codehaus/plexus/util/xml/Xpp3DomBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'InputLocationBuilder': member interfaces are implicitly static | 213 |
org/codehaus/plexus/util/xml/pull/MXParser.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'fastHash': private methods cannot be overridden | 283 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1194–1197 |
| UnusedLocalVariable | Avoid unused local variables such as 'name'. | 1777 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1933–1937 |
| UnusedLocalVariable | Avoid unused local variables such as 'prefix'. | 1957 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 2532–2554 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 2967–2977 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 2968–2976 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3044–3046 |
| UnnecessaryReturn | Unnecessary return statement | 3111 |
org/codehaus/plexus/util/xml/pull/MXSerializer.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 936–959 |
| EmptyControlStatement | Empty if statement | 1007–1032 |
| UnusedLocalVariable | Avoid unused local variables such as 'ch'. | 1050 |
Priority 4
org/codehaus/plexus/util/Base64.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `b1`. | 282 |
| UselessParentheses | Useless parentheses around `b2`. | 283 |
| UselessParentheses | Useless parentheses around `b3`. | 284 |
| UselessParentheses | Useless parentheses around `b1`. | 311 |
| UselessParentheses | Useless parentheses around `b1`. | 323 |
| UselessParentheses | Useless parentheses around `b2`. | 324 |
org/codehaus/plexus/util/CollectionUtils.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'CollectionUtils': 'subtract' is already in scope | 73 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'CollectionUtils': 'intersection' is already in scope | 74 |
org/codehaus/plexus/util/DirectoryWalker.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `percentageSize / count`. | 86 |
org/codehaus/plexus/util/ExceptionUtils.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'ExceptionUtils': 'getCause' is already in scope | 292 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'ExceptionUtils': 'getThrowables' is already in scope | 346 |
org/codehaus/plexus/util/FastMap.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `entry.getValue()`. | 578 |
| UselessParentheses | Useless parentheses around `_key.equals(entry.getKey())`. | 939 |
org/codehaus/plexus/util/FileUtils.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `i >= 0 ? filename.substring(0, i) : ""`. | 201 |
| UselessParentheses | Useless parentheses around `i >= 0 ? filename.substring(i + 1) : filename`. | 212 |
| UselessParentheses | Useless parentheses around `new File(baseFile, filenm)`. | 1148 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'FileUtils': 'FS' is already in scope because it is declared in an enclosing type | 1590 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'FileUtils': 'FS' is already in scope because it is declared in an enclosing type | 1602 |
org/codehaus/plexus/util/Os.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Os': 'isFamily' is already in scope | 344 |
| UselessParentheses | Useless parentheses around `validFamilies.contains(theFamily)`. | 372 |
org/codehaus/plexus/util/PathTool.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `toPath.startsWith(":", 1) && fromPath.startsWit...`. | 293 |
org/codehaus/plexus/util/SelectorUtils.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `patIdxTmp - patIdxStart - 1`. | 357 |
| UselessParentheses | Useless parentheses around `strIdxEnd - strIdxStart + 1`. | 358 |
| UselessParentheses | Useless parentheses around `patIdxTmp - patIdxStart - 1`. | 461 |
| UselessParentheses | Useless parentheses around `strIdxEnd - strIdxStart + 1`. | 462 |
| UselessParentheses | Useless parentheses around `patIdxTmp - patIdxStart - 1`. | 616 |
| UselessParentheses | Useless parentheses around `strIdxEnd - strIdxStart + 1`. | 617 |
org/codehaus/plexus/util/StringUtils.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `str == null ? "" : str.trim()`. | 111 |
| UselessParentheses | Useless parentheses around `str == null ? null : str.trim()`. | 125 |
| UselessParentheses | Useless parentheses around `(str != null) && (!str.isEmpty())`. | 157 |
| UselessParentheses | Useless parentheses around `(str == null) || str.isEmpty()`. | 171 |
| UselessParentheses | Useless parentheses around `str.isEmpty()`. | 171 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'StringUtils': 'isBlank' is already in scope | 222 |
| UselessParentheses | Useless parentheses around `str1 == null ? str2 == null : str1.equalsIgnore...`. | 263 |
| UselessParentheses | Useless parentheses around `arraySize == 0 ? 0 : (array[0].toString().lengt...`. | 616 |
| UselessParentheses | Useless parentheses around `Character.isWhitespace(str.charAt(i)) == false`. | 1659 |
| UselessParentheses | Useless parentheses around `maxWidth - 3`. | 1933 |
org/codehaus/plexus/util/TypeFormat.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `isNegative`. | 126 |
| UselessParentheses | Useless parentheses around `isNegative`. | 174 |
| UselessParentheses | Useless parentheses around `isNegative`. | 222 |
| UselessParentheses | Useless parentheses around `c - '0'`. | 294 |
| UselessParentheses | Useless parentheses around `c - '0'`. | 316 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.lang': 'IllegalArgumentException' is already in scope because it is declared in java.lang | 628 |
org/codehaus/plexus/util/cli/CommandLineUtils.java
| Rule | Violation | Line |
|---|---|---|
| DontCallThreadRun | Dont call Thread.run() explicitly, use Thread.start() | 244 |
org/codehaus/plexus/util/xml/XmlReader.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `m.find()`. | 526 |
org/codehaus/plexus/util/xml/Xpp3Dom.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `(str != null) && (str.length() > 0)`. | 552 |
| UselessParentheses | Useless parentheses around `(str == null) || (str.trim().length() == 0)`. | 556 |
org/codehaus/plexus/util/xml/Xpp3DomUtils.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `str != null && str.length() > 0`. | 241 |
| UselessParentheses | Useless parentheses around `str == null || str.length() == 0`. | 249 |
org/codehaus/plexus/util/xml/pull/MXParser.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `new String(cbuf, off, len)`. | 74 |
| UselessParentheses | Useless parentheses around `" (position:" + getPositionDescription()`. | 1021 |
| UselessParentheses | Useless parentheses around `" (position:" + getPositionDescription()`. | 1044 |
| UselessParentheses | Useless parentheses around `pos - 1`. | 1694 |
| UselessParentheses | Useless parentheses around `pos - 1`. | 1769 |
| UselessParentheses | Useless parentheses around `allStringsInterned && attributeName[j].equals(a...`. | 1883–1886 |
| UselessParentheses | Useless parentheses around `ch == 'x'`. | 2162 |
| UselessParentheses | Useless parentheses around `ch - '0'`. | 2169 |
| UselessParentheses | Useless parentheses around `ch - ('a' - 10)`. | 2172 |
| UselessParentheses | Useless parentheses around `ch - ('A' - 10)`. | 2175 |
| UselessParentheses | Useless parentheses around `ch - '0'`. | 2188 |
| UselessParentheses | Useless parentheses around `pos - 1`. | 2243 |
| UselessParentheses | Useless parentheses around `ch == ' ' || ch == '\n' || ch == '\r' || ch == ...`. | 3342 |
Files
org/codehaus/plexus/util/Base64.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `b1`. | 4 | 282 |
| UselessParentheses | Useless parentheses around `b2`. | 4 | 283 |
| UselessParentheses | Useless parentheses around `b3`. | 4 | 284 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 296–301 |
| UselessParentheses | Useless parentheses around `b1`. | 4 | 311 |
| UselessParentheses | Useless parentheses around `b1`. | 4 | 323 |
| UselessParentheses | Useless parentheses around `b2`. | 4 | 324 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 334–341 |
org/codehaus/plexus/util/CollectionUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'CollectionUtils': 'subtract' is already in scope | 4 | 73 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'CollectionUtils': 'intersection' is already in scope | 4 | 74 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 207–208 |
org/codehaus/plexus/util/DirectoryWalker.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `percentageSize / count`. | 4 | 86 |
org/codehaus/plexus/util/ExceptionUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'ExceptionUtils': 'getCause' is already in scope | 4 | 292 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'ExceptionUtils': 'getThrowables' is already in scope | 4 | 346 |
org/codehaus/plexus/util/Expand.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty for statement | 3 | 144–147 |
org/codehaus/plexus/util/FastMap.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `entry.getValue()`. | 4 | 578 |
| UselessParentheses | Useless parentheses around `_key.equals(entry.getKey())`. | 4 | 939 |
org/codehaus/plexus/util/FileUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `i >= 0 ? filename.substring(0, i) : ""`. | 4 | 201 |
| UselessParentheses | Useless parentheses around `i >= 0 ? filename.substring(i + 1) : filename`. | 4 | 212 |
| UselessParentheses | Useless parentheses around `new File(baseFile, filenm)`. | 4 | 1148 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1296–1300 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'FileUtils': 'FS' is already in scope because it is declared in an enclosing type | 4 | 1590 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'FileUtils': 'FS' is already in scope because it is declared in an enclosing type | 4 | 1602 |
org/codehaus/plexus/util/NioFiles.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| 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 | 49 |
| 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 | 52 |
| 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 | 55 |
| 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 | 59 |
| 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 | 62 |
| 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 | 65 |
org/codehaus/plexus/util/Os.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Os': 'isFamily' is already in scope | 4 | 344 |
| UselessParentheses | Useless parentheses around `validFamilies.contains(theFamily)`. | 4 | 372 |
org/codehaus/plexus/util/PathTool.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `toPath.startsWith(":", 1) && fromPath.startsWit...`. | 4 | 293 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'determineRelativePath': private methods cannot be overridden | 3 | 329 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'determineSeparator': private methods cannot be overridden | 3 | 365 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'buildRelativePath': private methods cannot be overridden | 3 | 388 |
org/codehaus/plexus/util/SelectorUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `patIdxTmp - patIdxStart - 1`. | 4 | 357 |
| UselessParentheses | Useless parentheses around `strIdxEnd - strIdxStart + 1`. | 4 | 358 |
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 371 |
| UselessParentheses | Useless parentheses around `patIdxTmp - patIdxStart - 1`. | 4 | 461 |
| UselessParentheses | Useless parentheses around `strIdxEnd - strIdxStart + 1`. | 4 | 462 |
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 475 |
| UselessParentheses | Useless parentheses around `patIdxTmp - patIdxStart - 1`. | 4 | 616 |
| UselessParentheses | Useless parentheses around `strIdxEnd - strIdxStart + 1`. | 4 | 617 |
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 629 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 659–662 |
org/codehaus/plexus/util/StringUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `str == null ? "" : str.trim()`. | 4 | 111 |
| UselessParentheses | Useless parentheses around `str == null ? null : str.trim()`. | 4 | 125 |
| UselessParentheses | Useless parentheses around `(str != null) && (!str.isEmpty())`. | 4 | 157 |
| UselessParentheses | Useless parentheses around `(str == null) || str.isEmpty()`. | 4 | 171 |
| UselessParentheses | Useless parentheses around `str.isEmpty()`. | 4 | 171 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'StringUtils': 'isBlank' is already in scope | 4 | 222 |
| UselessParentheses | Useless parentheses around `str1 == null ? str2 == null : str1.equalsIgnore...`. | 4 | 263 |
| UnusedLocalVariable | Avoid unused local variables such as 'sz'. | 3 | 283 |
| UselessParentheses | Useless parentheses around `arraySize == 0 ? 0 : (array[0].toString().lengt...`. | 4 | 616 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1003–1005 |
| UselessParentheses | Useless parentheses around `Character.isWhitespace(str.charAt(i)) == false`. | 4 | 1659 |
| UselessParentheses | Useless parentheses around `maxWidth - 3`. | 4 | 1933 |
org/codehaus/plexus/util/SweeperPool.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 253–254 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'debug': private methods cannot be overridden | 3 | 285 |
org/codehaus/plexus/util/TypeFormat.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 124 |
| UselessParentheses | Useless parentheses around `isNegative`. | 4 | 126 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 172 |
| UselessParentheses | Useless parentheses around `isNegative`. | 4 | 174 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 220 |
| UselessParentheses | Useless parentheses around `isNegative`. | 4 | 222 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 273 |
| UselessParentheses | Useless parentheses around `c - '0'`. | 4 | 294 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 310 |
| UselessParentheses | Useless parentheses around `c - '0'`. | 4 | 316 |
| EmptyControlStatement | Empty for statement | 3 | 408 |
| EmptyControlStatement | Empty for statement | 3 | 488 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.lang': 'IllegalArgumentException' is already in scope because it is declared in java.lang | 4 | 628 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'multE': private methods cannot be overridden | 3 | 757 |
org/codehaus/plexus/util/cli/CommandLineUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DontCallThreadRun | Dont call Thread.run() explicitly, use Thread.start() | 4 | 244 |
org/codehaus/plexus/util/cli/ShutdownHookUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 32–33 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 33–34 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 40–41 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 41–42 |
org/codehaus/plexus/util/cli/StreamConsumer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'consumeLine': the method is declared in an interface type | 3 | 73 |
org/codehaus/plexus/util/cli/StreamPumper.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 119 |
org/codehaus/plexus/util/introspection/ClassMap.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'methodKey'. | 3 | 118 |
org/codehaus/plexus/util/io/CachingOutputStream.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'path'. | 3 | 34 |
org/codehaus/plexus/util/xml/XmlReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 332–341 |
| UnusedFormalParameter | Avoid unused method parameters such as 'lenient'. | 3 | 381 |
| UselessParentheses | Useless parentheses around `m.find()`. | 4 | 526 |
org/codehaus/plexus/util/xml/Xpp3Dom.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 178 |
| UselessParentheses | Useless parentheses around `(str != null) && (str.length() > 0)`. | 4 | 552 |
| UselessParentheses | Useless parentheses around `(str == null) || (str.trim().length() == 0)`. | 4 | 556 |
org/codehaus/plexus/util/xml/Xpp3DomBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'InputLocationBuilder': member interfaces are implicitly static | 3 | 213 |
org/codehaus/plexus/util/xml/Xpp3DomUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `str != null && str.length() > 0`. | 4 | 241 |
| UselessParentheses | Useless parentheses around `str == null || str.length() == 0`. | 4 | 249 |
org/codehaus/plexus/util/xml/pull/MXParser.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `new String(cbuf, off, len)`. | 4 | 74 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'fastHash': private methods cannot be overridden | 3 | 283 |
| UselessParentheses | Useless parentheses around `" (position:" + getPositionDescription()`. | 4 | 1021 |
| UselessParentheses | Useless parentheses around `" (position:" + getPositionDescription()`. | 4 | 1044 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1194–1197 |
| UselessParentheses | Useless parentheses around `pos - 1`. | 4 | 1694 |
| UselessParentheses | Useless parentheses around `pos - 1`. | 4 | 1769 |
| UnusedLocalVariable | Avoid unused local variables such as 'name'. | 3 | 1777 |
| UselessParentheses | Useless parentheses around `allStringsInterned && attributeName[j].equals(a...`. | 4 | 1883–1886 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1933–1937 |
| UnusedLocalVariable | Avoid unused local variables such as 'prefix'. | 3 | 1957 |
| UselessParentheses | Useless parentheses around `ch == 'x'`. | 4 | 2162 |
| UselessParentheses | Useless parentheses around `ch - '0'`. | 4 | 2169 |
| UselessParentheses | Useless parentheses around `ch - ('a' - 10)`. | 4 | 2172 |
| UselessParentheses | Useless parentheses around `ch - ('A' - 10)`. | 4 | 2175 |
| UselessParentheses | Useless parentheses around `ch - '0'`. | 4 | 2188 |
| UselessParentheses | Useless parentheses around `pos - 1`. | 4 | 2243 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 2532–2554 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 2967–2977 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 2968–2976 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 3044–3046 |
| UnnecessaryReturn | Unnecessary return statement | 3 | 3111 |
| UselessParentheses | Useless parentheses around `ch == ' ' || ch == '\n' || ch == '\r' || ch == ...`. | 4 | 3342 |
org/codehaus/plexus/util/xml/pull/MXSerializer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 526 |
| EmptyControlStatement | Empty if statement | 3 | 936–959 |
| EmptyControlStatement | Empty if statement | 3 | 1007–1032 |
| UnusedLocalVariable | Avoid unused local variables such as 'ch'. | 3 | 1050 |