PMD Results
The following document contains the results of PMD 7.17.0.
Violations By Priority
Priority 3
org/codehaus/modello/plugin/java/JavaModelloGenerator.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 252–258 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1440–1443 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1993–1995 |
org/codehaus/modello/plugin/java/javasource/JClass.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 245–249 |
org/codehaus/modello/plugin/java/javasource/JConstructor.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty for statement | 186 |
org/codehaus/modello/plugin/java/javasource/JDocComment.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 186 |
org/codehaus/modello/plugin/java/javasource/JInterface.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 205–209 |
org/codehaus/modello/plugin/java/javasource/JMember.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getModifiers': the method is declared in an interface type | 89 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getName': the method is declared in an interface type | 95 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAnnotations': the method is declared in an interface type | 97 |
org/codehaus/modello/plugin/java/javasource/JSourceWriter.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 261–262 |
| UnnecessarySemicolon | Unnecessary semicolon | 263 |
| EmptyCatchBlock | Avoid empty catch blocks | 274–275 |
| UnnecessarySemicolon | Unnecessary semicolon | 276 |
| EmptyCatchBlock | Avoid empty catch blocks | 287–288 |
| UnnecessarySemicolon | Unnecessary semicolon | 289 |
| EmptyCatchBlock | Avoid empty catch blocks | 300–301 |
| UnnecessarySemicolon | Unnecessary semicolon | 302 |
| EmptyCatchBlock | Avoid empty catch blocks | 313–314 |
| UnnecessarySemicolon | Unnecessary semicolon | 315 |
| EmptyCatchBlock | Avoid empty catch blocks | 327–328 |
| EmptyCatchBlock | Avoid empty catch blocks | 341–342 |
| UnnecessarySemicolon | Unnecessary semicolon | 343 |
| EmptyCatchBlock | Avoid empty catch blocks | 354–355 |
| UnnecessarySemicolon | Unnecessary semicolon | 356 |
| EmptyCatchBlock | Avoid empty catch blocks | 367–368 |
| UnnecessarySemicolon | Unnecessary semicolon | 369 |
| EmptyCatchBlock | Avoid empty catch blocks | 382–383 |
| UnnecessarySemicolon | Unnecessary semicolon | 384 |
| EmptyCatchBlock | Avoid empty catch blocks | 390–391 |
| UnnecessarySemicolon | Unnecessary semicolon | 392 |
| EmptyCatchBlock | Avoid empty catch blocks | 400–401 |
| UnnecessarySemicolon | Unnecessary semicolon | 402 |
| EmptyCatchBlock | Avoid empty catch blocks | 412–413 |
| UnnecessarySemicolon | Unnecessary semicolon | 414 |
| EmptyCatchBlock | Avoid empty catch blocks | 424–425 |
| UnnecessarySemicolon | Unnecessary semicolon | 426 |
| EmptyCatchBlock | Avoid empty catch blocks | 437–438 |
| UnnecessarySemicolon | Unnecessary semicolon | 439 |
| EmptyCatchBlock | Avoid empty catch blocks | 449–450 |
| UnnecessarySemicolon | Unnecessary semicolon | 451 |
| EmptyCatchBlock | Avoid empty catch blocks | 480–481 |
| UnnecessarySemicolon | Unnecessary semicolon | 482 |
| EmptyCatchBlock | Avoid empty catch blocks | 502–503 |
| UnnecessarySemicolon | Unnecessary semicolon | 504 |
Priority 4
org/codehaus/modello/plugin/java/javasource/JComment.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `offset < length`. | 292 |
| UselessParentheses | Useless parentheses around `isWhitespace(ch) || isNewLine(ch)`. | 342 |
| UselessParentheses | Useless parentheses around `(ch == ' ') || (ch == '\t')`. | 346 |
| UselessParentheses | Useless parentheses around `(ch == '\n') || (ch == '\r')`. | 350 |
org/codehaus/modello/plugin/java/javasource/JMethodSignature.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `name.isEmpty()`. | 128 |
org/codehaus/modello/plugin/java/javasource/JModifiers.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `visibility == vPrivate`. | 207 |
| UselessParentheses | Useless parentheses around `visibility == vProtected`. | 215 |
| UselessParentheses | Useless parentheses around `visibility == vPublic`. | 223 |
| UselessParentheses | Useless parentheses around `visibility == vPackage`. | 231 |
org/codehaus/modello/plugin/java/javasource/JStructure.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `i - lastIdx`. | 778 |
org/codehaus/modello/plugin/java/javasource/JType.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses around `(this == BOOLEAN) || (this == BYTE) || (this ==...`. | 154–161 |
Files
org/codehaus/modello/plugin/java/JavaModelloGenerator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 252–258 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1440–1443 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1993–1995 |
org/codehaus/modello/plugin/java/javasource/JClass.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 245–249 |
org/codehaus/modello/plugin/java/javasource/JComment.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `offset < length`. | 4 | 292 |
| UselessParentheses | Useless parentheses around `isWhitespace(ch) || isNewLine(ch)`. | 4 | 342 |
| UselessParentheses | Useless parentheses around `(ch == ' ') || (ch == '\t')`. | 4 | 346 |
| UselessParentheses | Useless parentheses around `(ch == '\n') || (ch == '\r')`. | 4 | 350 |
org/codehaus/modello/plugin/java/javasource/JConstructor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty for statement | 3 | 186 |
org/codehaus/modello/plugin/java/javasource/JDocComment.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 186 |
org/codehaus/modello/plugin/java/javasource/JInterface.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 205–209 |
org/codehaus/modello/plugin/java/javasource/JMember.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getModifiers': the method is declared in an interface type | 3 | 89 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getName': the method is declared in an interface type | 3 | 95 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAnnotations': the method is declared in an interface type | 3 | 97 |
org/codehaus/modello/plugin/java/javasource/JMethodSignature.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `name.isEmpty()`. | 4 | 128 |
org/codehaus/modello/plugin/java/javasource/JModifiers.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `visibility == vPrivate`. | 4 | 207 |
| UselessParentheses | Useless parentheses around `visibility == vProtected`. | 4 | 215 |
| UselessParentheses | Useless parentheses around `visibility == vPublic`. | 4 | 223 |
| UselessParentheses | Useless parentheses around `visibility == vPackage`. | 4 | 231 |
org/codehaus/modello/plugin/java/javasource/JSourceWriter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 261–262 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 263 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 274–275 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 276 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 287–288 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 289 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 300–301 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 302 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 313–314 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 315 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 327–328 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 341–342 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 343 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 354–355 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 356 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 367–368 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 369 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 382–383 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 384 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 390–391 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 392 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 400–401 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 402 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 412–413 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 414 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 424–425 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 426 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 437–438 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 439 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 449–450 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 451 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 480–481 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 482 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 502–503 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 504 |
org/codehaus/modello/plugin/java/javasource/JStructure.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `i - lastIdx`. | 4 | 778 |
org/codehaus/modello/plugin/java/javasource/JType.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses around `(this == BOOLEAN) || (this == BYTE) || (this ==...`. | 4 | 154–161 |