The following document contains the results of Checkstyle 8.29 with src/main/checkstyle/checkstyle.xml ruleset. 
| Files |  |
 |
 |
|---|---|---|---|
| 22 | 0 | 0 | 121 |
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | NeedBraces | 57 | |
| RightCurly | 3 | | |
| coding | InnerAssignment | 1 | |
| MultipleVariableDeclarations | 4 | | |
| design | DesignForExtension | 11 | |
| HideUtilityClassConstructor | 1 | | |
| VisibilityModifier | 2 | | |
| javadoc | JavadocMethod | 2 | |
| JavadocPackage | 2 | | |
| JavadocStyle | 3 | | |
| misc | ArrayTypeStyle | 2 | |
| modifier | ModifierOrder | 2 | |
| RedundantModifier | 1 | | |
| naming | ConstantName | 8 | |
| sizes | LineLength
|
6 | |
| whitespace | FileTabCharacter | 2 | |
| GenericWhitespace | 8 | | |
| NoWhitespaceAfter | 2 | | |
| WhitespaceAround | 4 | |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 23 |
|
naming | ConstantName | Name 'characters' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 23 |
|
whitespace | NoWhitespaceAfter | '{' is followed by whitespace. | 23 |
|
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 30 |
|
naming | ConstantName | Name 'characterIndexes' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 30 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 34 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 52 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 61 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 63 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 96 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 113 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 115 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 132 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 136 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 17 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | DesignForExtension | Class 'Benchmarks' looks like designed for extension (can be subclassed), but the method 'hashContains' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'Benchmarks' final or making the method 'hashContains' static/final/abstract/empty, or adding allowed annotation for the method. | 12 |
|
design | DesignForExtension | Class 'Benchmarks' looks like designed for extension (can be subclassed), but the method 'decodeHash' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'Benchmarks' final or making the method 'decodeHash' static/final/abstract/empty, or adding allowed annotation for the method. | 17 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 43 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 77 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 79 |
|
design | DesignForExtension | Class 'Coverage' looks like designed for extension (can be subclassed), but the method 'toString' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'Coverage' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method. | 83 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
modifier | RedundantModifier | Redundant 'public' modifier. | 35 |
|
javadoc | JavadocMethod | Expected @param tag for 'count'. | 35 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 73 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 75 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
whitespace | GenericWhitespace | '>' is followed by whitespace. | 20 |
|
javadoc | JavadocStyle | First sentence should end with a period. | 33 |
|
javadoc | JavadocStyle | First sentence should end with a period. | 47 |
|
sizes | LineLength | Line is longer than 120 characters (found 197). | 57 |
|
sizes | LineLength | Line is longer than 120 characters (found 197). | 68 |
|
sizes | LineLength | Line is longer than 120 characters (found 122). | 108 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 19 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 21 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 23 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 25 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
whitespace | NoWhitespaceAfter | '{' is followed by whitespace. | 37 |
|
whitespace | GenericWhitespace | '>' is followed by whitespace. | 102 |
|
whitespace | GenericWhitespace | '>' is followed by whitespace. | 103 |
|
whitespace | GenericWhitespace | '>' is followed by whitespace. | 104 |
|
whitespace | GenericWhitespace | '>' is followed by whitespace. | 105 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 145 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 152 |
|
blocks | RightCurly | '}' at column 9 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | 169 |
|
blocks | RightCurly | '}' at column 9 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | 175 |
|
blocks | RightCurly | '}' at column 9 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | 181 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 260 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 264 |
|
whitespace | WhitespaceAround | '<=' is not followed by whitespace. | 351 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 367 |
|
coding | MultipleVariableDeclarations | Each variable declaration must be in its own statement. | 379 |
|
coding | MultipleVariableDeclarations | Each variable declaration must be in its own statement. | 380 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 391 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 398 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 461 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 463 |
|
coding | MultipleVariableDeclarations | Each variable declaration must be in its own statement. | 484 |
|
coding | MultipleVariableDeclarations | Each variable declaration must be in its own statement. | 485 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 491 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 495 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 502 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 506 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 580 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 585 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 587 |
|
design | VisibilityModifier | Variable 'count' must be private and have accessor methods. | 618 |
|
design | VisibilityModifier | Variable 'array' must be private and have accessor methods. | 620 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 623 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 624 |
|
coding | InnerAssignment | Inner assignments should be avoided. | 627 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 687 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 689 |
|
naming | ConstantName | Name 'values' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 695 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 698 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 715 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 717 |
|
naming | ConstantName | Name 'values' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 725 |
|
misc | ArrayTypeStyle | Array brackets at illegal position. | 728 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 745 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 747 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 760 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 762 |
|
whitespace | GenericWhitespace | '>' is followed by whitespace. | 812 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 849 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 865 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 871 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | DesignForExtension | Class 'GeoHashTest' looks like designed for extension (can be subclassed), but the method 'testSpeed' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GeoHashTest' final or making the method 'testSpeed' static/final/abstract/empty, or adding allowed annotation for the method. | 205 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 208 |
|
sizes | LineLength | Line is longer than 120 characters (found 178). | 325 |
|
whitespace | GenericWhitespace | '>' is followed by whitespace. | 489 |
|
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 616 |
|
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 616 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | DesignForExtension | Class 'LatLong' looks like designed for extension (can be subclassed), but the method 'toString' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'LatLong' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method. | 53 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 15 |
|
javadoc | JavadocMethod | Expected @param tag for '<T>'. | 23 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 29 |
|
sizes | LineLength | Line is longer than 120 characters (found 164). | 61 |
|
blocks | NeedBraces | 'while' construct must use '{}'s. | 67 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 80 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 86 |
|
sizes | LineLength | Line is longer than 120 characters (found 242). | 102 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 104 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 106 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 116 |
|
blocks | NeedBraces | 'for' construct must use '{}'s. | 141 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 161 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
javadoc | JavadocPackage | Missing package-info.java file. | 1 |
|
javadoc | JavadocStyle | Incomplete HTML tag found: * Returns the {@link Info}s where start<=time <finish and position is | 139 |
|
blocks | NeedBraces | 'if' construct must use '{}'s. | 153 |
|
blocks | NeedBraces | 'else' construct must use '{}'s. | 155 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
naming | ConstantName | Name 'topLeftLat' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 18 |
|
naming | ConstantName | Name 'topLeftLong' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 19 |
|
naming | ConstantName | Name 'bottomRightLat' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 20 |
|
naming | ConstantName | Name 'bottomRightLong' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 21 |
|
whitespace | GenericWhitespace | '>' is followed by whitespace. | 96 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
design | DesignForExtension | Class 'Info' looks like designed for extension (can be subclassed), but the method 'id' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'Info' final or making the method 'id' static/final/abstract/empty, or adding allowed annotation for the method. | 21 |
|
design | DesignForExtension | Class 'Info' looks like designed for extension (can be subclassed), but the method 'lat' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'Info' final or making the method 'lat' static/final/abstract/empty, or adding allowed annotation for the method. | 25 |
|
design | DesignForExtension | Class 'Info' looks like designed for extension (can be subclassed), but the method 'lon' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'Info' final or making the method 'lon' static/final/abstract/empty, or adding allowed annotation for the method. | 29 |
|
design | DesignForExtension | Class 'Info' looks like designed for extension (can be subclassed), but the method 'time' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'Info' final or making the method 'time' static/final/abstract/empty, or adding allowed annotation for the method. | 33 |
|
design | DesignForExtension | Class 'Info' looks like designed for extension (can be subclassed), but the method 'value' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'Info' final or making the method 'value' static/final/abstract/empty, or adding allowed annotation for the method. | 37 |
|
design | DesignForExtension | Class 'Info' looks like designed for extension (can be subclassed), but the method 'toString' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'Info' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method. | 41 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
|
javadoc | JavadocPackage | Missing package-info.java file. | 1 |
