The following document contains the results of Checkstyle 6.16 with odl_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | EmptyStatement | 1 | |
| MissingSwitchDefault | 1 | ||
| OverloadMethodsDeclarationOrder | 5 | ||
| imports | UnusedImports | 1 | |
| indentation | Indentation
|
20 | |
| javadoc | JavadocStyle
|
3 | |
| modifier | RedundantModifier | 1 | |
| naming | ConstantName | 1 | |
LocalVariableName
|
8 | ||
MemberName
|
1 | ||
MethodName
|
2 | ||
MethodName
|
2 | ||
ParameterName
|
1 | ||
| sizes | LineLength
|
16 | |
| whitespace | EmptyLineSeparator
|
4 | |
| GenericWhitespace | 1 | ||
| MethodParamPad | 5 | ||
OperatorWrap
|
6 | ||
WhitespaceAround
|
3 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 38 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 38 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | MethodName | Method name 'Init' must match pattern '^[a-z][a-zA-Z0-9_]*$'. | 45 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 46 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | ConstantName | Name 'revision' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 105 | |
| whitespace | EmptyLineSeparator | 'STATIC_INIT' should be separated from previous statement. | 106 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 156 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 160 | |
| sizes | LineLength | Line is longer than 120 characters (found 126). | 168 | |
| sizes | LineLength | Line is longer than 120 characters (found 135). | 176 | |
| whitespace | MethodParamPad | '(' should be on the previous line. | 311 | |
| whitespace | EmptyLineSeparator | 'STATIC_INIT' should be separated from previous statement. | 339 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | MethodParamPad | '(' should be on the previous line. | 143 | |
| whitespace | MethodParamPad | '(' should be on the previous line. | 146 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | GenericWhitespace | GenericWhitespace '>' should followed by whitespace. | 15 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 15 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 137). | 56 | |
| naming | LocalVariableName | Local variable name 'qName' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 82 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | LocalVariableName | Local variable name 'qNameStringEntry' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 52 | |
| sizes | LineLength | Line is longer than 120 characters (found 124). | 59 | |
| whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 123 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '49'. | 59 | |
| coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '54'. | 64 | |
| naming | LocalVariableName | Local variable name 'qNameStringEntry' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 109 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 147 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 148 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 149 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 150 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 151 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 152 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 153 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 124). | 49 | |
| whitespace | EmptyLineSeparator | 'STATIC_INIT' should be separated from previous statement. | 52 | |
| naming | LocalVariableName | Local variable name 'f' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 53 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 83 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 165 | |
| coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '187'. | 206 | |
| coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '194'. | 216 | |
| coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '118'. | 242 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 275 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | OperatorWrap | '+' should be on a new line. | 59 | |
| indentation | Indentation | 'method def modifier' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 67 | |
| indentation | Indentation | 'method def' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 70 | |
| indentation | Indentation | 'method def' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 71 | |
| indentation | Indentation | 'if' have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 72 | |
| indentation | Indentation | 'if' child have incorrect indentation level 24, expected level should be one of the following: 28, 32. | 73 | |
| indentation | Indentation | 'if rcurly' have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 74 | |
| indentation | Indentation | 'if' child have incorrect indentation level 24, expected level should be one of the following: 28, 32. | 75 | |
| indentation | Indentation | 'if rcurly' have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 76 | |
| indentation | Indentation | 'else' child have incorrect indentation level 24, expected level should be one of the following: 28, 32. | 77 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 77 | |
| indentation | Indentation | 'else rcurly' have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 78 | |
| indentation | Indentation | 'method def' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 79 | |
| indentation | Indentation | 'method def rcurly' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 80 | |
| indentation | Indentation | 'object def rcurly' have incorrect indentation level 12, expected level should be one of the following: 16, 20. | 81 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.w3c.dom.Element. | 50 | |
| naming | LocalVariableName | Local variable name 'f' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 65 | |
| coding | MissingSwitchDefault | switch without "default" clause. | 178 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 190 | |
| sizes | LineLength | Line is longer than 120 characters (found 123). | 235 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 121). | 80 | |
| whitespace | OperatorWrap | '?' should be on a new line. | 83 | |
| whitespace | WhitespaceAround | WhitespaceAround: ':' is not preceded with whitespace. | 84 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 169 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 135 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 135 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | MethodParamPad | '(' should be on the previous line. | 85 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 259 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 285 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 311 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 337 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | MethodName | Method name 'Init' must match pattern '^[a-z][a-zA-Z0-9_]*$'. | 64 | |
| coding | EmptyStatement | Empty statement. | 89 | |
| sizes | LineLength | Line is longer than 120 characters (found 125). | 100 | |
| naming | LocalVariableName | Local variable name 'LeafNode1' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 121 | |
| sizes | LineLength | Line is longer than 120 characters (found 125). | 135 | |
| naming | LocalVariableName | Local variable name 'LeafNode2' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 165 | |
| naming | LocalVariableName | Local variable name 'LeafNode3' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 173 | |
| naming | MemberName | Member name 'qName' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 180 | |
| naming | ParameterName | Parameter name 'qName' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 183 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 137). | 93 | |
| whitespace | MethodParamPad | '(' should be on the previous line. | 135 |