The following document contains the results of Checkstyle 6.16 with odl_checks.xml ruleset. 
| File | |||
|---|---|---|---|
| org/opendaylight/netvirt/utils/mdsal/utils/MdsalUtilsTest.java | 0 | 9 | 0 |
| org/opendaylight/netvirt/utils/mdsal/utils/NotifyingDataChangeListener.java | 0 | 16 | 0 |
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | IllegalCatch | 1 | |
| imports | CustomImportOrder
|
5 | |
| javadoc | JavadocStyle
|
2 | |
| NonEmptyAtclauseDescription | 3 | ||
| naming | AbbreviationAsWordInName
|
1 | |
LocalVariableName
|
3 | ||
MemberName
|
2 | ||
| sizes | LineLength
|
5 | |
| whitespace | WhitespaceAround
|
1 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 33 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 34 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 36 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 36 | |
| sizes | LineLength | Line is longer than 120 characters (found 134). | 70 | |
| sizes | LineLength | Line is longer than 120 characters (found 150). | 72 | |
| sizes | LineLength | Line is longer than 120 characters (found 132). | 85 | |
| sizes | LineLength | Line is longer than 120 characters (found 148). | 87 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 101 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.HashSet' import. Should be before 'org.slf4j.LoggerFactory'. | 21 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 22 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Set' import. Should be before 'org.slf4j.LoggerFactory'. | 23 | |
| naming | MemberName | Member name 'RETRY_WAIT' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 37 | |
| naming | AbbreviationAsWordInName | Abbreviation in name 'MDSAL_TIMEOUT' must contain no more than '4' capital letters. | 38 | |
| naming | MemberName | Member name 'MDSAL_TIMEOUT' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 38 | |
| javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 42 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 42 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 44 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 53 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 60 | |
| javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 62 | |
| naming | LocalVariableName | Local variable name '_start' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 114 | |
| naming | LocalVariableName | Local variable name '_start' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 129 | |
| naming | LocalVariableName | Local variable name '_start' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 144 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 158 |