The following document contains the results of Checkstyle 6.16 with odl_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | EmptyBlock
|
1 | |
| coding | IllegalCatch | 1 | |
| MultipleVariableDeclarations | 1 | ||
| OverloadMethodsDeclarationOrder | 2 | ||
| VariableDeclarationUsageDistance | 1 | ||
| imports | CustomImportOrder
|
2 | |
| indentation | Indentation
|
1 | |
| javadoc | JavadocParagraph
|
14 | |
JavadocStyle
|
8 | ||
| JavadocTagContinuationIndentation | 2 | ||
| modifier | RedundantModifier | 2 | |
| naming | LocalVariableName
|
7 | |
ParameterName
|
1 | ||
| sizes | LineLength
|
3 | |
| whitespace | EmptyLineSeparator
|
1 | |
FileTabCharacter
|
1 | ||
| MethodParamPad | 2 | ||
OperatorWrap
|
1 | ||
| ParenPad | 204 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | ParenPad | '(' is followed by whitespace. | 53 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 53 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 53 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 53 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 72 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 72 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 72 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 72 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 73 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 73 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | ParenPad | '(' is followed by whitespace. | 19 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 19 | |
| sizes | LineLength | Line is longer than 120 characters (found 124). | 23 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 31 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 45 | |
| indentation | Indentation | 'method def' child have incorrect indentation level 16, expected level should be 8. | 100 | |
| sizes | LineLength | Line is longer than 120 characters (found 132). | 100 | |
| whitespace | FileTabCharacter | Line contains a tab character. | 100 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 151 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 151 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 162 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 162 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 199 | |
| coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '111'. | 221 | |
| coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '287'. | 314 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 336 | |
| coding | IllegalCatch | Catching 'RuntimeException' is not allowed. | 360 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 369 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 386 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 388 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 150). | 152 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | 'java.net.URI' should be separated from previous import group. | 14 | |
| whitespace | EmptyLineSeparator | ',' should be separated from previous statement. | 19 | |
| coding | MultipleVariableDeclarations | Each variable declaration must be in its own statement. | 19 | |
| naming | LocalVariableName | Local variable name 'qName' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 29 | |
| naming | LocalVariableName | Local variable name 'qName' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 37 | |
| naming | LocalVariableName | Local variable name 'qName' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 45 | |
| naming | LocalVariableName | Local variable name 'A' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 64 | |
| coding | VariableDeclarationUsageDistance | Distance between variable 'B' declaration and its first usage is 4, but allowed 3. Consider to make that variable as final if you still need to store its value in advance (before method calls that might do side effect on original value). | 65 | |
| naming | LocalVariableName | Local variable name 'B' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 65 | |
| naming | LocalVariableName | Local variable name 'a' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 67 | |
| naming | LocalVariableName | Local variable name 'b' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 68 | |
| blocks | EmptyBlock | Empty catch block. | 137 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'public' modifier. | 76 | |
| naming | ParameterName | Parameter name 'o' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 133 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocParagraph | <p> tag should be preceded with an empty line. | 52 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 96 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 105 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | ParenPad | '(' is followed by whitespace. | 37 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 38 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 77 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 79 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 137 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 137 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 154 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 154 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 163 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 163 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 179 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 179 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 188 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 188 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 190 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 190 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 203 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 203 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 204 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 205 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 215 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 222 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 223 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 224 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 225 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 238 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 238 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 239 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 239 | |
| javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 249 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 256 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 257 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 258 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 259 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 267 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 267 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 277 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 277 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 281 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 283 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 285 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 285 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 287 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 287 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 290 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 290 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 296 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 296 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 307 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 307 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 308 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 308 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 324 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 325 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 326 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 326 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 336 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 336 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 337 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 337 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 349 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 349 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 350 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 350 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 361 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 362 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 363 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 363 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 379 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 380 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 381 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 381 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 390 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 390 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 391 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 391 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 400 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 400 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 403 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 403 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 418 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 421 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 423 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 425 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | 'java.io.ByteArrayInputStream' should be separated from previous import group. | 14 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 33 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 33 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 34 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 34 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 35 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 35 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 36 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 36 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 38 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 38 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 39 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 39 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 44 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 44 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 45 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 45 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 47 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 47 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 48 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 48 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 49 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 49 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 50 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 50 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 52 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 52 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 53 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 54 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 55 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 56 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 57 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 58 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 59 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 60 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 61 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 61 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 66 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 66 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 68 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 68 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 69 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 69 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 71 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 71 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 72 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 73 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 74 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 75 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 76 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 76 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 81 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 81 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 83 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 83 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 84 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 84 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 87 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 87 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 88 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 88 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 90 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 90 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 91 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 92 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 93 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 94 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 99 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 99 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 101 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 101 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 102 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 102 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 106 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 106 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 108 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 109 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 110 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 111 | |
| whitespace | MethodParamPad | '(' should be on the previous line. | 127 | |
| whitespace | MethodParamPad | '(' should be on the previous line. | 129 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 135 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 142 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 142 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 153 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 153 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 155 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 155 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 166 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 167 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 170 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 172 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 174 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 174 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 175 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 176 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 177 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 177 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 178 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 178 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 179 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 179 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 180 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 180 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 181 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 181 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 182 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 182 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 183 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 183 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 184 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 184 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 187 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 187 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 188 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 188 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 189 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 189 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 17 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 17 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 22 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 22 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 27 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 27 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 32 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 32 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 37 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 73 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 73 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'private' modifier. | 40 |