The following document contains the results of Checkstyle 7.6.1 with odl_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | IllegalCatch | 1 | |
| IllegalThrows | 2 | ||
| OverloadMethodsDeclarationOrder | 2 | ||
| imports | CustomImportOrder
|
6 | |
| indentation | Indentation
|
26 | |
| javadoc | JavadocStyle
|
1 | |
| modifier | RedundantModifier | 6 | |
| naming | AbbreviationAsWordInName
|
1 | |
| ConstantName | 3 | ||
ParameterName
|
1 | ||
| sizes | LineLength
|
29 | |
| whitespace | EmptyLineSeparator
|
5 | |
WhitespaceAround
|
22 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 125). | 30 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 31 | |
| indentation | Indentation | 'method def modifier' have incorrect indentation level 5, expected level should be 4. | 45 | |
| sizes | LineLength | Line is longer than 120 characters (found 126). | 46 | |
| indentation | Indentation | 'if' have incorrect indentation level 9, expected level should be 8. | 47 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 47 | |
| indentation | Indentation | 'if' child have incorrect indentation level 13, expected level should be 12. | 48 | |
| indentation | Indentation | 'if rcurly' have incorrect indentation level 9, expected level should be 8. | 49 | |
| indentation | Indentation | 'method def' child have incorrect indentation level 9, expected level should be 8. | 50 | |
| indentation | Indentation | 'method def rcurly' have incorrect indentation level 5, expected level should be 4. | 51 | |
| coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '29'. | 61 | |
| coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '45'. | 71 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | ParameterName | Parameter name 'e' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 18 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | 'com.google.common.base.Preconditions' should be separated from previous import group by one line. | 13 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 141). | 40 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 47 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | 'com.google.common.base.Preconditions' should be separated from previous import group by one line. | 15 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 74 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 74 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 81 | |
| whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 84 | |
| whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 87 | |
| whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 91 | |
| whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 95 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 97 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 121). | 33 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 54 | |
| sizes | LineLength | Line is longer than 120 characters (found 142). | 55 | |
| sizes | LineLength | Line is longer than 120 characters (found 124). | 56 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 70 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 71 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 71 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 72 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 73 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 74 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 75 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 76 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 77 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 78 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 79 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 104 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 105 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 106 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 107 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 108 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 109 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 110 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 111 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 112 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 113 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 127). | 45 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 47 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 75 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 76 | |
| sizes | LineLength | Line is longer than 120 characters (found 128). | 79 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 82 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 23 | |
| javadoc | JavadocStyle | Javadoc has empty description section. | 23 | |
| whitespace | EmptyLineSeparator | 'CTOR_DEF' should be separated from previous statement. | 27 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'final' modifier. | 16 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'final' modifier. | 16 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 16 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | 'io.netty.buffer.ByteBuf' should be separated from previous import group by one line. | 14 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | AbbreviationAsWordInName | Abbreviation in name 'testFormatTlvASCII' must contain no more than '5' consecutive capital letters. | 78 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 87 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 87 | |
| coding | IllegalThrows | Throwing 'Throwable' is not allowed. | 88 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 98 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 98 | |
| coding | IllegalThrows | Throwing 'Throwable' is not allowed. | 99 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'final' modifier. | 30 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 50 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 124). | 17 | |
| naming | ConstantName | Name 'context' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 18 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 27 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 27 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 124). | 19 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 144). | 35 | |
| sizes | LineLength | Line is longer than 120 characters (found 133). | 50 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 60 | |
| sizes | LineLength | Line is longer than 120 characters (found 133). | 70 | |
| sizes | LineLength | Line is longer than 120 characters (found 134). | 80 | |
| sizes | LineLength | Line is longer than 120 characters (found 137). | 90 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 36 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 45 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | 'org.junit.Before' should be separated from previous import group by one line. | 11 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 124). | 30 | |
| sizes | LineLength | Line is longer than 120 characters (found 140). | 39 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 68 | |
| sizes | LineLength | Line is longer than 120 characters (found 140). | 71 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | 'io.netty.buffer.ByteBuf' should be separated from previous import group by one line. | 13 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 49 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 56 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 56 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 62 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 62 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 68 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 68 | |
| whitespace | WhitespaceAround | WhitespaceAround: '-' is not preceded with whitespace. | 70 | |
| whitespace | WhitespaceAround | WhitespaceAround: '-' is not followed by whitespace. | 70 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 95 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 123). | 58 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | 'io.netty.buffer.ByteBuf' should be separated from previous import group by one line. | 14 | |
| naming | ConstantName | Name 'descriptionTlvType' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 37 | |
| naming | ConstantName | Name 'otherTlvType' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 38 | |
| sizes | LineLength | Line is longer than 120 characters (found 125). | 59 |