The following document contains the results of Checkstyle 6.16 with odl_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | IllegalCatch | 1 | |
| IllegalThrows | 3 | ||
| imports | CustomImportOrder
|
7 | |
| indentation | Indentation
|
53 | |
| javadoc | JavadocStyle
|
2 | |
| modifier | RedundantModifier | 2 | |
| naming | AbbreviationAsWordInName
|
1 | |
| ConstantName | 3 | ||
ParameterName
|
1 | ||
| sizes | LineLength
|
37 | |
| whitespace | EmptyLineSeparator
|
5 | |
WhitespaceAround
|
22 |
| 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. | 13 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 141). | 38 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 45 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | 'com.google.common.base.Preconditions' should be separated from previous import group. | 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). | 31 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 52 | |
| sizes | LineLength | Line is longer than 120 characters (found 142). | 53 | |
| sizes | LineLength | Line is longer than 120 characters (found 124). | 54 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 68 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 69 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 70 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 70 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 71 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 72 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 73 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 74 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 75 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 76 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 77 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 78 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 103 | |
| indentation | Indentation | 'if' have incorrect indentation level 12, expected level should be 16. | 104 | |
| indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 20. | 105 | |
| indentation | Indentation | 'if rcurly' have incorrect indentation level 12, expected level should be 16. | 106 | |
| indentation | Indentation | 'else' child have incorrect indentation level 16, expected level should be 20. | 107 | |
| indentation | Indentation | 'else rcurly' 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 | 'block' child have incorrect indentation level 12, expected level should be 16. | 111 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 112 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 113 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 114 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 115 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 116 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 127). | 51 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 53 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 81 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 82 | |
| sizes | LineLength | Line is longer than 120 characters (found 128). | 85 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 109 | |
| sizes | LineLength | Line is longer than 120 characters (found 137). | 113 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 141 |
| 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 |
|---|---|---|---|---|
| javadoc | JavadocStyle | First sentence should end with a period. | 22 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 65 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 66 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 67 | |
| sizes | LineLength | Line is longer than 120 characters (found 132). | 67 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 68 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 69 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 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 130). | 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 | |
| sizes | LineLength | Line is longer than 120 characters (found 137). | 75 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 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 | |
| sizes | LineLength | Line is longer than 120 characters (found 151). | 84 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 88 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 89 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 90 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 91 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 92 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 93 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 94 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 95 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 96 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 97 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 98 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 99 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 100 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 101 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | 'io.netty.buffer.ByteBuf' should be separated from previous import group. | 12 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 24 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 39 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 52 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 65 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 72 | |
| whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 72 | |
| coding | IllegalThrows | Throwing 'Throwable' is not allowed. | 73 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | 'io.netty.buffer.ByteBuf' should be separated from previous import group. | 14 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | AbbreviationAsWordInName | Abbreviation in name 'testFormatTlvASCII' must contain no more than '4' 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 |
|---|---|---|---|---|
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 48 |
| 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. | 35 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 44 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | 'org.junit.Before' should be separated from previous import group. | 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. | 13 | |
| sizes | LineLength | Line is longer than 120 characters (found 139). | 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. | 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 |