The following document contains the results of Checkstyle 7.6.1 with odl_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | OverloadMethodsDeclarationOrder | 1 | |
| imports | CustomImportOrder
|
1 | |
| indentation | Indentation
|
5 | |
| javadoc | JavadocStyle
|
1 | |
| modifier | RedundantModifier | 2 | |
| naming | ConstantName | 12 | |
LocalVariableName
|
1 | ||
ParameterName
|
1 | ||
| sizes | LineLength
|
79 | |
| whitespace | MethodParamPad | 1 | |
| ParenPad | 1 | ||
SeparatorWrap
|
7 | ||
SeparatorWrap
|
7 | ||
WhitespaceAround
|
5 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 147). | 51 | |
| sizes | LineLength | Line is longer than 120 characters (found 152). | 52 | |
| sizes | LineLength | Line is longer than 120 characters (found 143). | 53 | |
| sizes | LineLength | Line is longer than 120 characters (found 127). | 69 | |
| sizes | LineLength | Line is longer than 120 characters (found 127). | 94 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 121 | |
| sizes | LineLength | Line is longer than 120 characters (found 156). | 122 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 126 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 126 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 132 | |
| sizes | LineLength | Line is longer than 120 characters (found 137). | 140 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 140 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 140 | |
| sizes | LineLength | Line is longer than 120 characters (found 138). | 142 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 148 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 130). | 26 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 125). | 34 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | 'io.netty.buffer.ByteBuf' should be separated from previous import group by one line. | 13 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | ParenPad | '(' is followed by whitespace. | 74 | |
| sizes | LineLength | Line is longer than 120 characters (found 145). | 75 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 75 | |
| sizes | LineLength | Line is longer than 120 characters (found 461). | 76 | |
| sizes | LineLength | Line is longer than 120 characters (found 140). | 77 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 78 | |
| sizes | LineLength | Line is longer than 120 characters (found 150). | 79 | |
| sizes | LineLength | Line is longer than 120 characters (found 471). | 80 | |
| sizes | LineLength | Line is longer than 120 characters (found 144). | 81 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 93 | |
| sizes | LineLength | Line is longer than 120 characters (found 133). | 113 | |
| naming | LocalVariableName | Local variable name 'i' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 124 | |
| sizes | LineLength | Line is longer than 120 characters (found 155). | 166 | |
| naming | ParameterName | Parameter name 'mPathSupported' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 166 | |
| sizes | LineLength | Line is longer than 120 characters (found 125). | 188 | |
| coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '166'. | 216 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 233 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 233 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 236 | |
| sizes | LineLength | Line is longer than 120 characters (found 185). | 245 | |
| sizes | LineLength | Line is longer than 120 characters (found 152). | 251 | |
| sizes | LineLength | Line is longer than 120 characters (found 145). | 256 | |
| sizes | LineLength | Line is longer than 120 characters (found 132). | 257 | |
| sizes | LineLength | Line is longer than 120 characters (found 251). | 258 | |
| sizes | LineLength | Line is longer than 120 characters (found 145). | 261 | |
| sizes | LineLength | Line is longer than 120 characters (found 140). | 262 | |
| sizes | LineLength | Line is longer than 120 characters (found 260). | 263 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | ConstantName | Name 'LU_REACH_NLRI_IPv4' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 69 | |
| indentation | Indentation | 'member def modifier' have incorrect indentation level 3, expected level should be 4. | 84 | |
| naming | ConstantName | Name 'LU_UNREACH_NLRI_IPv4' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 84 | |
| naming | ConstantName | Name 'LU_REACH_NLRI_IPv4_ADD_PATH' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 103 | |
| naming | ConstantName | Name 'LU_UNREACH_NLRI_IPv4_ADD_PATH' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 120 | |
| naming | ConstantName | Name 'LU_REACH_NLRI_IPv6' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 139 | |
| indentation | Indentation | 'member def modifier' have incorrect indentation level 3, expected level should be 4. | 157 | |
| naming | ConstantName | Name 'LU_UNREACH_NLRI_IPv6' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 157 | |
| naming | ConstantName | Name 'LU_REACH_NLRI_IPv6_ADD_PATH' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 178 | |
| indentation | Indentation | 'member def modifier' have incorrect indentation level 3, expected level should be 4. | 197 | |
| naming | ConstantName | Name 'LU_UNREACH_NLRI_IPv6_ADD_PATH' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 197 | |
| naming | ConstantName | Name 'IPv4_PREFIX' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 212 | |
| naming | ConstantName | Name 'IPv6_PREFIX' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 213 | |
| whitespace | MethodParamPad | '(' should be on the previous line. | 265 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 310 | |
| indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 510 | |
| indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 518 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 121). | 44 | |
| sizes | LineLength | Line is longer than 120 characters (found 250). | 50 | |
| sizes | LineLength | Line is longer than 120 characters (found 278). | 51 | |
| sizes | LineLength | Line is longer than 120 characters (found 126). | 56 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | ConstantName | Name 'IPv4_PREFIX' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 62 | |
| sizes | LineLength | Line is longer than 120 characters (found 143). | 69 | |
| sizes | LineLength | Line is longer than 120 characters (found 150). | 70 | |
| sizes | LineLength | Line is longer than 120 characters (found 140). | 72 | |
| sizes | LineLength | Line is longer than 120 characters (found 188). | 74 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 74 | |
| sizes | LineLength | Line is longer than 120 characters (found 220). | 75 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 75 | |
| sizes | LineLength | Line is longer than 120 characters (found 238). | 76 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 76 | |
| sizes | LineLength | Line is longer than 120 characters (found 155). | 77 | |
| sizes | LineLength | Line is longer than 120 characters (found 135). | 87 | |
| sizes | LineLength | Line is longer than 120 characters (found 143). | 101 | |
| sizes | LineLength | Line is longer than 120 characters (found 152). | 131 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 161 | |
| sizes | LineLength | Line is longer than 120 characters (found 149). | 172 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 194 | |
| sizes | LineLength | Line is longer than 120 characters (found 148). | 197 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 125). | 44 | |
| sizes | LineLength | Line is longer than 120 characters (found 142). | 50 | |
| sizes | LineLength | Line is longer than 120 characters (found 128). | 51 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 52 | |
| sizes | LineLength | Line is longer than 120 characters (found 141). | 53 | |
| sizes | LineLength | Line is longer than 120 characters (found 175). | 58 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | ConstantName | Name 'IPv6_PREFIX' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 62 | |
| sizes | LineLength | Line is longer than 120 characters (found 143). | 69 | |
| sizes | LineLength | Line is longer than 120 characters (found 150). | 70 | |
| sizes | LineLength | Line is longer than 120 characters (found 152). | 72 | |
| sizes | LineLength | Line is longer than 120 characters (found 124). | 73 | |
| sizes | LineLength | Line is longer than 120 characters (found 188). | 74 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 74 | |
| sizes | LineLength | Line is longer than 120 characters (found 224). | 75 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 75 | |
| sizes | LineLength | Line is longer than 120 characters (found 144). | 76 | |
| sizes | LineLength | Line is longer than 120 characters (found 154). | 77 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 77 | |
| sizes | LineLength | Line is longer than 120 characters (found 135). | 88 | |
| sizes | LineLength | Line is longer than 120 characters (found 143). | 102 | |
| sizes | LineLength | Line is longer than 120 characters (found 152). | 132 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 162 | |
| sizes | LineLength | Line is longer than 120 characters (found 149). | 173 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 195 | |
| sizes | LineLength | Line is longer than 120 characters (found 160). | 198 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 124). | 37 | |
| sizes | LineLength | Line is longer than 120 characters (found 161). | 40 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 159). | 24 | |
| sizes | LineLength | Line is longer than 120 characters (found 159). | 25 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 125). | 25 | |
| sizes | LineLength | Line is longer than 120 characters (found 143). | 27 | |
| sizes | LineLength | Line is longer than 120 characters (found 144). | 28 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 136). | 26 | |
| sizes | LineLength | Line is longer than 120 characters (found 136). | 27 |