Checkstyle Results
The following document contains the results of Checkstyle 6.16 with odl_checks.xml ruleset. 
InfoFiles
Rules
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | LeftCurly
|
1 |  Warning |
| NeedBraces | 1 | Warning | |
| coding | IllegalCatch | 69 | Warning |
| MissingSwitchDefault | 1 | Warning | |
| OverloadMethodsDeclarationOrder | 3 | Warning | |
| VariableDeclarationUsageDistance | 6 | Warning | |
| imports | CustomImportOrder
|
68 | Warning |
| RedundantImport | 1 | Warning | |
| UnusedImports | 9 | Warning | |
| indentation | Indentation
|
47 | Warning |
| javadoc | JavadocStyle
|
13 | Warning |
| JavadocTagContinuationIndentation | 10 | Warning | |
| naming | ConstantName | 1 | Warning |
LocalVariableName
|
19 | Warning | |
MemberName
|
3 | Warning | |
ParameterName
|
8 | Warning | |
| regexp | RegexpSingleline
|
21 |  Error |
RegexpSinglelineJava
|
15 | Warning | |
| sizes | LineLength
|
75 | Warning |
| whitespace | GenericWhitespace | 27 | Warning |
| MethodParamPad | 19 | Warning | |
OperatorWrap
|
52 | Warning | |
SeparatorWrap
|
40 | Warning | |
SeparatorWrap
|
40 | Warning | |
WhitespaceAround
|
47 | Warning |
Details
org/opendaylight/netvirt/neutronvpn/InterfaceStateToTransportZoneListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 210). | 20 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 20 |
org/opendaylight/netvirt/neutronvpn/NeutronBgpvpnChangeListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
naming | ParameterName | Parameter name 'nVpnMgr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 48 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 54 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 54 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'importRouteTargets' 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). | 88 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'exportRouteTargets' declaration and its first usage is 5, 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). | 89 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 129 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 131 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 219 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 220 |
org/opendaylight/netvirt/neutronvpn/NeutronFloatingToFixedIpMappingChangeListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Import statement for 'org.opendaylight.netvirt.neutronvpn.NeutronvpnUtils.buildfloatingIpIdToPortMappingIdentifier' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 37 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 120 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 124 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 127 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 164 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 166 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 169 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 208 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 220 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 244 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 258 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 261 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'portsIdentifierBuilder' 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). | 279 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 287 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 302 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 307 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 308 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 317 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 320 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 321 |
org/opendaylight/netvirt/neutronvpn/NeutronHostConfigChangeListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 47 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 60 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' should followed by whitespace. | 74 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 74 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' should followed by whitespace. | 79 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 79 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' should followed by whitespace. | 84 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 84 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 91 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 94 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 114 |
| Warning |
coding | MissingSwitchDefault | switch without "default" clause. | 129 |
| Warning |
blocks | LeftCurly | '{' at column 5 should be on the previous line. | 165 |
org/opendaylight/netvirt/neutronvpn/NeutronNetworkChangeListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'java.util.Objects'. | 14 |
| Warning |
naming | ParameterName | Parameter name 'nVpnMgr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 53 |
| Warning |
naming | ParameterName | Parameter name 'nVpnNatMgr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 54 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '<' is preceded with whitespace. | 193 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 135). | 197 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 198 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 218 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 133). | 236 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '210'. | 248 |
org/opendaylight/netvirt/neutronvpn/NeutronPortChangeListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.rev150602.ElanInstances. | 43 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.rev150602.elan.instances.ElanInstance. | 45 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.rev150602.elan.instances.ElanInstanceKey. | 46 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.PortAddedToSubnetBuilder. | 52 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.PortRemovedFromSubnetBuilder. | 53 |
| Warning |
naming | ParameterName | Parameter name 'nVpnMgr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 78 |
| Warning |
naming | ParameterName | Parameter name 'nVpnNatMgr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 78 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 212 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 226 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 316 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 332 |
| Warning |
indentation | Indentation | 'method def modifier' have incorrect indentation level 20, expected level should be one of the following: 12, 16. | 455 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 457 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 458 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'futures' 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). | 458 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 460 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 461 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 462 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 464 |
| Warning |
indentation | Indentation | 'if' have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 466 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 28, expected level should be one of the following: 20, 24. | 467 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 28, expected level should be one of the following: 20, 24. | 469 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 28, expected level should be one of the following: 20, 24. | 470 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 28, expected level should be one of the following: 20, 24. | 472 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 473 |
| Warning |
indentation | Indentation | 'if' have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 474 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 28, expected level should be one of the following: 20, 24. | 475 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 28, expected level should be one of the following: 20, 24. | 476 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 477 |
| Warning |
indentation | Indentation | 'if' have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 478 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 28, expected level should be one of the following: 20, 24. | 480 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 28, expected level should be one of the following: 20, 24. | 481 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 28, expected level should be one of the following: 20, 24. | 483 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 28, expected level should be one of the following: 20, 24. | 484 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 485 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 486 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 24, expected level should be one of the following: 16, 20. | 487 |
| Warning |
indentation | Indentation | 'method def rcurly' have incorrect indentation level 20, expected level should be one of the following: 12, 16. | 488 |
| Warning |
indentation | Indentation | 'object def rcurly' have incorrect indentation level 16, expected level should be one of the following: 8, 12. | 489 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 540 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 556 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 585 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 618 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 619 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 623 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 624 |
org/opendaylight/netvirt/neutronvpn/NeutronPortChangeListenerTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 50 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.Futures' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 51 |
| Warning |
naming | MemberName | Member name 'nVpnMgr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 61 |
| Warning |
naming | MemberName | Member name 'nVpnNatMgr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 63 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 97 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 104 |
org/opendaylight/netvirt/neutronvpn/NeutronQosPolicyChangeListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 64 |
org/opendaylight/netvirt/neutronvpn/NeutronQosUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 286 |
org/opendaylight/netvirt/neutronvpn/NeutronRouterChangeListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
naming | ParameterName | Parameter name 'nVpnMgr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 38 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 39 |
| Warning |
naming | ParameterName | Parameter name 'nVpnNatMgr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 39 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 135 |
org/opendaylight/netvirt/neutronvpn/NeutronSecurityRuleListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 55 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 87 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 211 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 144). | 217 |
org/opendaylight/netvirt/neutronvpn/NeutronSubnetChangeListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
naming | ParameterName | Parameter name 'nVpnMgr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 36 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 118 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 133 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 145 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 168 |
org/opendaylight/netvirt/neutronvpn/NeutronSubnetGwMacResolver.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.ThreadFactoryBuilder' import. Should be before 'org.slf4j.LoggerFactory'. | 41 |
| Warning |
coding | IllegalCatch | Catching 'Throwable' is not allowed. | 76 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '89'. | 140 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 162 |
org/opendaylight/netvirt/neutronvpn/NeutronvpnManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Objects' import. Should be before 'java.util.concurrent.Callable'. | 19 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.neutron.vpn.portip.port.data.VpnPortipToPort. | 87 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 123 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.ListenableFuture' import. Should be before 'org.slf4j.LoggerFactory'. | 124 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.SettableFuture' import. Should be before 'org.slf4j.LoggerFactory'. | 125 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 138 |
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 138 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 238 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 245 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 247 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 255 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 259 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 298 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 391 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 402 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 456 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 474 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 520 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 546 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 575 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 598 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'ips' 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). | 611 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 667 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 703 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 708 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 712 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 734 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 762 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 770 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 770 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 808 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 812 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 856 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 914 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 925 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 938 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 943 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 947 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 974 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1034 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1036 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1039 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1044 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 1048 |
| Warning |
indentation | Indentation | 'operator new lparen' have incorrect indentation level 32, expected level should be 24. | 1066 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 1066 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1078 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1089 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1102 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 1196 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 1216 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 1345 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 1361 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 1367 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 132). | 1369 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 1369 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1373 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 1374 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1381 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1397 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1450 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1500 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1528 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1556 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1586 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 1587 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1626 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1640 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 1643 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 1644 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1674 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1676 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1678 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1682 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1689 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 1693 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1713 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1728 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1731 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1733 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1737 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1743 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 130). | 1745 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 1746 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 1749 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1768 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1772 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1774 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1776 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1780 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1786 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 1790 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 1791 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1822 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1824 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1826 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1830 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 1841 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 1842 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1863 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1866 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1867 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1884 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1887 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1889 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 1893 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 1922 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1955 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1955 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1962 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 1967 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1989 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 1992 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1997 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 1999 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 2004 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 2004 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 2013 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 2014 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 2015 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 2016 |
| Warning |
naming | LocalVariableName | Local variable name 'VpnList' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 2032 |
| Warning |
naming | LocalVariableName | Local variable name 'Vpn' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 2033 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 2060 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 2061 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 2063 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 2064 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 2066 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 2067 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 2072 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 2113 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 2147 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 2173 |
org/opendaylight/netvirt/neutronvpn/NeutronvpnNatManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.external.networks.NetworksKey'. | 26 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 36 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 46 |
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 46 |
| Warning |
naming | LocalVariableName | Local variable name 'ext_net_changed' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 65 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 69 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 76 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 84 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 92 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 98 |
| Warning |
naming | LocalVariableName | Local variable name 'orig_ext_net' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 105 |
| Warning |
naming | LocalVariableName | Local variable name 'new_ext_net' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 106 |
| Warning |
naming | LocalVariableName | Local variable name 'orig_ext_gw' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 132 |
| Warning |
naming | LocalVariableName | Local variable name 'new_ext_gw' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 133 |
| Warning |
naming | LocalVariableName | Local variable name 'orig_ext_gw' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 153 |
| Warning |
naming | LocalVariableName | Local variable name 'new_ext_gw' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 154 |
| Warning |
naming | LocalVariableName | Local variable name 'orig_ext_fixed_ips' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 177 |
| Warning |
naming | LocalVariableName | Local variable name 'orig_fixed_ip_set' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 178 |
| Warning |
naming | LocalVariableName | Local variable name 'fixed_ip' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 179 |
| Warning |
naming | LocalVariableName | Local variable name 'new_ext_fixed_ips' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 182 |
| Warning |
naming | LocalVariableName | Local variable name 'upd_fixed_ip_set' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 183 |
| Warning |
naming | LocalVariableName | Local variable name 'fixed_ip' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 184 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 191 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 205 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 214 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 219 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 234 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 235 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 244 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 253 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 260 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 263 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 264 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 277 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 284 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 294 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 313 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 314 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 326 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 333 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 346 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 347 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 356 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 367 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 378 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 379 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 388 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 410 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 411 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 430 |
| Warning |
naming | LocalVariableName | Local variable name 'ext_fixed_ips' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 447 |
| Warning |
naming | LocalVariableName | Local variable name 'fixed_ips' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 448 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 468 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 469 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 491 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 492 |
| Warning |
naming | LocalVariableName | Local variable name 'ext_fixed_ips' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 510 |
| Warning |
naming | LocalVariableName | Local variable name 'fixed_ips' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 511 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 517 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 522 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 523 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 535 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 552 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 553 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 567 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 583 |
org/opendaylight/netvirt/neutronvpn/NeutronvpnUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.math.BigInteger' import. Should be before 'java.util.Iterator'. | 13 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'java.util.Iterator'. | 14 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Collection' import. Should be before 'java.util.Iterator'. | 15 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Collections' import. Should be before 'java.util.Iterator'. | 16 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.HashMap' import. Should be before 'java.util.Iterator'. | 17 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.HashSet' import. Should be before 'java.util.Iterator'. | 18 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.ImmutableBiMap' import. Should be before 'org.apache.commons.lang3.tuple.ImmutablePair'. | 32 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Sets' import. Should be before 'org.apache.commons.lang3.tuple.ImmutablePair'. | 33 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.md.sal.binding.api.WriteTransaction' import. Should be before 'org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType'. | 38 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.neutronvpn.api.utils.NeutronConstants' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 46 |
| Warning |
imports | RedundantImport | Duplicate import to line 40 - org.opendaylight.netvirt.neutronvpn.api.utils.NeutronConstants. | 46 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.huawei.params.xml.ns.yang.l3vpn.rev140815.VpnInterfaces' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 47 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.huawei.params.xml.ns.yang.l3vpn.rev140815.vpn.interfaces.VpnInterface' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 48 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.huawei.params.xml.ns.yang.l3vpn.rev140815.vpn.interfaces.VpnInterfaceKey' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 49 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.IpAddress' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 50 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.Ipv6Address' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 51 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev140508.Interfaces' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 52 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev140508.interfaces.Interface' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 53 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev140508.interfaces.InterfaceKey' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 54 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.rev130715.Uuid' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 55 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.rev150602.SegmentTypeBase' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 56 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.rev150602.SegmentTypeFlat' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 57 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.rev150602.SegmentTypeGre' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 58 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.rev150602.SegmentTypeVlan' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 59 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.rev150602.SegmentTypeVxlan' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 60 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.l3vpn.rev130911.NeutronRouterDpns' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 61 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.l3vpn.rev130911.LearntVpnVipToPortData' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 62 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.l3vpn.rev130911.neutron.router.dpns.RouterDpnList' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 63 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.l3vpn.rev130911.neutron.router.dpns.router.dpn.list.DpnVpninterfacesList' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 64 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.l3vpn.rev130911.learnt.vpn.vip.to.port.data.LearntVpnVipToPort' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 65 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.l3vpn.rev130911.learnt.vpn.vip.to.port.data.LearntVpnVipToPortKey' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 66 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ExtRouters' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.ProviderTypes'. | 67 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.genius.idmanager.rev160406.AllocateIdInput' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 99 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.genius.idmanager.rev160406.AllocateIdInputBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 100 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.genius.idmanager.rev160406.AllocateIdOutput' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 101 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.genius.idmanager.rev160406.IdManagerService' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 102 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.genius.idmanager.rev160406.ReleaseIdInput' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 103 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.genius.idmanager.rev160406.ReleaseIdInputBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 104 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.NetworkMaps' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 105 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.Subnetmaps' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 106 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.VpnMaps' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 107 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.networkmaps.NetworkMap' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 108 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.networkmaps.NetworkMapKey' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 109 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.subnetmaps.Subnetmap' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 110 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.subnetmaps.SubnetmapKey' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 111 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.vpnmaps.VpnMap' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 112 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.neutronvpn.rev150602.vpnmaps.VpnMapKey' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.subnets.rev150712.subnets.attributes.subnets.SubnetKey'. | 113 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.rev130715.MacAddress' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.inter.vpn.link.rev160311.inter.vpn.links.InterVpnLink'. | 119 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.aclservice.rev160608.IpPrefixOrAddress' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.inter.vpn.link.rev160311.inter.vpn.links.InterVpnLink'. | 120 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.aclservice.rev160608.InterfaceAclBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.inter.vpn.link.rev160311.inter.vpn.links.InterVpnLink'. | 121 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.aclservice.rev160608.interfaces._interface.AllowedAddressPairs' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.inter.vpn.link.rev160311.inter.vpn.links.InterVpnLink'. | 122 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.aclservice.rev160608.interfaces._interface.AllowedAddressPairsBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.inter.vpn.link.rev160311.inter.vpn.links.InterVpnLink'. | 123 |
| Warning |
naming | ConstantName | Name 'logger' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 132 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 133 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 342 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 149). | 456 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 151). | 457 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 466 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 468 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 139). | 470 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 139). | 472 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 153). | 474 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '512'. | 542 |
| Warning |
indentation | Indentation | 'portAllowedAddressPair' have incorrect indentation level 16, expected level should be 17. | 547 |
| Warning |
whitespace | OperatorWrap | '?' should be on a new line. | 591 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 148). | 612 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 149). | 613 |
| Warning |
whitespace | OperatorWrap | '?' should be on a new line. | 618 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 639 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 673 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 748 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 761 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 794 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 799 |
| Warning |
regexp | RegexpSinglelineJava | Line contains console output. | 801 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 812 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 819 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 821 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 838 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 852 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 853 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 933 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 939 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 1025 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1036 |
| Warning |
indentation | Indentation | 'if' have incorrect indentation level 15, expected level should be 16. | 1051 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1051 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 19, expected level should be 20. | 1052 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 15, expected level should be 16. | 1053 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 19, expected level should be 20. | 1054 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 15, expected level should be 16. | 1055 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 19, expected level should be 20. | 1056 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 15, expected level should be 16. | 1057 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 19, expected level should be 20. | 1058 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 15, expected level should be 16. | 1059 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1084 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1084 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 1125 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 1218 |
org/opendaylight/netvirt/neutronvpn/QosInterfaceStateChangeListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 139). | 29 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 38 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 74 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 133). | 80 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 86 |
org/opendaylight/netvirt/neutronvpn/SubnetmapChangeListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.slf4j.LoggerFactory'. | 31 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 32 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 59 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 156 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'catch' is not preceded with whitespace. | 177 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '}' is not followed by whitespace. | 177 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 177 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 189 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'catch' is not preceded with whitespace. | 191 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '}' is not followed by whitespace. | 191 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 191 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 192 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 206 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 218 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 136). | 219 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 225 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 240 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 255 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 269 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 281 |
org/opendaylight/netvirt/neutronvpn/ToTransportZoneTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 86 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.Futures' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 87 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 95 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 101 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 107 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 109 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 112 |
| Warning |
naming | MemberName | Member name 'nVpnMgr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 117 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 126 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 132 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 134 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 156 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 158 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 162 |
| Warning |
indentation | Indentation | 'method call' child have incorrect indentation level 8, expected level should be 12. | 163 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 167 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 169 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 173 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 149). | 177 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 158). | 178 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 179 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 183 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 184 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 185 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 186 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 187 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 188 |
| Warning |
indentation | Indentation | 'method call' child have incorrect indentation level 8, expected level should be 12. | 189 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 159). | 192 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 192 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 192 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 196 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 149). | 199 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 158). | 200 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 201 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 153). | 205 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 207 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 208 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 209 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 210 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 211 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 212 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 213 |
| Warning |
indentation | Indentation | 'method call' child have incorrect indentation level 8, expected level should be 12. | 214 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 159). | 217 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 217 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 217 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 221 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 149). | 224 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 158). | 225 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 226 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 153). | 230 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 232 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 233 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 234 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 235 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 236 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 237 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 238 |
| Warning |
indentation | Indentation | 'method call' child have incorrect indentation level 8, expected level should be 12. | 239 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 159). | 242 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 242 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 242 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 149). | 249 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 158). | 250 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 251 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 150). | 255 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 257 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 258 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 259 |
| Warning |
indentation | Indentation | 'method call' child have incorrect indentation level 8, expected level should be 12. | 260 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 159). | 263 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 263 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 263 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 266 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 269 |
| Warning |
indentation | Indentation | 'method call' child have incorrect indentation level 8, expected level should be 12. | 270 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 277 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'subnets' 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). | 297 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 300 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 304 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 141). | 323 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 330 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 332 |
| Warning |
indentation | Indentation | 'method call' child have incorrect indentation level 8, expected level should be 12. | 333 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 162). | 338 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 342 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 384 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 386 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 394 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 394 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 395 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 395 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 400 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 400 |
org/opendaylight/netvirt/neutronvpn/TransportZoneNotificationUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 126 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'try' is not followed by whitespace. | 133 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 133 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 134 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 161 |
| Warning |
indentation | Indentation | '.' have incorrect indentation level 16, expected level should be 17. | 216 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 285 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 285 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 340 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 341 |
org/opendaylight/netvirt/neutronvpn/l2gw/AddL2GwDevicesToTransportZoneJob.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 85 |
org/opendaylight/netvirt/neutronvpn/l2gw/L2GatewayListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.controller.md.sal.binding.api.DataChangeListener. | 17 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.controller.sal.binding.api.RpcProviderRegistry. | 22 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yangtools.concepts.ListenerRegistration. | 39 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 118 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 118 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 145). | 151 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 147). | 152 |
org/opendaylight/netvirt/neutronvpn/l2gw/L2GatewayUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.genius.utils.hwvtep.HwvtepSouthboundConstants' import. Should be before 'org.opendaylight.netvirt.neutronvpn.api.l2gw.L2GatewayDevice'. | 15 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.genius.utils.hwvtep.HwvtepSouthboundUtils' import. Should be before 'org.opendaylight.netvirt.neutronvpn.api.l2gw.L2GatewayDevice'. | 16 |