The following document contains the results of Checkstyle 6.18 with /src/main/resources/development/dsol-checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | EmptyBlock | 1 | |
| NeedBraces | 5 | ||
| design | DesignForExtension | 99 | |
| FinalClass | 1 | ||
| VisibilityModifier | 4 | ||
| javadoc | JavadocMethod
|
5 | |
| JavadocStyle | 10 | ||
| misc | FinalParameters | 52 | |
| modifier | ModifierOrder | 2 | |
| RedundantModifier | 1 | ||
| naming | StaticVariableName | 1 | |
| sizes | LineLength
|
6 | |
| ParameterNumber | 1 | ||
| whitespace | WhitespaceAfter | 2 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | ModifierOrder | 'private' modifier out of order with the JLS suggestions. | 26 | |
| modifier | ModifierOrder | 'private' modifier out of order with the JLS suggestions. | 29 | |
| misc | FinalParameters | Parameter decodedBytesPerLine should be final. | 43 | |
| misc | FinalParameters | Parameter extraSpaceAfterEvery should be final. | 43 | |
| design | DesignForExtension | Method 'getResult' is not designed for extension - needs to be abstract, final or empty. | 54 | |
| design | DesignForExtension | Method 'getMaximumWidth' is not designed for extension - needs to be abstract, final or empty. | 72 | |
| design | DesignForExtension | Method 'append' is not designed for extension - needs to be abstract, final or empty. | 79 | |
| misc | FinalParameters | Parameter address should be final. | 80 | |
| misc | FinalParameters | Parameter theByte should be final. | 80 | |
| blocks | EmptyBlock | Must have at least one statement. | 119 | |
| design | DesignForExtension | Method 'ignoreForIdenticalOutputCheck' is not designed for extension - needs to be abstract, final or empty. | 139 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 73 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | FinalParameters | Parameter extraSpaceAfterEvery should be final. | 31 | |
| design | DesignForExtension | Method 'getResult' is not designed for extension - needs to be abstract, final or empty. | 43 | |
| design | DesignForExtension | Method 'getMaximumWidth' is not designed for extension - needs to be abstract, final or empty. | 52 | |
| design | DesignForExtension | Method 'append' is not designed for extension - needs to be abstract, final or empty. | 59 | |
| misc | FinalParameters | Parameter address should be final. | 60 | |
| misc | FinalParameters | Parameter theByte should be final. | 60 | |
| design | DesignForExtension | Method 'ignoreForIdenticalOutputCheck' is not designed for extension - needs to be abstract, final or empty. | 73 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 80 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocStyle | First sentence should end with a period. | 43 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | StaticVariableName | Name 'SUPPRESSEDOUTPUTINDICATORLINE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 45 | |
| design | DesignForExtension | Method 'setOutputStream' is not designed for extension - needs to be abstract, final or empty. | 69 | |
| design | DesignForExtension | Method 'setSuppressMultipleIdenticalLines' is not designed for extension - needs to be abstract, final or empty. | 81 | |
| design | DesignForExtension | Method 'addDecoder' is not designed for extension - needs to be abstract, final or empty. | 91 | |
| design | DesignForExtension | Method 'addDecoder' is not designed for extension - needs to be abstract, final or empty. | 103 | |
| design | DesignForExtension | Method 'append' is not designed for extension - needs to be abstract, final or empty. | 159 | |
| design | DesignForExtension | Method 'append' is not designed for extension - needs to be abstract, final or empty. | 180 | |
| design | DesignForExtension | Method 'append' is not designed for extension - needs to be abstract, final or empty. | 194 | |
| design | DesignForExtension | Method 'append' is not designed for extension - needs to be abstract, final or empty. | 212 | |
| design | DesignForExtension | Method 'flush' is not designed for extension - needs to be abstract, final or empty. | 229 | |
| design | DesignForExtension | Method 'getMaximumWidth' is not designed for extension - needs to be abstract, final or empty. | 257 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 268 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getResult' is not designed for extension - needs to be abstract, final or empty. | 34 | |
| design | DesignForExtension | Method 'getMaximumWidth' is not designed for extension - needs to be abstract, final or empty. | 46 | |
| design | DesignForExtension | Method 'append' is not designed for extension - needs to be abstract, final or empty. | 53 | |
| misc | FinalParameters | Parameter address should be final. | 54 | |
| misc | FinalParameters | Parameter theByte should be final. | 54 | |
| design | DesignForExtension | Method 'ignoreForIdenticalOutputCheck' is not designed for extension - needs to be abstract, final or empty. | 61 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 68 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'result' must be private and have accessor methods. | 30 | |
| design | DesignForExtension | Method 'getResult' is not designed for extension - needs to be abstract, final or empty. | 33 | |
| design | DesignForExtension | Method 'getMaximumWidth' is not designed for extension - needs to be abstract, final or empty. | 42 | |
| design | DesignForExtension | Method 'append' is not designed for extension - needs to be abstract, final or empty. | 49 | |
| misc | FinalParameters | Parameter address should be final. | 50 | |
| misc | FinalParameters | Parameter theByte should be final. | 50 | |
| design | DesignForExtension | Method 'ignoreForIdenticalOutputCheck' is not designed for extension - needs to be abstract, final or empty. | 57 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 64 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getResult' is not designed for extension - needs to be abstract, final or empty. | 47 | |
| design | DesignForExtension | Method 'getMaximumWidth' is not designed for extension - needs to be abstract, final or empty. | 56 | |
| design | DesignForExtension | Method 'append' is not designed for extension - needs to be abstract, final or empty. | 63 | |
| misc | FinalParameters | Parameter address should be final. | 64 | |
| misc | FinalParameters | Parameter theByte should be final. | 64 | |
| design | DesignForExtension | Method 'ignoreForIdenticalOutputCheck' is not designed for extension - needs to be abstract, final or empty. | 77 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 84 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 74 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getResult' is not designed for extension - needs to be abstract, final or empty. | 22 | |
| design | DesignForExtension | Method 'getMaximumWidth' is not designed for extension - needs to be abstract, final or empty. | 31 | |
| design | DesignForExtension | Method 'append' is not designed for extension - needs to be abstract, final or empty. | 38 | |
| misc | FinalParameters | Parameter address should be final. | 39 | |
| misc | FinalParameters | Parameter theByte should be final. | 39 | |
| design | DesignForExtension | Method 'ignoreForIdenticalOutputCheck' is not designed for extension - needs to be abstract, final or empty. | 50 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 57 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 131). | 83 | |
| sizes | LineLength | Line is longer than 128 characters (found 133). | 252 | |
| sizes | ParameterNumber | More than 7 parameters (found 8). | 400 | |
| sizes | LineLength | Line is longer than 128 characters (found 132). | 496 | |
| sizes | LineLength | Line is longer than 128 characters (found 132). | 526 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'copyOrWrap' must be private and have accessor methods. | 26 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getCollection' is not designed for extension - needs to be abstract, final or empty. | 55 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'copyOrWrap' must be private and have accessor methods. | 32 | |
| design | VisibilityModifier | Variable 'cachedValues' must be private and have accessor methods. | 35 | |
| design | DesignForExtension | Method 'values' is not designed for extension - needs to be abstract, final or empty. | 106 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getCollection' is not designed for extension - needs to be abstract, final or empty. | 53 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getCollection' is not designed for extension - needs to be abstract, final or empty. | 71 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class ImmutableCollections should be declared as final. | 20 | |
| misc | FinalParameters | Parameter il should be final. | 71 | |
| misc | FinalParameters | Parameter key should be final. | 71 | |
| javadoc | JavadocMethod | Expected @param tag for '<T>'. | 86 | |
| misc | FinalParameters | Parameter il should be final. | 86 | |
| misc | FinalParameters | Parameter key should be final. | 86 | |
| misc | FinalParameters | Parameter comparator should be final. | 86 | |
| misc | FinalParameters | Parameter ic1 should be final. | 98 | |
| misc | FinalParameters | Parameter ic2 should be final. | 98 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 103 | |
| misc | FinalParameters | Parameter ic1 should be final. | 110 | |
| misc | FinalParameters | Parameter c2 should be final. | 110 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 115 | |
| misc | FinalParameters | Parameter c1 should be final. | 122 | |
| misc | FinalParameters | Parameter ic2 should be final. | 122 | |
| misc | FinalParameters | Parameter source should be final. | 135 | |
| misc | FinalParameters | Parameter target should be final. | 135 | |
| misc | FinalParameters | Parameter source should be final. | 148 | |
| misc | FinalParameters | Parameter target should be final. | 148 | |
| misc | FinalParameters | Parameter source should be final. | 161 | |
| misc | FinalParameters | Parameter target should be final. | 161 | |
| misc | FinalParameters | Parameter source should be final. | 174 | |
| misc | FinalParameters | Parameter target should be final. | 174 | |
| misc | FinalParameters | Parameter source should be final. | 187 | |
| misc | FinalParameters | Parameter target should be final. | 187 | |
| misc | FinalParameters | Parameter source should be final. | 200 | |
| misc | FinalParameters | Parameter target should be final. | 200 | |
| javadoc | JavadocMethod | Expected @param tag for '<T>'. | 211 | |
| misc | FinalParameters | Parameter ic should be final. | 211 | |
| javadoc | JavadocMethod | Expected @param tag for '<T>'. | 223 | |
| misc | FinalParameters | Parameter ic should be final. | 223 | |
| misc | FinalParameters | Parameter comparator should be final. | 223 | |
| javadoc | JavadocMethod | Expected @param tag for '<T>'. | 234 | |
| misc | FinalParameters | Parameter ic should be final. | 234 | |
| javadoc | JavadocMethod | Expected @param tag for '<T>'. | 246 | |
| misc | FinalParameters | Parameter ic should be final. | 246 | |
| misc | FinalParameters | Parameter comparator should be final. | 246 | |
| misc | FinalParameters | Parameter ic should be final. | 257 | |
| misc | FinalParameters | Parameter o should be final. | 257 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'entrySet' is not designed for extension - needs to be abstract, final or empty. | 98 | |
| whitespace | WhitespaceAfter | ',' is not followed by whitespace. | 103 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getCollection' is not designed for extension - needs to be abstract, final or empty. | 62 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'entrySet' is not designed for extension - needs to be abstract, final or empty. | 98 | |
| whitespace | WhitespaceAfter | ',' is not followed by whitespace. | 103 | |
| design | DesignForExtension | Method 'values' is not designed for extension - needs to be abstract, final or empty. | 114 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getCollection' is not designed for extension - needs to be abstract, final or empty. | 65 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | FinalParameters | Parameter obj should be final. | 244 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 246 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 248 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 250 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 255 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 258 | |
| misc | FinalParameters | Parameter cmp should be final. | 310 | |
| misc | FinalParameters | Parameter cmp should be final. | 328 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'entrySet' is not designed for extension - needs to be abstract, final or empty. | 99 | |
| misc | FinalParameters | Parameter o1 should be final. | 109 | |
| misc | FinalParameters | Parameter o2 should be final. | 109 | |
| design | DesignForExtension | Method 'values' is not designed for extension - needs to be abstract, final or empty. | 125 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getCollection' is not designed for extension - needs to be abstract, final or empty. | 71 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getCollection' is not designed for extension - needs to be abstract, final or empty. | 82 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | FinalParameters | Parameter path should be final. | 181 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 255 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 132). | 32 | |
| design | DesignForExtension | Method 'always' is not designed for extension - needs to be abstract, final or empty. | 322 | |
| design | DesignForExtension | Method 'filter' is not designed for extension - needs to be abstract, final or empty. | 335 | |
| design | DesignForExtension | Method 'filter' is not designed for extension - needs to be abstract, final or empty. | 353 | |
| design | DesignForExtension | Method 'trace' is not designed for extension - needs to be abstract, final or empty. | 399 | |
| design | DesignForExtension | Method 'trace' is not designed for extension - needs to be abstract, final or empty. | 409 | |
| design | DesignForExtension | Method 'trace' is not designed for extension - needs to be abstract, final or empty. | 420 | |
| design | DesignForExtension | Method 'trace' is not designed for extension - needs to be abstract, final or empty. | 430 | |
| design | DesignForExtension | Method 'trace' is not designed for extension - needs to be abstract, final or empty. | 441 | |
| design | DesignForExtension | Method 'trace' is not designed for extension - needs to be abstract, final or empty. | 453 | |
| design | DesignForExtension | Method 'debug' is not designed for extension - needs to be abstract, final or empty. | 465 | |
| design | DesignForExtension | Method 'debug' is not designed for extension - needs to be abstract, final or empty. | 475 | |
| design | DesignForExtension | Method 'debug' is not designed for extension - needs to be abstract, final or empty. | 486 | |
| design | DesignForExtension | Method 'debug' is not designed for extension - needs to be abstract, final or empty. | 496 | |
| design | DesignForExtension | Method 'debug' is not designed for extension - needs to be abstract, final or empty. | 507 | |
| design | DesignForExtension | Method 'debug' is not designed for extension - needs to be abstract, final or empty. | 519 | |
| design | DesignForExtension | Method 'info' is not designed for extension - needs to be abstract, final or empty. | 531 | |
| design | DesignForExtension | Method 'info' is not designed for extension - needs to be abstract, final or empty. | 541 | |
| design | DesignForExtension | Method 'info' is not designed for extension - needs to be abstract, final or empty. | 552 | |
| design | DesignForExtension | Method 'info' is not designed for extension - needs to be abstract, final or empty. | 562 | |
| design | DesignForExtension | Method 'info' is not designed for extension - needs to be abstract, final or empty. | 573 | |
| design | DesignForExtension | Method 'info' is not designed for extension - needs to be abstract, final or empty. | 585 | |
| design | DesignForExtension | Method 'warn' is not designed for extension - needs to be abstract, final or empty. | 597 | |
| design | DesignForExtension | Method 'warn' is not designed for extension - needs to be abstract, final or empty. | 607 | |
| design | DesignForExtension | Method 'warn' is not designed for extension - needs to be abstract, final or empty. | 618 | |
| design | DesignForExtension | Method 'warn' is not designed for extension - needs to be abstract, final or empty. | 628 | |
| design | DesignForExtension | Method 'warn' is not designed for extension - needs to be abstract, final or empty. | 639 | |
| design | DesignForExtension | Method 'warn' is not designed for extension - needs to be abstract, final or empty. | 651 | |
| design | DesignForExtension | Method 'error' is not designed for extension - needs to be abstract, final or empty. | 663 | |
| design | DesignForExtension | Method 'error' is not designed for extension - needs to be abstract, final or empty. | 673 | |
| design | DesignForExtension | Method 'error' is not designed for extension - needs to be abstract, final or empty. | 684 | |
| design | DesignForExtension | Method 'error' is not designed for extension - needs to be abstract, final or empty. | 694 | |
| design | DesignForExtension | Method 'error' is not designed for extension - needs to be abstract, final or empty. | 705 | |
| design | DesignForExtension | Method 'error' is not designed for extension - needs to be abstract, final or empty. | 717 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 38 | |
| design | DesignForExtension | Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 58 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 75 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 130). | 14 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getStringValue' is not designed for extension - needs to be abstract, final or empty. | 55 | |
| design | DesignForExtension | Method 'getClassValue' is not designed for extension - needs to be abstract, final or empty. | 64 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 70 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 76 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 91 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 141 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getParameterDescriptor' is not designed for extension - needs to be abstract, final or empty. | 84 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 89 | |
| design | DesignForExtension | Method 'getParameterTypes' is not designed for extension - needs to be abstract, final or empty. | 94 | |
| design | DesignForExtension | Method 'getReturnDescriptor' is not designed for extension - needs to be abstract, final or empty. | 102 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 107 | |
| design | DesignForExtension | Method 'getReturnType' is not designed for extension - needs to be abstract, final or empty. | 112 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 118 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 133 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 177 |