The following document contains the results of Checkstyle 6.16 with odl_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | IllegalCatch | 8 | |
| MissingSwitchDefault | 1 | ||
| VariableDeclarationUsageDistance | 1 | ||
| imports | CustomImportOrder
|
20 | |
| naming | MemberName
|
5 | |
| regexp | RegexpSinglelineJava
|
37 | |
| sizes | LineLength
|
24 | |
| whitespace | EmptyLineSeparator
|
1 | |
| MethodParamPad | 3 | ||
OperatorWrap
|
7 | ||
SeparatorWrap
|
1 | ||
SeparatorWrap
|
1 | ||
WhitespaceAround
|
37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.slf4j.LoggerFactory'. | 32 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Arrays' import. Should be before 'org.slf4j.LoggerFactory'. | 33 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 34 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.ExecutionException' import. Should be before 'org.slf4j.LoggerFactory'. | 35 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.Future' import. Should be before 'org.slf4j.LoggerFactory'. | 36 | |
| naming | MemberName | Member name 'Logger' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 41 | |
| coding | MissingSwitchDefault | switch without "default" clause. | 282 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 27 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.slf4j.LoggerFactory'. | 28 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Arrays' import. Should be before 'org.slf4j.LoggerFactory'. | 30 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 31 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.ExecutionException' import. Should be before 'org.slf4j.LoggerFactory'. | 32 | |
| naming | MemberName | Member name 'Logger' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 37 | |
| sizes | LineLength | Line is longer than 120 characters (found 127). | 39 | |
| sizes | LineLength | Line is longer than 120 characters (found 127). | 42 | |
| coding | VariableDeclarationUsageDistance | Distance between variable 'dcBuilder' declaration and its first usage is 4, but allowed 3. Consider to make that variable as final if you still need to store its value in advance (before method calls that might do side effect on original value). | 62 | |
| whitespace | OperatorWrap | '&&' should be on a new line. | 66 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 83 | |
| whitespace | MethodParamPad | '(' is preceded with whitespace. | 85 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 108 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 21 | |
| naming | MemberName | Member name 'Logger' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 26 | |
| whitespace | WhitespaceAround | WhitespaceAround: ':' is not preceded with whitespace. | 51 | |
| whitespace | WhitespaceAround | WhitespaceAround: ':' is not followed by whitespace. | 51 | |
| whitespace | WhitespaceAround | WhitespaceAround: ':' is not preceded with whitespace. | 52 | |
| whitespace | WhitespaceAround | WhitespaceAround: ':' is not followed by whitespace. | 52 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 53 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 66 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.l3vpn.rev130911.SubnetOpData' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.subnetmaps.SubnetmapKey'. | 21 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.l3vpn.rev130911.subnet.op.data.SubnetOpDataEntry' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.subnetmaps.SubnetmapKey'. | 22 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.l3vpn.rev130911.subnet.op.data.SubnetOpDataEntryKey' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.subnetmaps.SubnetmapKey'. | 23 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.slf4j.LoggerFactory'. | 29 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.HashMap' import. Should be before 'org.slf4j.LoggerFactory'. | 30 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 31 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.slf4j.LoggerFactory'. | 32 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 34 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 37 | |
| sizes | LineLength | Line is longer than 120 characters (found 131). | 40 | |
| naming | MemberName | Member name 'LOG' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 44 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 54 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'try' is not followed by whitespace. | 56 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 56 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 59 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 60 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 63 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 66 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 70 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'else' is not preceded with whitespace. | 74 | |
| whitespace | WhitespaceAround | WhitespaceAround: '}' is not followed by whitespace. | 74 | |
| whitespace | SeparatorWrap | '.' should be on a new line. | 75 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 79 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 80 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 81 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 81 | |
| whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 82 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 82 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 83 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 85 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'else' is not preceded with whitespace. | 86 | |
| whitespace | WhitespaceAround | WhitespaceAround: '}' is not followed by whitespace. | 86 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 91 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 92 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 93 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 93 | |
| sizes | LineLength | Line is longer than 120 characters (found 132). | 96 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 99 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 101 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'catch' is not preceded with whitespace. | 101 | |
| whitespace | WhitespaceAround | WhitespaceAround: '}' is not followed by whitespace. | 101 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 102 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 117 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 124 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 131 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'else' is not preceded with whitespace. | 132 | |
| whitespace | WhitespaceAround | WhitespaceAround: '}' is not followed by whitespace. | 132 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'else' is not followed by whitespace. | 132 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 132 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 138 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'else' is not preceded with whitespace. | 139 | |
| whitespace | WhitespaceAround | WhitespaceAround: '}' is not followed by whitespace. | 139 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'else' is not followed by whitespace. | 139 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 139 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 149 | |
| sizes | LineLength | Line is longer than 120 characters (found 128). | 149 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 150 | |
| sizes | LineLength | Line is longer than 120 characters (found 134). | 150 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.slf4j.LoggerFactory'. | 28 | |
| imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 29 | |
| sizes | LineLength | Line is longer than 120 characters (found 184). | 31 | |
| sizes | LineLength | Line is longer than 120 characters (found 126). | 34 | |
| sizes | LineLength | Line is longer than 120 characters (found 148). | 36 | |
| naming | MemberName | Member name 'LOG' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 39 | |
| whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 47 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 48 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'try' is not followed by whitespace. | 50 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 50 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 54 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 55 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 56 | |
| sizes | LineLength | Line is longer than 120 characters (found 136). | 56 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 57 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 58 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 59 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 64 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 65 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 70 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 71 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'else' is not preceded with whitespace. | 72 | |
| whitespace | WhitespaceAround | WhitespaceAround: '}' is not followed by whitespace. | 72 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 73 | |
| sizes | LineLength | Line is longer than 120 characters (found 130). | 73 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'else' is not preceded with whitespace. | 74 | |
| whitespace | WhitespaceAround | WhitespaceAround: '}' is not followed by whitespace. | 74 | |
| whitespace | WhitespaceAround | WhitespaceAround: 'else' is not followed by whitespace. | 74 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 74 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 75 | |
| whitespace | MethodParamPad | '(' should be on the previous line. | 76 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 80 | |
| sizes | LineLength | Line is longer than 120 characters (found 136). | 80 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 81 | |
| sizes | LineLength | Line is longer than 120 characters (found 135). | 81 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 81 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 83 | |
| sizes | LineLength | Line is longer than 120 characters (found 134). | 83 | |
| sizes | LineLength | Line is longer than 120 characters (found 132). | 85 | |
| whitespace | MethodParamPad | '(' should be on the previous line. | 86 | |
| sizes | LineLength | Line is longer than 120 characters (found 123). | 87 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 89 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 93 | |
| sizes | LineLength | Line is longer than 120 characters (found 136). | 93 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 94 | |
| sizes | LineLength | Line is longer than 120 characters (found 135). | 94 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 94 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 96 | |
| sizes | LineLength | Line is longer than 120 characters (found 134). | 96 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 98 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 100 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 101 | |
| sizes | LineLength | Line is longer than 120 characters (found 126). | 101 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 114 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 120 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 126 | |
| whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 132 | |
| regexp | RegexpSinglelineJava | Line contains console output. | 139 | |
| sizes | LineLength | Line is longer than 120 characters (found 125). | 147 |