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 |