The following document contains the results of Checkstyle 6.2 with odl_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| imports | CustomImportOrder
|
11 | |
| indentation | Indentation
|
4 | |
| javadoc | JavadocMethod
|
4 | |
| JavadocParagraph | 65 | ||
| naming | ParameterName
|
1 | |
| sizes | LineLength
|
3 | |
| whitespace | EmptyLineSeparator
|
22 | |
OperatorWrap
|
2 | ||
WhitespaceAround
|
4 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 14 | |
| javadoc | SummaryJavadoc | First sentence should be present. | 52 | |
| javadoc | SummaryJavadoc | First sentence should be present. | 59 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 45 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 60 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 81 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 13 | |
| sizes | LineLength | Line is longer than 120 characters (found 166). | 49 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 15 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 17 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 21 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 26 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 34 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 41 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 51 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 58 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 64 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 70 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 75 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 78 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 91 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 93 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 96 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 119 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 122 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 128 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 135 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 139 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 173 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 191 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 197 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 275 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 442 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 456 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 460 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Preconditions' import. | 14 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 17 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 19 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 22 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 36 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 15 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 24 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yangtools.yang.common.RpcError.ErrorType' import. | 12 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 15 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 18 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 22 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 14 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 17 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 19 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 29 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 55 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 15 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 25 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 48 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 50 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 53 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| indentation | Indentation | 'method def modifier' have incorrect indentation level 16, expected level should be one of the following: 8, 12. | 24 | |
| indentation | Indentation | 'method def' child have incorrect indentation level 20, expected level should be one of the following: 12, 16. | 26 | |
| indentation | Indentation | 'method def rcurly' have incorrect indentation level 16, expected level should be one of the following: 8, 12. | 27 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 14 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 18 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 30 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 33 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 17 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 32 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 38 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 53 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 76 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 82 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | OperatorWrap | '+' should be on a new line. | 28 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 29 | |
| sizes | LineLength | Line is longer than 120 characters (found 136). | 31 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 14 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 16 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 12 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 24 | |
| javadoc | SummaryJavadoc | Forbidden summary fragment. | 30 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 13 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 51 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 98 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 31 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 32 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 34 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 11 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 12 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 19 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 26 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 30 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 33 | |
| javadoc | SummaryJavadoc | Forbidden summary fragment. | 50 | |
| javadoc | SummaryJavadoc | Forbidden summary fragment. | 59 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 63 | |
| naming | ParameterName | Parameter name 'o' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 69 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| javadoc | SummaryJavadoc | First sentence should be present. | 25 | |
| indentation | Indentation | 'method def modifier' have incorrect indentation level 3, expected level should be 4. | 28 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 11 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| imports | CustomImportOrder | 'org.opendaylight.yangtools.concepts.ObjectRegistration'should be separated from previous import group. | 11 | |
| sizes | LineLength | Line is longer than 120 characters (found 123). | 24 | |
| javadoc | SummaryJavadoc | First sentence should be present. | 27 | |
| javadoc | SummaryJavadoc | First sentence should be present. | 32 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 11 | |
| javadoc | JavadocParagraph | <p> tag should be precede with empty line. | 37 |