The following document contains the results of Checkstyle 7.6.1 with odl_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | IllegalCatch | 8 | |
| imports | CustomImportOrder
|
4 | |
| indentation | Indentation
|
23 | |
| javadoc | JavadocStyle
|
11 | |
| NonEmptyAtclauseDescription | 5 | ||
| modifier | ModifierOrder | 1 | |
| RedundantModifier | 4 | ||
| naming | ParameterName
|
3 | |
| sizes | LineLength
|
79 | |
| whitespace | EmptyLineSeparator
|
1 | |
| MethodParamPad | 1 | ||
OperatorWrap
|
2 | ||
| ParenPad | 2 | ||
SeparatorWrap
|
5 | ||
SeparatorWrap
|
5 | ||
WhitespaceAround
|
3 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocStyle | Javadoc has empty description section. | 18 | |
| sizes | LineLength | Line is longer than 120 characters (found 194). | 26 | |
| sizes | LineLength | Line is longer than 120 characters (found 327). | 30 | |
| sizes | LineLength | Line is longer than 120 characters (found 126). | 51 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocStyle | Javadoc has empty description section. | 14 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 121). | 28 | |
| sizes | LineLength | Line is longer than 120 characters (found 191). | 36 | |
| sizes | LineLength | Line is longer than 120 characters (found 321). | 40 | |
| sizes | LineLength | Line is longer than 120 characters (found 126). | 48 | |
| sizes | LineLength | Line is longer than 120 characters (found 136). | 49 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 59 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 81 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 82 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 86 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 88 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 135). | 16 | |
| sizes | LineLength | Line is longer than 120 characters (found 135). | 21 | |
| sizes | LineLength | Line is longer than 120 characters (found 140). | 29 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 137). | 32 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | OperatorWrap | '+' should be on a new line. | 123 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'final' modifier. | 179 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 180 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 181 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 124). | 33 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 19 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 19 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 26 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 26 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 148 | |
| sizes | LineLength | Line is longer than 120 characters (found 123). | 170 | |
| sizes | LineLength | Line is longer than 120 characters (found 135). | 171 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 202 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 334 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 384 | |
| sizes | LineLength | Line is longer than 120 characters (found 124). | 388 | |
| sizes | LineLength | Line is longer than 120 characters (found 167). | 392 | |
| sizes | LineLength | Line is longer than 120 characters (found 163). | 393 | |
| sizes | LineLength | Line is longer than 120 characters (found 128). | 406 | |
| sizes | LineLength | Line is longer than 120 characters (found 137). | 407 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 417 | |
| sizes | LineLength | Line is longer than 120 characters (found 138). | 418 | |
| sizes | LineLength | Line is longer than 120 characters (found 195). | 424 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 436 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 453 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType' import. Should be before 'org.opendaylight.mdsal.singleton.common.api.ServiceGroupIdentifier'. | 27 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.md.sal.common.api.data.TransactionCommitFailedException' import. Should be before 'org.opendaylight.mdsal.singleton.common.api.ServiceGroupIdentifier'. | 28 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.md.sal.dom.api.DOMDataBroker' import. Should be before 'org.opendaylight.mdsal.singleton.common.api.ServiceGroupIdentifier'. | 29 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.md.sal.dom.api.DOMDataWriteTransaction' import. Should be before 'org.opendaylight.mdsal.singleton.common.api.ServiceGroupIdentifier'. | 30 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 158). | 58 | |
| sizes | LineLength | Line is longer than 120 characters (found 149). | 59 | |
| sizes | LineLength | Line is longer than 120 characters (found 149). | 65 | |
| sizes | LineLength | Line is longer than 120 characters (found 135). | 75 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 81 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 81 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 84 | |
| sizes | LineLength | Line is longer than 120 characters (found 164). | 89 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 113 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 113 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 116 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 117 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 118 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 119 | |
| sizes | LineLength | Line is longer than 120 characters (found 171). | 138 | |
| sizes | LineLength | Line is longer than 120 characters (found 134). | 145 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 175 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 175 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 197 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 197 | |
| sizes | LineLength | Line is longer than 120 characters (found 139). | 208 | |
| sizes | LineLength | Line is longer than 120 characters (found 220). | 210 | |
| sizes | LineLength | Line is longer than 120 characters (found 143). | 259 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 148). | 92 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 95 | |
| naming | ParameterName | Parameter name 'e' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 104 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 131 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 155 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'catch' is not followed by whitespace. | 155 | |
| sizes | LineLength | Line is longer than 120 characters (found 143). | 176 | |
| sizes | LineLength | Line is longer than 120 characters (found 136). | 208 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 209 | |
| sizes | LineLength | Line is longer than 120 characters (found 154). | 217 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 128). | 105 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 126 | |
| sizes | LineLength | Line is longer than 120 characters (found 143). | 131 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 216 | |
| whitespace | OperatorWrap | '||' should be on a new line. | 216 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 217 | |
| sizes | LineLength | Line is longer than 120 characters (found 126). | 220 | |
| sizes | LineLength | Line is longer than 120 characters (found 124). | 242 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 257 | |
| sizes | LineLength | Line is longer than 120 characters (found 139). | 272 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 288 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 290 | |
| sizes | LineLength | Line is longer than 120 characters (found 134). | 293 | |
| sizes | LineLength | Line is longer than 120 characters (found 123). | 296 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 299 | |
| sizes | LineLength | Line is longer than 120 characters (found 126). | 302 | |
| sizes | LineLength | Line is longer than 120 characters (found 128). | 305 | |
| sizes | LineLength | Line is longer than 120 characters (found 128). | 308 | |
| sizes | LineLength | Line is longer than 120 characters (found 139). | 320 | |
| sizes | LineLength | Line is longer than 120 characters (found 156). | 321 | |
| sizes | LineLength | Line is longer than 120 characters (found 137). | 328 | |
| sizes | LineLength | Line is longer than 120 characters (found 156). | 329 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 333 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 336 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 339 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'public' modifier. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 143). | 53 | |
| sizes | LineLength | Line is longer than 120 characters (found 139). | 57 | |
| sizes | LineLength | Line is longer than 120 characters (found 142). | 79 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 80 | |
| sizes | LineLength | Line is longer than 120 characters (found 134). | 81 | |
| sizes | LineLength | Line is longer than 120 characters (found 144). | 89 | |
| sizes | LineLength | Line is longer than 120 characters (found 166). | 94 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 146). | 21 | |
| sizes | LineLength | Line is longer than 120 characters (found 154). | 22 | |
| sizes | LineLength | Line is longer than 120 characters (found 146). | 23 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 31 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 46 | |
| sizes | LineLength | Line is longer than 120 characters (found 126). | 54 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 64 | |
| naming | ParameterName | Parameter name 'k' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 70 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| indentation | Indentation | 'final' have incorrect indentation level 4, expected level should be 8. | 27 | |
| indentation | Indentation | 'final' have incorrect indentation level 4, expected level should be 8. | 28 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 57 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 121 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 133 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | WhitespaceAround | WhitespaceAround: '->' is not preceded with whitespace. | 92 | |
| whitespace | WhitespaceAround | WhitespaceAround: '->' is not preceded with whitespace. | 100 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 125). | 47 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 58 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 105 | |
| indentation | Indentation | 'if' have incorrect indentation level 12, expected level should be 16. | 106 | |
| indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 20. | 107 | |
| indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 20. | 108 | |
| indentation | Indentation | 'if rcurly' have incorrect indentation level 12, expected level should be 16. | 109 | |
| indentation | Indentation | 'else' child have incorrect indentation level 16, expected level should be 20. | 110 | |
| indentation | Indentation | 'else' child have incorrect indentation level 16, expected level should be 20. | 111 | |
| indentation | Indentation | 'else rcurly' 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 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 114 | |
| indentation | Indentation | 'if' have incorrect indentation level 12, expected level should be 16. | 115 | |
| indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 20. | 116 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 116 | |
| indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 20. | 117 | |
| indentation | Indentation | 'if rcurly' have incorrect indentation level 12, expected level should be 16. | 118 | |
| indentation | Indentation | 'else' child have incorrect indentation level 16, expected level should be 20. | 119 | |
| indentation | Indentation | 'else rcurly' have incorrect indentation level 12, expected level should be 16. | 120 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 121 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 122 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 123 | |
| sizes | LineLength | Line is longer than 120 characters (found 123). | 123 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 124 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 125 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 80 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 91 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | MethodParamPad | '(' is preceded with whitespace. | 25 | |
| naming | ParameterName | Parameter name 'e' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 46 |