The following document contains the results of Checkstyle 6.2 with odl_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | RightCurly
|
1 | |
| RightCurly | 1 | ||
| coding | OverloadMethodsDeclarationOrder | 1 | |
| imports | CustomImportOrder
|
40 | |
| indentation | Indentation
|
10 | |
| javadoc | JavadocMethod
|
5 | |
| JavadocParagraph | 12 | ||
| modifier | ModifierOrder | 1 | |
| naming | AbbreviationAsWordInName
|
7 | |
LocalVariableName
|
1 | ||
MemberName
|
2 | ||
ParameterName
|
2 | ||
| sizes | LineLength
|
20 | |
| whitespace | EmptyLineSeparator
|
23 | |
WhitespaceAround
|
1 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| imports | CustomImportOrder | 'org.opendaylight.yangtools.sal.binding.model.api.Type'should be separated from previous import group. | 11 | |
| javadoc | SummaryJavadoc | First sentence should be present. | 13 |
| 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 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 15 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 16 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 17 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 18 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 19 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 20 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 21 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 22 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 23 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 24 | |
| sizes | LineLength | Line is longer than 120 characters (found 123). | 96 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 112 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 129 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 155 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 181 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 195 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 219 | |
| coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '169'. | 234 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 327 | |
| naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 424 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 424 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 491 | |
| whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 644 | |
| whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 648 | |
| whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 652 | |
| javadoc | SummaryJavadoc | First sentence should be present. | 659 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| naming | ParameterName | Parameter name 't' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 49 | |
| naming | ParameterName | Parameter name 't' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 53 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| javadoc | SummaryJavadoc | First sentence should be present. | 10 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 11 | |
| javadoc | SummaryJavadoc | First sentence should be present. | 18 |
| 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. | 11 |
| 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. | 18 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 19 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 20 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 21 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 22 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 23 | |
| indentation | Indentation | 'method def modifier' have incorrect indentation level 16, expected level should be one of the following: 8, 12. | 40 | |
| indentation | Indentation | 'method def' child have incorrect indentation level 20, expected level should be one of the following: 12, 16. | 42 | |
| indentation | Indentation | 'method def rcurly' have incorrect indentation level 16, expected level should be one of the following: 8, 12. | 43 | |
| javadoc | SummaryJavadoc | First sentence should be present. | 60 | |
| javadoc | SummaryJavadoc | First sentence should be present. | 94 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 105 | |
| javadoc | SummaryJavadoc | First sentence should be present. | 117 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 224 | |
| modifier | ModifierOrder | '@Nullable' annotation modifier does not preceed non-annotation modifiers. | 224 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 226 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 234 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 262 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 266 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 290 | |
| javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 334 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 60 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 78 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 79 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 80 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 81 | |
| indentation | Indentation | 'case' child have incorrect indentation level 8, expected level should be 12. | 82 | |
| indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be 16. | 83 | |
| sizes | LineLength | Line is longer than 120 characters (found 124). | 87 | |
| naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 88 | |
| sizes | LineLength | Line is longer than 120 characters (found 138). | 89 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 105 | |
| sizes | LineLength | Line is longer than 120 characters (found 123). | 129 |
| 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 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 13 | |
| sizes | LineLength | Line is longer than 120 characters (found 148). | 27 | |
| sizes | LineLength | Line is longer than 120 characters (found 153). | 90 | |
| naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 96 | |
| sizes | LineLength | Line is longer than 120 characters (found 158). | 98 | |
| sizes | LineLength | Line is longer than 120 characters (found 133). | 116 | |
| sizes | LineLength | Line is longer than 120 characters (found 135). | 130 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 139 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 161 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 190 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| imports | CustomImportOrder | 'org.opendaylight.yangtools.sal.binding.model.api.AccessModifier'should be separated from previous import group. | 12 |
| 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. | 12 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 13 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 14 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 15 |
| 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 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 13 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 14 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| imports | CustomImportOrder | 'org.opendaylight.yangtools.sal.binding.model.api.Constant'should be separated from previous import group. | 11 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 91 |
| 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 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 13 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 14 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 44 | |
| blocks | RightCurly | '}' should be on the same line. | 123 | |
| naming | LocalVariableName | Local variable name 'i' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 279 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| sizes | LineLength | Line is longer than 120 characters (found 145). | 17 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| sizes | LineLength | Line is longer than 120 characters (found 151). | 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. | 11 | |
| imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 12 | |
| naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 25 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 25 | |
| naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 36 | |
| naming | MemberName | Member name 'SUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 36 | |
| indentation | Indentation | 'GeneratedTransferObject' have incorrect indentation level 4, expected level should be 8. | 181 | |
| naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 192 | |
| naming | MemberName | Member name 'SUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 192 |
| 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 | 'org.opendaylight.yangtools.sal.binding.model.api.MethodSignature.Parameter'should be separated from previous import group. | 11 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| imports | CustomImportOrder | 'org.opendaylight.yangtools.sal.binding.model.api.AnnotationType'should be separated from previous import group. | 13 | |
| sizes | LineLength | Line is longer than 120 characters (found 132). | 19 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 | |
| imports | CustomImportOrder | 'org.opendaylight.yangtools.sal.binding.model.api.AccessModifier'should be separated from previous import group. | 12 |