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 |