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 | 2 | ||
| naming | ParameterName
|
3 | |
| sizes | LineLength
|
71 | |
| whitespace | EmptyLineSeparator
|
1 | |
| MethodParamPad | 3 | ||
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 |
|---|---|---|---|---|
| whitespace | MethodParamPad | '(' is preceded with whitespace. | 87 | |
| whitespace | MethodParamPad | '(' is preceded with whitespace. | 136 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 162 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 176 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 18 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 18 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 25 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 25 |
| 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). | 333 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 383 | |
| sizes | LineLength | Line is longer than 120 characters (found 124). | 387 | |
| sizes | LineLength | Line is longer than 120 characters (found 167). | 391 | |
| sizes | LineLength | Line is longer than 120 characters (found 163). | 392 | |
| sizes | LineLength | Line is longer than 120 characters (found 128). | 405 | |
| sizes | LineLength | Line is longer than 120 characters (found 137). | 406 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 416 | |
| sizes | LineLength | Line is longer than 120 characters (found 138). | 417 | |
| sizes | LineLength | Line is longer than 120 characters (found 195). | 423 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 435 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 452 |
| 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'. | 28 | |
| 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'. | 29 | |
| 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'. | 30 | |
| 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'. | 31 |
| 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 |
|---|---|---|---|---|
| naming | ParameterName | Parameter name 'e' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 107 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 134 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 158 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'catch' is not followed by whitespace. | 158 | |
| sizes | LineLength | Line is longer than 120 characters (found 136). | 212 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 213 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 128). | 107 | |
| sizes | LineLength | Line is longer than 120 characters (found 143). | 133 | |
| whitespace | ParenPad | '(' is followed by whitespace. | 218 | |
| whitespace | OperatorWrap | '||' should be on a new line. | 218 | |
| whitespace | ParenPad | ')' is preceded with whitespace. | 219 | |
| sizes | LineLength | Line is longer than 120 characters (found 126). | 222 | |
| sizes | LineLength | Line is longer than 120 characters (found 124). | 244 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 259 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 291 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 293 | |
| sizes | LineLength | Line is longer than 120 characters (found 134). | 296 | |
| sizes | LineLength | Line is longer than 120 characters (found 123). | 299 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 302 | |
| sizes | LineLength | Line is longer than 120 characters (found 126). | 305 | |
| sizes | LineLength | Line is longer than 120 characters (found 128). | 308 | |
| sizes | LineLength | Line is longer than 120 characters (found 128). | 311 | |
| sizes | LineLength | Line is longer than 120 characters (found 139). | 323 | |
| sizes | LineLength | Line is longer than 120 characters (found 156). | 324 | |
| sizes | LineLength | Line is longer than 120 characters (found 137). | 331 | |
| sizes | LineLength | Line is longer than 120 characters (found 156). | 332 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 336 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 339 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 342 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'public' modifier. | 38 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 143). | 54 | |
| sizes | LineLength | Line is longer than 120 characters (found 139). | 58 | |
| sizes | LineLength | Line is longer than 120 characters (found 142). | 80 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 81 | |
| sizes | LineLength | Line is longer than 120 characters (found 134). | 82 | |
| sizes | LineLength | Line is longer than 120 characters (found 144). | 90 | |
| sizes | LineLength | Line is longer than 120 characters (found 166). | 95 |
| 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. | 28 | |
| indentation | Indentation | 'final' have incorrect indentation level 4, expected level should be 8. | 29 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 58 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 122 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 134 |
| 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). | 49 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 60 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 107 | |
| indentation | Indentation | 'if' have incorrect indentation level 12, expected level should be 16. | 108 | |
| indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 20. | 109 | |
| indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 20. | 110 | |
| indentation | Indentation | 'if rcurly' have incorrect indentation level 12, expected level should be 16. | 111 | |
| indentation | Indentation | 'else' child have incorrect indentation level 16, expected level should be 20. | 112 | |
| indentation | Indentation | 'else' child have incorrect indentation level 16, expected level should be 20. | 113 | |
| indentation | Indentation | 'else rcurly' have incorrect indentation level 12, expected level should be 16. | 114 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 115 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 116 | |
| indentation | Indentation | 'if' have incorrect indentation level 12, expected level should be 16. | 117 | |
| indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 20. | 118 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 118 | |
| indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 20. | 119 | |
| indentation | Indentation | 'if rcurly' have incorrect indentation level 12, expected level should be 16. | 120 | |
| indentation | Indentation | 'else' child have incorrect indentation level 16, expected level should be 20. | 121 | |
| indentation | Indentation | 'else rcurly' have incorrect indentation level 12, expected level should be 16. | 122 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 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 | |
| sizes | LineLength | Line is longer than 120 characters (found 123). | 125 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 126 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 127 |
| 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 |