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 |