PMD Results
The following document contains the results of PMD 7.7.0.
Violations By Priority
Priority 2
org/opentrafficsim/editor/DocumentReader.java
| Rule | Violation | Line |
|---|---|---|
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 207 |
Priority 3
org/opentrafficsim/editor/Actions.java
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 149 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 335–338 |
org/opentrafficsim/editor/Dialogs.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 31 |
org/opentrafficsim/editor/OtsEditor.java
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 481 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 482 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 482 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 691 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 706 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 710 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 740–744 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 951 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 962 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 966 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 966 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1236 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1238 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1439 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1440 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1526 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1559 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1779 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1779 |
org/opentrafficsim/editor/PropertiesDialog.java
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 132 |
org/opentrafficsim/editor/XsdTreeNode.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 677–680 |
org/opentrafficsim/editor/XsdTreeNodeUtil.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'node'. | 179 |
| UnusedFormalParameter | Avoid unused method parameters such as 'node'. | 211 |
org/opentrafficsim/editor/decoration/AbstractNodeDecoratorAttribute.java
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 60 |
org/opentrafficsim/editor/decoration/string/AttributesStringFunction.java
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 54 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 60 |
org/opentrafficsim/editor/decoration/validation/AttributesNotEqualValidator.java
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 85 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 86 |
org/opentrafficsim/editor/decoration/validation/KeyValidator.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'fieldIndex'. | 291 |
org/opentrafficsim/editor/decoration/validation/ParentValidator.java
| Rule | Violation | Line |
|---|---|---|
| MisplacedNullCheck | The null check here is misplaced; if the variable 'parent' is null there will be a NullPointerException | 178 |
org/opentrafficsim/editor/decoration/validation/TrafficLightValidator.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 149–153 |
org/opentrafficsim/editor/decoration/validation/ValueValidator.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 834 |
org/opentrafficsim/editor/decoration/validation/XPathValidator.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 420–431 |
| EmptyCatchBlock | Avoid empty catch blocks | 535–538 |
| UnnecessarySemicolon | Unnecessary semicolon | 638 |
org/opentrafficsim/editor/extensions/map/FlattenerListener.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 61–64 |
org/opentrafficsim/editor/extensions/map/MapLinkData.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 878–881 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 1482 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 1566 |
org/opentrafficsim/editor/extensions/map/edit/MapInputListener.java
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 514 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 558 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 565 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 576 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 602 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 608 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 614 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 718 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 740 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 762 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 766 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 824 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 827 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 840 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 882 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 886 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 889 |
| UnusedFormalParameter | Avoid unused method parameters such as 'linkData'. | 909 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 924 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1038 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1070 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1073 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1078 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 1088 |
org/opentrafficsim/editor/listeners/XsdTreeEditorListener.java
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 67 |
org/opentrafficsim/editor/render/StringCellRenderer.java
| Rule | Violation | Line |
|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 100 |
Priority 4
org/opentrafficsim/editor/Actions.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 167 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 191 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 220 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 246 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 272 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 299 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 326 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 356 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 383 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 410 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 438 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 466 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 517 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 543 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 570 |
org/opentrafficsim/editor/Dialogs.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 175 |
org/opentrafficsim/editor/DocumentReader.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DocumentReader': 'getChildren' is already in scope | 194 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DocumentReader': 'getChildren' is already in scope | 196 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DocumentReader': 'getChildren' is already in scope | 204 |
org/opentrafficsim/editor/Navigation.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 84 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 104 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 139 |
org/opentrafficsim/editor/OtsEditor.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 1386 |
org/opentrafficsim/editor/Schema.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 334 |
org/opentrafficsim/editor/XiIncludeNode.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Node': 'TEXT_NODE' is already in scope because it is inherited by an enclosing type | 59 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Node': 'ATTRIBUTE_NODE' is already in scope because it is inherited by an enclosing type | 316 |
org/opentrafficsim/editor/XsdTreeNode.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XsdTreeNodeRoot': 'XsdTreeNode::OPTION_CHANGED' is already in scope because it is declared in an enclosing type | 496 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XsdTreeNodeRoot': 'XsdTreeNode::ACTIVATION_CHANGED' is already in scope because it is declared in an enclosing type | 1148 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XsdTreeNodeRoot': 'XsdTreeNode::ACTIVATION_CHANGED' is already in scope because it is declared in an enclosing type | 1173 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XsdTreeNode': 'ACTIVATION_CHANGED' is already in scope because it is declared in an enclosing type | 1188 |
org/opentrafficsim/editor/XsdTreeNodeUtil.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XsdTreeNodeUtil': 'append' is already in scope | 121 |
org/opentrafficsim/editor/decoration/validation/CoupledValidator.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 70–71 |
| UselessParentheses | Useless parentheses. | 71 |
| UselessParentheses | Useless parentheses. | 93 |
org/opentrafficsim/editor/extensions/OdEditor.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 57 |
org/opentrafficsim/editor/extensions/RouteEditor.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 84 |
org/opentrafficsim/editor/extensions/map/EditorMap.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Component': 'LEFT_ALIGNMENT' is already in scope because it is inherited by an enclosing type | 462 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'ChangeListener': 'CHANGE_EVENT' is already in scope because it is inherited by an enclosing type | 568 |
org/opentrafficsim/editor/extensions/map/MapLinkData.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Map' is already in scope because it is imported in this file | 156 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Map' is already in scope because it is imported in this file | 159 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Map' is already in scope because it is imported in this file | 733 |
org/opentrafficsim/editor/extensions/map/SynchronizableMapStripe.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 56 |
| UselessParentheses | Useless parentheses. | 66 |
org/opentrafficsim/editor/extensions/map/edit/MapInputListener.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 191 |
| UselessParentheses | Useless parentheses. | 1029 |
org/opentrafficsim/editor/listeners/ChangesListener.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 99 |
org/opentrafficsim/editor/listeners/XsdTreeEditorListener.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 59 |
org/opentrafficsim/editor/listeners/XsdTreeListener.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 98 |
org/opentrafficsim/editor/render/AttributeCellRenderer.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'SwingConstants': 'CENTER' is already in scope because it is inherited by an enclosing type | 206 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'SwingConstants': 'LEFT' is already in scope because it is inherited by an enclosing type | 210 |
org/opentrafficsim/editor/render/StringCellRenderer.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 86–87 |
| UselessParentheses | Useless parentheses. | 87 |
| UselessParentheses | Useless parentheses. | 100 |
Files
org/opentrafficsim/editor/Actions.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 149 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 167 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 191 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 220 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 246 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 272 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 299 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 326 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 335–338 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 356 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 383 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 410 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 438 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 466 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 517 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 543 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 570 |
org/opentrafficsim/editor/Dialogs.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 31 |
| UselessParentheses | Useless parentheses. | 4 | 175 |
org/opentrafficsim/editor/DocumentReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DocumentReader': 'getChildren' is already in scope | 4 | 194 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DocumentReader': 'getChildren' is already in scope | 4 | 196 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DocumentReader': 'getChildren' is already in scope | 4 | 204 |
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 207 |
org/opentrafficsim/editor/Navigation.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 84 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 104 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Action': 'ACCELERATOR_KEY' is already in scope because it is inherited by an enclosing type | 4 | 139 |
org/opentrafficsim/editor/OtsEditor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 481 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 482 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 482 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 691 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 706 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 710 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 740–744 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 951 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 962 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 966 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 966 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1236 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1238 |
| UselessParentheses | Useless parentheses. | 4 | 1386 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1439 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1440 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1526 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1559 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1779 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1779 |
org/opentrafficsim/editor/PropertiesDialog.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 132 |
org/opentrafficsim/editor/Schema.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 334 |
org/opentrafficsim/editor/XiIncludeNode.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Node': 'TEXT_NODE' is already in scope because it is inherited by an enclosing type | 4 | 59 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Node': 'ATTRIBUTE_NODE' is already in scope because it is inherited by an enclosing type | 4 | 316 |
org/opentrafficsim/editor/XsdTreeNode.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XsdTreeNodeRoot': 'XsdTreeNode::OPTION_CHANGED' is already in scope because it is declared in an enclosing type | 4 | 496 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 677–680 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XsdTreeNodeRoot': 'XsdTreeNode::ACTIVATION_CHANGED' is already in scope because it is declared in an enclosing type | 4 | 1148 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XsdTreeNodeRoot': 'XsdTreeNode::ACTIVATION_CHANGED' is already in scope because it is declared in an enclosing type | 4 | 1173 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XsdTreeNode': 'ACTIVATION_CHANGED' is already in scope because it is declared in an enclosing type | 4 | 1188 |
org/opentrafficsim/editor/XsdTreeNodeUtil.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'XsdTreeNodeUtil': 'append' is already in scope | 4 | 121 |
| UnusedFormalParameter | Avoid unused method parameters such as 'node'. | 3 | 179 |
| UnusedFormalParameter | Avoid unused method parameters such as 'node'. | 3 | 211 |
org/opentrafficsim/editor/decoration/AbstractNodeDecoratorAttribute.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 60 |
org/opentrafficsim/editor/decoration/string/AttributesStringFunction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 54 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 60 |
org/opentrafficsim/editor/decoration/validation/AttributesNotEqualValidator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 85 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 86 |
org/opentrafficsim/editor/decoration/validation/CoupledValidator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 70–71 |
| UselessParentheses | Useless parentheses. | 4 | 71 |
| UselessParentheses | Useless parentheses. | 4 | 93 |
org/opentrafficsim/editor/decoration/validation/KeyValidator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'fieldIndex'. | 3 | 291 |
org/opentrafficsim/editor/decoration/validation/ParentValidator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MisplacedNullCheck | The null check here is misplaced; if the variable 'parent' is null there will be a NullPointerException | 3 | 178 |
org/opentrafficsim/editor/decoration/validation/TrafficLightValidator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 149–153 |
org/opentrafficsim/editor/decoration/validation/ValueValidator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 834 |
org/opentrafficsim/editor/decoration/validation/XPathValidator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 420–431 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 535–538 |
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 638 |
org/opentrafficsim/editor/extensions/OdEditor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 57 |
org/opentrafficsim/editor/extensions/RouteEditor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 84 |
org/opentrafficsim/editor/extensions/map/EditorMap.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Component': 'LEFT_ALIGNMENT' is already in scope because it is inherited by an enclosing type | 4 | 462 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'ChangeListener': 'CHANGE_EVENT' is already in scope because it is inherited by an enclosing type | 4 | 568 |
org/opentrafficsim/editor/extensions/map/FlattenerListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 61–64 |
org/opentrafficsim/editor/extensions/map/MapLinkData.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Map' is already in scope because it is imported in this file | 4 | 156 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Map' is already in scope because it is imported in this file | 4 | 159 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'java.util': 'Map' is already in scope because it is imported in this file | 4 | 733 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 878–881 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 1482 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 1566 |
org/opentrafficsim/editor/extensions/map/SynchronizableMapStripe.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 56 |
| UselessParentheses | Useless parentheses. | 4 | 66 |
org/opentrafficsim/editor/extensions/map/edit/MapInputListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 191 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 514 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 558 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 565 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 576 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 602 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 608 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 614 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 718 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 740 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 762 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 766 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 824 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 827 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 840 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 882 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 886 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 889 |
| UnusedFormalParameter | Avoid unused method parameters such as 'linkData'. | 3 | 909 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 924 |
| UselessParentheses | Useless parentheses. | 4 | 1029 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1038 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1070 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1073 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1078 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 1088 |
org/opentrafficsim/editor/listeners/ChangesListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 99 |
org/opentrafficsim/editor/listeners/XsdTreeEditorListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 59 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 67 |
org/opentrafficsim/editor/listeners/XsdTreeListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 98 |
org/opentrafficsim/editor/render/AttributeCellRenderer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'SwingConstants': 'CENTER' is already in scope because it is inherited by an enclosing type | 4 | 206 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'SwingConstants': 'LEFT' is already in scope because it is inherited by an enclosing type | 4 | 210 |
org/opentrafficsim/editor/render/StringCellRenderer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 86–87 |
| UselessParentheses | Useless parentheses. | 4 | 87 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 100 |
| UselessParentheses | Useless parentheses. | 4 | 100 |