The following document contains the results of Checkstyle 6.16 with odl_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | IllegalCatch | 2 | |
| MissingSwitchDefault | 1 | ||
| OverloadMethodsDeclarationOrder | 5 | ||
| indentation | Indentation
|
7 | |
| javadoc | JavadocStyle
|
3 | |
| modifier | RedundantModifier | 1 | |
| naming | ConstantName | 1 | |
LocalVariableName
|
7 | ||
MemberName
|
1 | ||
MethodName
|
2 | ||
MethodName
|
2 | ||
ParameterName
|
1 | ||
| sizes | LineLength
|
13 | |
| whitespace | EmptyLineSeparator
|
4 | |
| MethodParamPad | 7 | ||
OperatorWrap
|
4 | ||
| ParenPad | 1 | ||
WhitespaceAround
|
1 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | MethodName | Method name 'Init' must match pattern '^[a-z][a-zA-Z0-9_]*$'. | 43 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 44 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | ConstantName | Name 'revision' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 106 | |
| whitespace | EmptyLineSeparator | 'STATIC_INIT' should be separated from previous statement. | 107 | |
| whitespace | EmptyLineSeparator | 'STATIC_INIT' should be separated from previous statement. | 116 | |
| whitespace | MethodParamPad | '(' should be on the previous line. | 307 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | MethodParamPad | '(' should be on the previous line. | 135 | |
| whitespace | MethodParamPad | '(' should be on the previous line. | 138 |
| 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 |
|---|---|---|---|---|
| whitespace | MethodParamPad | '(' should be on the previous line. | 108 | |
| whitespace | MethodParamPad | '(' should be on the previous line. | 159 | |
| sizes | LineLength | Line is longer than 120 characters (found 125). | 192 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 300 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | IllegalCatch | Catching 'RuntimeException' is not allowed. | 80 | |
| coding | IllegalCatch | Catching 'RuntimeException' is not allowed. | 104 |
| 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 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 80 | |
| 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 |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 121). | 71 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 130 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 17 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocStyle | Javadoc has empty description section. | 90 | |
| coding | MissingSwitchDefault | switch without "default" clause. | 192 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 204 | |
| sizes | LineLength | Line is longer than 120 characters (found 123). | 249 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 130 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 130 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | MethodParamPad | '(' should be on the previous line. | 83 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 242 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 265 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 288 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 311 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | MethodName | Method name 'Init' must match pattern '^[a-z][a-zA-Z0-9_]*$'. | 63 | |
| sizes | LineLength | Line is longer than 120 characters (found 125). | 94 | |
| naming | LocalVariableName | Local variable name 'LeafNode1' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 115 | |
| sizes | LineLength | Line is longer than 120 characters (found 125). | 129 | |
| naming | LocalVariableName | Local variable name 'LeafNode2' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 159 | |
| naming | LocalVariableName | Local variable name 'LeafNode3' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 167 | |
| naming | MemberName | Member name 'qName' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 174 | |
| naming | ParameterName | Parameter name 'qName' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 177 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 137). | 77 | |
| whitespace | MethodParamPad | '(' should be on the previous line. | 119 |