The following document contains the results of Checkstyle 8.35 with /config/dsol-checks.xml ruleset. 
| Files |  |
 |
 |
|---|---|---|---|
| 244 | 6 | 33 | 0 |
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | EmptyBlock | 1 | |
| NeedBraces | 11 | | |
| design | DesignForExtension | 5 | |
| VisibilityModifier | 4 | | |
| javadoc | JavadocPackage | 1 | |
| misc | TodoComment | 8 | |
| naming | StaticVariableName | 4 | |
| sizes | LineLength
|
3 | |
| MethodLength | 2 | |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
javadoc | JavadocPackage | Missing package-info.java file. | 1 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
misc | TodoComment | Comment matches to-do format 'TODO:'. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
misc | TodoComment | Comment matches to-do format 'TODO:'. | 89 |
|
sizes | MethodLength | Method length is 336 lines (max allowed is 150). | 91 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | LineLength | Line is longer than 128 characters (found 129). | 14 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | DesignForExtension | Class 'DirectedPoint' looks like designed for extension (can be subclassed), but the method 'getRotX' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'DirectedPoint' final or making the method 'getRotX' static/final/abstract/empty, or adding allowed annotation for the method. | 58 |
|
design | DesignForExtension | Class 'DirectedPoint' looks like designed for extension (can be subclassed), but the method 'getRotY' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'DirectedPoint' final or making the method 'getRotY' static/final/abstract/empty, or adding allowed annotation for the method. | 63 |
|
design | DesignForExtension | Class 'DirectedPoint' looks like designed for extension (can be subclassed), but the method 'getRotZ' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'DirectedPoint' final or making the method 'getRotZ' static/final/abstract/empty, or adding allowed annotation for the method. | 68 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | MethodLength | Method length is 171 lines (max allowed is 150). | 86 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
sizes | LineLength | Line is longer than 128 characters (found 131). | 374 |
|
sizes | LineLength | Line is longer than 128 characters (found 129). | 375 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | EmptyBlock | Must have at least one statement. | 257 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
misc | TodoComment | Comment matches to-do format 'TODO:'. | 128 |
|
naming | StaticVariableName | Name 'ALIGNED' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 129 |
|
design | VisibilityModifier | Variable 'ALIGNED' must be private and have accessor methods. | 129 |
|
misc | TodoComment | Comment matches to-do format 'TODO:'. | 132 |
|
naming | StaticVariableName | Name 'ALIGN_COUNT' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 133 |
|
design | VisibilityModifier | Variable 'ALIGN_COUNT' must be private and have accessor methods. | 133 |
|
misc | TodoComment | Comment matches to-do format 'TODO:'. | 859 |
|
naming | StaticVariableName | Name 'MOVE_EVENT' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1023 |
|
design | VisibilityModifier | Variable 'MOVE_EVENT' must be private and have accessor methods. | 1023 |
|
naming | StaticVariableName | Name 'DESTROY_EVENT' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1036 |
|
design | VisibilityModifier | Variable 'DESTROY_EVENT' must be private and have accessor methods. | 1036 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | DesignForExtension | Class 'DirectEgoPerception' looks like designed for extension (can be subclassed), but the method 'hashCode' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'DirectEgoPerception' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method. | 161 |
|
design | DesignForExtension | Class 'DirectEgoPerception' looks like designed for extension (can be subclassed), but the method 'equals' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'DirectEgoPerception' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method. | 173 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 176 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 178 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 180 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 185 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 188 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 192 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 195 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 199 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 202 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 206 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 209 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
misc | TodoComment | Comment matches to-do format 'TODO:'. | 67 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
misc | TodoComment | Comment matches to-do format 'TODO:'. | 773 |
|
misc | TodoComment | Comment matches to-do format 'TODO:'. | 897 |