The following document contains the results of PMD 6.46.0.
| Rule | Violation | Line |
|---|---|---|
| DontUseFloatTypeForLoopIndices | Dont use floating point for loop indices. If you must use floating point, use double. | 72 |
| UnnecessarySemicolon | Unnecessary semicolon | 93 |
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 150–152 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Collision': nested enums are implicitly static | 84–95 |
| UnnecessarySemicolon | Unnecessary semicolon | 95 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 669 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Thumb': nested enums are implicitly static | 85–239 |
| PrimitiveWrapperInstantiation | Do not use `new Float(...)`, prefer `Float.valueOf(...)` | 431 |
| PrimitiveWrapperInstantiation | Do not use `new Float(...)`, prefer `Float.valueOf(...)` | 432 |
| CollapsibleIfStatements | These nested if statements could be combined | 530–612 |
| CollapsibleIfStatements | These nested if statements could be combined | 719–723 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1093 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1524 |
| CollapsibleIfStatements | These nested if statements could be combined | 1547–1551 |
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 63–65 |
| EmptyControlStatement | Empty if statement | 155–355 |
| EmptyControlStatement | Empty if statement | 267–307 |
| EmptyControlStatement | Empty if statement | 310–350 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'tween': private methods cannot be overridden | 144–155 |
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 139–142 |
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 252 |
| EmptyControlStatement | Empty try body | 601–608 |
| EmptyCatchBlock | Avoid empty catch blocks | 605–608 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'size'. | 882 |
| CollapsibleIfStatements | These nested if statements could be combined | 1014–1017 |
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'source'. | 304 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 437 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 443 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 451 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 857 |
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 107–110 |
| EmptyCatchBlock | Avoid empty catch blocks | 144–147 |
| Rule | Violation | Line |
|---|---|---|
| PrimitiveWrapperInstantiation | Do not use `new Double(...)`, prefer `Double.valueOf(...)` | 97 |
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 266 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 266 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 277 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 286 |
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 67 |
| UselessParentheses | Useless parentheses. | 78 |
| UselessParentheses | Useless parentheses. | 81 |
| UselessParentheses | Useless parentheses. | 84 |
| UselessParentheses | Useless parentheses. | 87 |
| UselessParentheses | Useless parentheses. | 90 |
| UselessParentheses | Useless parentheses. | 93 |
| UselessParentheses | Useless parentheses. | 109–110 |
| UselessParentheses | Useless parentheses. | 109 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.security.AccessControlException' due to existing import 'java.security.AccessControlException' | 135 |
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 298 |
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 105 |
| UselessParentheses | Useless parentheses. | 105 |
| UselessParentheses | Useless parentheses. | 115 |
| UselessParentheses | Useless parentheses. | 115 |
| UselessParentheses | Useless parentheses. | 144 |
| UselessParentheses | Useless parentheses. | 144 |
| UselessParentheses | Useless parentheses. | 148 |
| UselessParentheses | Useless parentheses. | 148 |
| UselessParentheses | Useless parentheses. | 164 |
| UselessParentheses | Useless parentheses. | 164 |
| UselessParentheses | Useless parentheses. | 175 |
| UselessParentheses | Useless parentheses. | 175 |
| UselessParentheses | Useless parentheses. | 456 |
| UselessParentheses | Useless parentheses. | 596 |
| UselessParentheses | Useless parentheses. | 951 |
| UselessParentheses | Useless parentheses. | 1130 |
| UselessParentheses | Useless parentheses. | 1130 |
| UselessParentheses | Useless parentheses. | 1146 |
| UselessParentheses | Useless parentheses. | 1146 |
| UselessParentheses | Useless parentheses. | 1233 |
| UselessParentheses | Useless parentheses. | 1374 |
| UselessParentheses | Useless parentheses. | 1375 |
| UselessParentheses | Useless parentheses. | 1392 |
| UselessParentheses | Useless parentheses. | 1393 |
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 266 |
| UselessParentheses | Useless parentheses. | 309 |
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 74 |
| UselessParentheses | Useless parentheses. | 186 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DontUseFloatTypeForLoopIndices | Dont use floating point for loop indices. If you must use floating point, use double. | 3 | 72 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 93 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 67 |
| UselessParentheses | Useless parentheses. | 4 | 78 |
| UselessParentheses | Useless parentheses. | 4 | 81 |
| UselessParentheses | Useless parentheses. | 4 | 84 |
| UselessParentheses | Useless parentheses. | 4 | 87 |
| UselessParentheses | Useless parentheses. | 4 | 90 |
| UselessParentheses | Useless parentheses. | 4 | 93 |
| UselessParentheses | Useless parentheses. | 4 | 109–110 |
| UselessParentheses | Useless parentheses. | 4 | 109 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.security.AccessControlException' due to existing import 'java.security.AccessControlException' | 4 | 135 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 150–152 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Collision': nested enums are implicitly static | 3 | 84–95 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 95 |
| UselessParentheses | Useless parentheses. | 4 | 298 |
| PrimitiveWrapperInstantiation | Do not use `new Integer(...)`, prefer `Integer.valueOf(...)` | 3 | 669 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Thumb': nested enums are implicitly static | 3 | 85–239 |
| UselessParentheses | Useless parentheses. | 4 | 105 |
| UselessParentheses | Useless parentheses. | 4 | 105 |
| UselessParentheses | Useless parentheses. | 4 | 115 |
| UselessParentheses | Useless parentheses. | 4 | 115 |
| UselessParentheses | Useless parentheses. | 4 | 144 |
| UselessParentheses | Useless parentheses. | 4 | 144 |
| UselessParentheses | Useless parentheses. | 4 | 148 |
| UselessParentheses | Useless parentheses. | 4 | 148 |
| UselessParentheses | Useless parentheses. | 4 | 164 |
| UselessParentheses | Useless parentheses. | 4 | 164 |
| UselessParentheses | Useless parentheses. | 4 | 175 |
| UselessParentheses | Useless parentheses. | 4 | 175 |
| PrimitiveWrapperInstantiation | Do not use `new Float(...)`, prefer `Float.valueOf(...)` | 3 | 431 |
| PrimitiveWrapperInstantiation | Do not use `new Float(...)`, prefer `Float.valueOf(...)` | 3 | 432 |
| UselessParentheses | Useless parentheses. | 4 | 456 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 530–612 |
| UselessParentheses | Useless parentheses. | 4 | 596 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 719–723 |
| UselessParentheses | Useless parentheses. | 4 | 951 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1093 |
| UselessParentheses | Useless parentheses. | 4 | 1130 |
| UselessParentheses | Useless parentheses. | 4 | 1130 |
| UselessParentheses | Useless parentheses. | 4 | 1146 |
| UselessParentheses | Useless parentheses. | 4 | 1146 |
| UselessParentheses | Useless parentheses. | 4 | 1233 |
| UselessParentheses | Useless parentheses. | 4 | 1374 |
| UselessParentheses | Useless parentheses. | 4 | 1375 |
| UselessParentheses | Useless parentheses. | 4 | 1392 |
| UselessParentheses | Useless parentheses. | 4 | 1393 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1524 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 1547–1551 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 63–65 |
| EmptyControlStatement | Empty if statement | 3 | 155–355 |
| UselessParentheses | Useless parentheses. | 4 | 266 |
| EmptyControlStatement | Empty if statement | 3 | 267–307 |
| UselessParentheses | Useless parentheses. | 4 | 309 |
| EmptyControlStatement | Empty if statement | 3 | 310–350 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'tween': private methods cannot be overridden | 3 | 144–155 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 74 |
| UselessParentheses | Useless parentheses. | 4 | 186 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 139–142 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 252 |
| EmptyControlStatement | Empty try body | 3 | 601–608 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 605–608 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'size'. | 3 | 882 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 1014–1017 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'source'. | 3 | 304 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 437 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 443 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 451 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 857 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 107–110 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 144–147 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| PrimitiveWrapperInstantiation | Do not use `new Double(...)`, prefer `Double.valueOf(...)` | 3 | 97 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 266 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 266 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 277 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 286 |