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
|
6 |  Warning |
| NeedBraces | 17 | Warning | |
RightCurly
|
2 | Warning | |
| RightCurly | 2 | Warning | |
| coding | IllegalCatch | 20 | Warning |
| VariableDeclarationUsageDistance | 114 | Warning | |
| imports | CustomImportOrder
|
132 | Warning |
| UnusedImports | 28 | Warning | |
| indentation | Indentation
|
70 | Warning |
| javadoc | JavadocParagraph
|
5 | Warning |
JavadocStyle
|
89 | Warning | |
| JavadocTagContinuationIndentation | 1 | Warning | |
| modifier | ModifierOrder | 3 | Warning |
| naming | AbbreviationAsWordInName
|
2 | Warning |
| ConstantName | 7 | Warning | |
LocalVariableName
|
3 | Warning | |
MemberName
|
13 | Warning | |
ParameterName
|
9 | Warning | |
| regexp | RegexpSingleline
|
3 |  Error |
| sizes | LineLength
|
365 | Warning |
| whitespace | EmptyLineSeparator
|
33 | Warning |
| GenericWhitespace | 4 | Warning | |
| MethodParamPad | 29 | Warning | |
OperatorWrap
|
109 | Warning | |
| ParenPad | 14 | Warning | |
SeparatorWrap
|
1 | Warning | |
SeparatorWrap
|
1 | Warning | |
WhitespaceAround
|
192 | Warning |
Details
org/opendaylight/netvirt/openstack/netvirt/providers/ConfigActivator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 139). | 170 |
| Warning |
naming | LocalVariableName | Local variable name 'ConfigurationServiceTracker' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 232 |
| Warning |
naming | LocalVariableName | Local variable name 'NodeCacheManagerTracker' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 248 |
org/opendaylight/netvirt/openstack/netvirt/providers/ConfigInterface.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 16 |
org/opendaylight/netvirt/openstack/netvirt/providers/NetvirtProvidersConfigImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.ThreadFactoryBuilder' import. Should be before 'java.util.concurrent.ThreadFactory'. | 15 |
| Warning |
naming | ConstantName | Name 'threadFactory' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 36 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 106 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 109 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 112 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 115 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 118 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 121 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 133). | 124 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 133). | 127 |
org/opendaylight/netvirt/openstack/netvirt/providers/NetvirtProvidersProvider.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 31 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 183 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 186 |
org/opendaylight/netvirt/openstack/netvirt/providers/NetvirtProvidersProviderTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 16 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 16 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 21 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 21 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 31 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 31 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 44 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 44 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/AbstractServiceInstance.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 57 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 68 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 81 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 81 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 86 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 86 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 94 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 94 |
| Warning |
indentation | Indentation | 'createNodePath' have incorrect indentation level 4, expected level should be 8. | 128 |
| Warning |
javadoc | SummaryJavadoc | Forbidden summary fragment. | 134 |
| Warning |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 137 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 165 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 183 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 198 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 198 |
| Warning |
indentation | Indentation | 'getOpenFlowNode' have incorrect indentation level 4, expected level should be 8. | 207 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 216 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 216 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'matchBuilder' declaration and its first usage is 8, 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). | 242 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'flowBuilder' declaration and its first usage is 6, 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). | 243 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeBuilder' declaration and its first usage is 15, 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). | 250 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' 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). | 253 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/AbstractServiceInstanceTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Constants' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.NetvirtProvidersProvider'. | 36 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Southbound' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.NetvirtProvidersProvider'. | 37 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.powermock.modules.junit4.PowerMockRunner'. | 52 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.powermock.modules.junit4.PowerMockRunner'. | 53 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 55 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 55 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 139). | 62 |
| Warning |
naming | MemberName | Member name 'ID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 70 |
| Warning |
naming | MemberName | Member name 'NODE_ID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 71 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 73 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 73 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 145). | 79 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 82 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 82 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 96 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 96 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 110 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 113 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 113 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 123 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 138). | 124 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 127 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 127 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 168). | 133 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 205). | 134 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 168). | 139 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 205). | 140 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 143 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 143 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 160 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 148). | 164 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 150). | 165 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 169 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 169 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 186 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 193 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 193 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 209 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 217 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 217 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/OF13Provider.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 105 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 165 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 169 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 174 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 174 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 179 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 179 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 179 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 185 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 185 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 185 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 185 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 188 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 282 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 146). | 285 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 293 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 312 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 335 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 155). | 338 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 130). | 367 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 155). | 371 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 382 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 154). | 385 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 396 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 430 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 433 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 145). | 563 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 145). | 633 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 145). | 667 |
| Warning |
naming | AbbreviationAsWordInName | Abbreviation in name 'getIntegrationBridgeOFDPID' must contain no more than '4' capital letters. | 706 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 714 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 714 |
| Warning |
blocks | LeftCurly | '{' at column 5 should be on the previous line. | 721 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 723 |
| Warning |
blocks | LeftCurly | '{' at column 5 should be on the previous line. | 733 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 776 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 813 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 815 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 820 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 834 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 135). | 844 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 844 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 845 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 867 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 872 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 919 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 924 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 961 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 1031 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1040 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 1065 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 140). | 1066 |
| Warning |
blocks | LeftCurly | '{' at column 13 should be on the previous line. | 1072 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 1082 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1108 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 1117 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 1152 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 1172 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 1175 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 136). | 1231 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 1249 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 1276 |
| Warning |
whitespace | OperatorWrap | '&&' should be on a new line. | 1342 |
| Warning |
whitespace | OperatorWrap | '||' should be on a new line. | 1350 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeBuilder' declaration and its first usage is 9, 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). | 1387 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' 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). | 1396 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 1399 |
| Warning |
naming | ParameterName | Parameter name 'OFPortOut' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 1490 |
| Warning |
naming | ParameterName | Parameter name 'OFPortOut' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 1520 |
| Warning |
naming | ParameterName | Parameter name 'OFPortOut' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 1534 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 1613 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 143). | 1635 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 1636 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 1644 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 1644 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 147). | 1655 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 1656 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 1658 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 1664 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 1664 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 147). | 1673 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 1674 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 1682 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 1682 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 1689 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 1692 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 1706 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 1706 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1712 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1712 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 161). | 1729 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'bucketsBuilder' 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). | 1805 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'bucketList' 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). | 1806 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'bucketBuilder' declaration and its first usage is 8, 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). | 1825 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'bucketList' declaration and its first usage is 7, 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). | 1826 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1871 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1871 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 157). | 1887 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '<' is not preceded with whitespace. | 1946 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'bucketsBuilder' 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). | 1969 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'bucketList' 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). | 1970 |
| Warning |
whitespace | OperatorWrap | '||' should be on a new line. | 2004 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/OF13ProviderTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.MdsalHelper' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.NeutronNetwork'. | 44 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.BridgeConfigurationManager' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.NeutronNetwork'. | 45 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.ConfigurationService' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.NeutronNetwork'. | 46 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.EgressAclProvider' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.NeutronNetwork'. | 47 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.IngressAclProvider' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.NeutronNetwork'. | 48 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.NetworkingProvider' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.NeutronNetwork'. | 49 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.NetworkingProviderManager' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.NeutronNetwork'. | 50 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.SecurityServicesManager' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.NeutronNetwork'. | 51 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Southbound' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.NeutronNetwork'. | 52 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Status' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.NeutronNetwork'. | 53 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.StatusCode' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.NeutronNetwork'. | 54 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 68 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 68 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 87 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 113 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 190). | 116 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 195). | 121 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 195). | 125 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 189). | 126 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 148). | 136 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 149). | 145 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 151 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 176). | 156 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 145). | 159 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 173). | 161 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 144). | 163 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 176). | 165 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 141). | 175 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 144). | 176 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 186 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 187 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 196). | 191 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 194 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 193). | 198 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 201 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 188). | 206 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 135). | 207 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 177). | 208 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 163). | 209 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 177). | 210 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 189). | 211 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 213 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 173). | 215 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 216 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 163). | 217 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 150). | 218 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 163). | 219 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 175). | 220 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 188). | 225 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 135). | 226 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 177). | 227 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 163). | 228 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 177). | 229 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 189). | 230 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 148). | 233 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 173). | 235 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 236 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 163). | 237 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 150). | 238 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 163). | 239 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 175). | 240 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 171). | 245 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 177). | 246 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 189). | 247 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 167). | 250 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 157). | 252 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 163). | 253 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 175). | 254 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 186). | 259 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 167). | 261 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 171). | 263 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 171). | 268 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 177). | 269 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 189). | 270 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 272 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 157). | 274 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 163). | 275 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 175). | 276 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 195). | 281 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 135). | 282 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 181). | 283 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 140). | 285 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 180). | 287 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 288 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 167). | 289 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 195). | 294 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 135). | 295 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 181). | 296 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 140). | 298 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 180). | 300 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 301 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 167). | 302 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 169). | 308 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 179). | 309 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 159). | 311 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 155). | 313 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 165). | 314 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 171). | 319 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 147). | 321 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 157). | 323 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 179). | 328 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 158). | 330 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 165). | 332 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 169). | 337 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 171). | 338 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 136). | 340 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 155). | 342 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 157). | 343 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 347 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 132). | 349 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 176). | 364 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 169). | 365 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 185). | 371 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 176). | 372 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 176). | 373 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 155). | 376 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 152). | 377 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 211). | 378 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 185). | 380 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 161). | 381 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 215). | 382 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'neutronNetwork' declaration and its first usage is 21, 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). | 390 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeCacheManager' 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). | 391 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 420 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 158). | 423 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 165). | 425 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 164). | 426 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 429 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 165). | 431 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 190). | 435 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 183). | 436 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 165). | 439 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 196). | 440 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 190). | 441 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 240). | 449 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 153). | 451 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 260). | 452 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 455 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 458 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 463 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 468 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 477 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 512 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 513 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 133). | 514 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 519 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 520 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 526 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 529 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/PipelineOrchestrator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 25 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 26 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 27 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 28 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 29 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 30 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 31 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 32 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/PipelineOrchestratorImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | UnusedImports | Unused import - java.util.Collections. | 12 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.NodeCacheListener' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.NetvirtProvidersProvider'. | 25 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Southbound' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.NetvirtProvidersProvider'. | 26 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.providers.ConfigInterface' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.NetvirtProvidersProvider'. | 27 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.ThreadFactoryBuilder' import. Should be before 'org.slf4j.LoggerFactory'. | 35 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 40 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 40 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 49 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 49 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 99 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 111 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 125 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 158 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/PipelineOrchestratorImplTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 32 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 32 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 42 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 42 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 52 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 52 |
| Warning |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 63 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 63 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 63 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 85 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 85 |
| Warning |
indentation | Indentation | 'method def modifier' have incorrect indentation level 2, expected level should be 4. | 130 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 6, expected level should be 8. | 132 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 6, expected level should be 8. | 133 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 6, expected level should be 8. | 135 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 6, expected level should be 8. | 136 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 6, expected level should be 8. | 138 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 6, expected level should be 8. | 141 |
| Warning |
indentation | Indentation | 'method def rcurly' have incorrect indentation level 2, expected level should be 4. | 142 |
| Warning |
indentation | Indentation | 'method def modifier' have incorrect indentation level 2, expected level should be 4. | 144 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 6, expected level should be 8. | 145 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 6, expected level should be 8. | 146 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 6, expected level should be 8. | 147 |
| Warning |
indentation | Indentation | 'method def rcurly' have incorrect indentation level 2, expected level should be 4. | 148 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/PipelineOrchestratorTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 18 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/Service.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 15 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 17 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 18 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 19 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 20 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 21 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 22 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 23 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 24 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 25 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 26 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 27 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 28 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 29 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 30 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 31 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 32 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/AbstractAclService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'javax.annotation.Nullable' import. Should be before 'org.slf4j.LoggerFactory'. | 42 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.math.BigInteger' import. Should be before 'org.slf4j.LoggerFactory'. | 43 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.slf4j.LoggerFactory'. | 44 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 46 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 48 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 48 |
| Warning |
naming | ConstantName | Name 'TCP_SECURITY_GROUP_RULE_IPv4_ANY' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 58 |
| Warning |
naming | ConstantName | Name 'UDP_SECURITY_GROUP_RULE_IPv4_ANY' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 59 |
| Warning |
naming | ConstantName | Name 'ICMP_SECURITY_GROUP_RULE_IPv4_ANY' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 60 |
| Warning |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 119 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 151 |
| Warning |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 160 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 194 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/ArpResponderService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 75 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' declaration and its first usage is 38, 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). | 90 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' declaration and its first usage is 37, 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). | 91 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'ib' declaration and its first usage is 34, 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). | 92 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'aab' declaration and its first usage is 33, 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). | 93 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 95 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 95 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/ArpResponderServiceTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 41 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 44 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 44 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 191). | 77 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 79 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 194). | 84 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/ClassifierService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 46 |
| Warning |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 47 |
| Warning |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 48 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' declaration and its first usage is 13, 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). | 83 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' declaration and its first usage is 8, 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). | 86 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 90 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 136). | 128 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' declaration and its first usage is 9, 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). | 148 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 151 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' 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). | 198 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 201 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'ib' declaration and its first usage is 6, 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). | 241 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' declaration and its first usage is 14, 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). | 242 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' declaration and its first usage is 9, 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). | 245 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 247 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' 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). | 303 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeBuilder' declaration and its first usage is 10, 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). | 335 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' 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). | 346 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 349 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' 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). | 378 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/ClassifierServiceTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 32 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 34 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 34 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 62 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 62 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 69 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 79 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 79 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 86 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 96 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 96 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 103 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 113 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 113 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 120 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 130 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 130 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 137 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 147 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 147 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 154 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/EgressAclLearnServiceUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'java.util.List'. | 13 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.LearnConstants' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 16 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.LearnConstants.LearnFlowModsType' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 17 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.providers.NetvirtProvidersProvider' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 18 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.list.InstructionBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.list.InstructionKey'. | 30 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.FlowModAddMatchFromFieldCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 33 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.FlowModAddMatchFromValueCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 34 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.FlowModCopyFieldIntoFieldCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 35 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.FlowModCopyValueIntoFieldCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 36 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.FlowModOutputToPortCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 37 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.flow.mod.add.match.from.field._case.FlowModAddMatchFromFieldBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 38 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.flow.mod.add.match.from.value._case.FlowModAddMatchFromValueBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 39 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.flow.mod.copy.field.into.field._case.FlowModCopyFieldIntoFieldBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 40 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.flow.mod.copy.value.into.field._case.FlowModCopyValueIntoFieldBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 41 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.flow.mod.output.to.port._case.FlowModOutputToPortBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 42 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nodes.node.table.flow.instructions.instruction.instruction.apply.actions._case.apply.actions.action.action.NxActionLearnNodesNodeTableFlowApplyActionsCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 43 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.learn.grouping.NxLearnBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 44 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.learn.grouping.nx.learn.FlowMods' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 45 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.learn.grouping.nx.learn.FlowModsBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 46 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 135). | 56 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 150). | 57 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'listAction' declaration and its first usage is 9, 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). | 61 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'header' declaration and its first usage is 9, 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). | 75 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructionsBuilder' 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). | 133 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 134 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 153 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 150). | 154 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'listAction' declaration and its first usage is 9, 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). | 158 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'header' declaration and its first usage is 9, 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). | 170 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructionsBuilder' 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). | 228 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 229 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 248 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 150). | 249 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'listAction' declaration and its first usage is 8, 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). | 254 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'header' declaration and its first usage is 8, 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). | 256 |
| Warning |
indentation | Indentation | 'array initialization' child have incorrect indentation level 16, expected level should be one of the following: 12, 36, 38. | 290 |
| Warning |
indentation | Indentation | 'array initialization' child have incorrect indentation level 16, expected level should be one of the following: 12, 36, 38. | 295 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructionsBuilder' 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). | 309 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 310 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 342 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 343 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 416 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/EgressAclService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.HashMap' import. Should be before 'java.util.List'. | 16 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 65 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 68 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '<' is preceded with whitespace. | 78 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '<' is preceded with whitespace. | 78 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 107 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 285 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 287 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 289 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 291 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 293 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 295 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 17, expected level should be 16. | 404 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 17, expected level should be 16. | 486 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 569 |
| Warning |
blocks | LeftCurly | '{' at column 9 should be on the previous line. | 570 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 572 |
| Warning |
indentation | Indentation | 'else' child have incorrect indentation level 13, expected level should be 12. | 588 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 589 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 133). | 669 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 20. | 735 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 161). | 896 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 924 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/EgressAclServiceTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.SecurityGroupCacheManger' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.Neutron_IPs'. | 38 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.SecurityServicesManager' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.Neutron_IPs'. | 39 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.PipelineOrchestrator' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.Neutron_IPs'. | 40 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbTerminationPointAugmentation. | 51 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.powermock.modules.junit4.PowerMockRunner'. | 58 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 59 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 59 |
| Warning |
whitespace | EmptyLineSeparator | 'CLASS_DEF' should be separated from previous statement. | 62 |
| Warning |
naming | MemberName | Member name 'neutron_ip_src' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 85 |
| Warning |
naming | MemberName | Member name 'neutron_ip_dest_1' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 86 |
| Warning |
naming | MemberName | Member name 'neutron_ip_dest_2' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 87 |
| Warning |
naming | MemberName | Member name 'neutron_ipv6_dest_1' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 88 |
| Warning |
naming | MemberName | Member name 'neutron_ipv6_dest_2' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 89 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 167 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 171 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 218 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 138). | 220 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 235 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 138). | 237 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 252 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 268 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 294 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 294 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 296 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 296 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 297 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 321 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 321 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 323 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 323 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 324 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 351 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 351 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 352 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 379 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 379 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 380 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 404 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 404 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 405 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 405 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 406 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 408 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 411 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 438 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 438 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 439 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 439 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 440 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 442 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 445 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 473 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 473 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 474 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 474 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 475 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 475 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 477 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 477 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 480 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 508 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 508 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 509 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 509 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 510 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 510 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 512 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 512 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 515 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 538 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 538 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 560 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 560 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 586 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 610 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 634 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 636 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 639 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 666 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 668 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 671 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 699 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 701 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 704 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 732 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 734 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 737 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 767 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 794 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 821 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 848 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 876 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 878 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 881 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 912 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 914 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 917 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 948 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 950 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 953 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 984 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 986 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 989 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1016 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1040 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1064 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1088 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1112 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1114 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1117 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1144 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1146 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1149 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1176 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1178 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1181 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1208 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1210 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1213 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1244 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1272 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1300 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1328 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1362 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1399 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1435 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1471 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 1485 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 1499 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1506 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1506 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 138). | 1515 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1520 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1520 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1534 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1534 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1549 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1549 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/IcmpEchoResponderService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 49 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 66 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 68 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 130). | 73 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 146). | 87 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' declaration and its first usage is 27, 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). | 117 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' declaration and its first usage is 26, 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). | 118 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'ib' declaration and its first usage is 23, 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). | 119 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'aab' declaration and its first usage is 22, 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). | 120 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 122 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/IcmpEchoResponderServiceTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | EmptyLineSeparator | 'import' should be separated from previous statement. | 14 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.junit.Assert.assertEquals' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 14 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Matchers.any' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 15 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Matchers.anyBoolean' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 16 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Mockito.mock' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 17 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Mockito.times' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 18 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Mockito.verify' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 19 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Mockito.when' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 20 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Action' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 28 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Status' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 29 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 37 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.net.InetAddress' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 39 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 41 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 41 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 195). | 74 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 76 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 198). | 81 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/InboundNatService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' declaration and its first usage is 10, 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). | 73 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 74 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' 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). | 137 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 138 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/InboundNatServiceTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.powermock.api.support.membermodification.MemberModifier'. | 42 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 44 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 44 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 71 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 75 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 75 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 89 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 102 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 102 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/IngressAclLearnServiceUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'java.util.List'. | 13 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.LearnConstants' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 18 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.LearnConstants.LearnFlowModsType' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 19 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.providers.NetvirtProvidersProvider' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 20 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.flow.InstructionsBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.list.InstructionBuilder'. | 26 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.instruction.ApplyActionsCase' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.list.InstructionBuilder'. | 27 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.instruction.ApplyActionsCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.list.InstructionBuilder'. | 28 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.instruction.apply.actions._case.ApplyActions' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.list.InstructionBuilder'. | 29 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.instruction.apply.actions._case.ApplyActionsBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.list.InstructionBuilder'. | 30 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.list.Instruction' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.list.InstructionBuilder'. | 31 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.FlowModAddMatchFromFieldCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 35 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.FlowModAddMatchFromValueCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 36 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.FlowModCopyFieldIntoFieldCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 37 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.FlowModCopyValueIntoFieldCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 38 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.FlowModOutputToPortCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 39 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.flow.mod.add.match.from.field._case.FlowModAddMatchFromFieldBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 40 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.flow.mod.add.match.from.value._case.FlowModAddMatchFromValueBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 41 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.flow.mod.copy.field.into.field._case.FlowModCopyFieldIntoFieldBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 42 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.flow.mod.copy.value.into.field._case.FlowModCopyValueIntoFieldBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 43 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.flow.mod.spec.flow.mod.spec.flow.mod.output.to.port._case.FlowModOutputToPortBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 44 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nodes.node.table.flow.instructions.instruction.instruction.apply.actions._case.apply.actions.action.action.NxActionLearnNodesNodeTableFlowApplyActionsCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 45 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.learn.grouping.NxLearnBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 46 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.learn.grouping.nx.learn.FlowMods' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 47 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.learn.grouping.nx.learn.FlowModsBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 48 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 135). | 58 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 150). | 59 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'listAction' declaration and its first usage is 9, 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). | 63 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'header' declaration and its first usage is 9, 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). | 76 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructionsBuilder' 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). | 134 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 135 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 153 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 150). | 154 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'listAction' declaration and its first usage is 9, 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). | 158 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'header' declaration and its first usage is 9, 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). | 171 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructionsBuilder' 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). | 232 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 233 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 251 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 150). | 252 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 255 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'listAction' declaration and its first usage is 8, 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). | 257 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'header' declaration and its first usage is 8, 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). | 259 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructionsBuilder' 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). | 310 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 311 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 344 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 345 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 418 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/IngressAclService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.HashMap' import. Should be before 'java.util.List'. | 16 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.nodes.NodeBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.NodeId'. | 40 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.LearnConstants' import. Should be before 'org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.NodeId'. | 41 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '<' is preceded with whitespace. | 74 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '<' is preceded with whitespace. | 74 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 103 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 279 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 281 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 283 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 285 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 287 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 289 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 20. | 570 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 658 |
| Warning |
blocks | LeftCurly | '{' at column 9 should be on the previous line. | 659 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 661 |
| Warning |
indentation | Indentation | 'else' child have incorrect indentation level 13, expected level should be 12. | 677 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 678 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 13, expected level should be 12. | 698 |
| Warning |
indentation | Indentation | 'if' have incorrect indentation level 13, expected level should be 12. | 699 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 17, expected level should be 16. | 700 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 13, expected level should be 12. | 702 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 133). | 758 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 811 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 839 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/IngressAclServiceTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | UnusedImports | Unused import - org.mockito.invocation.InvocationOnMock. | 28 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.SecurityGroupCacheManger' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.translator.Neutron_IPs'. | 41 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.powermock.modules.junit4.PowerMockRunner'. | 58 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 60 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 60 |
| Warning |
naming | MemberName | Member name 'neutron_ip_src' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 87 |
| Warning |
naming | MemberName | Member name 'neutron_ip_dest_1' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 88 |
| Warning |
naming | MemberName | Member name 'neutron_ip_dest_2' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 89 |
| Warning |
naming | MemberName | Member name 'neutron_ipv6_dest_1' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 90 |
| Warning |
naming | MemberName | Member name 'neutron_ipv6_dest_2' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 91 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 121 |
| Warning |
whitespace | MethodParamPad | '(' should be on the previous line. | 167 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 169 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 172 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 177 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 219 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 138). | 221 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 236 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 138). | 238 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 253 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 270 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 301 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 317 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 329 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 357 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 373 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 385 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 413 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 413 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 415 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 415 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 437 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 449 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 449 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 451 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 451 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 486 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 486 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 488 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 488 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 512 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 524 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 524 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 526 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 526 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 557 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 573 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 582 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 607 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 623 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 632 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 656 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 658 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 679 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 688 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 690 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 722 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 722 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 724 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 724 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 746 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 756 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 756 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 758 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 758 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 792 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 808 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 820 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 848 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 864 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 876 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 904 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 904 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 906 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 906 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 928 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 940 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 940 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 942 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 942 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 976 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 976 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 978 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 978 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 1000 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1012 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1012 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1014 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1014 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1045 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 1061 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1070 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1095 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 1111 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1120 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1145 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1145 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1147 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1147 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 1169 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1178 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1178 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1180 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1180 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1211 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1211 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1213 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1213 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 1235 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1244 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1244 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1246 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 1246 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 1384 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1392 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1394 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1397 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 1419 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1428 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1430 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1433 |
| Warning |
indentation | Indentation | 'method call' child have incorrect indentation level 8, expected level should be 12. | 1450 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1463 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1465 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1468 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1498 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 1500 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 1503 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 1517 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 1531 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1538 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1538 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 138). | 1547 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1551 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1551 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 1554 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1564 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1564 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 1567 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 138). | 1573 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1577 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1577 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 1580 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/L2ForwardingLearnService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 55 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 171). | 70 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeBuilder' declaration and its first usage is 18, 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). | 74 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'listAction' declaration and its first usage is 6, 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). | 78 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 81 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'header' 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). | 93 |
| Warning |
indentation | Indentation | 'array initialization' child have incorrect indentation level 16, expected level should be 12. | 94 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructionsBuilder' declaration and its first usage is 9, 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). | 117 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 119 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 159 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 160 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/L2ForwardingService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.providers.ConfigInterface' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 19 |
| Warning |
whitespace | EmptyLineSeparator | 'CTOR_DEF' should be separated from previous statement. | 55 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 132). | 113 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 113 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeBuilder' declaration and its first usage is 6, 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). | 166 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'flowBuilder' 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). | 167 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 230 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeBuilder' 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). | 239 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 451 |
| Warning |
naming | ParameterName | Parameter name 'OFPortOut' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 451 |
| Warning |
naming | ParameterName | Parameter name 'OFPortOut' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 536 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeBuilder' declaration and its first usage is 6, 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). | 540 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'flowBuilder' 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). | 541 |
| Warning |
naming | ParameterName | Parameter name 'OFPortOut' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 613 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeBuilder' declaration and its first usage is 6, 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). | 617 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'flowBuilder' 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). | 618 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 681 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 726 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 726 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 160). | 739 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/L2FowardingServiceTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.powermock.api.support.membermodification.MemberModifier'. | 39 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.powermock.api.support.membermodification.MemberModifier'. | 40 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 42 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 42 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 73 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 83 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 87 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 87 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 93 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 103 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 103 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 109 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 119 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 119 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 125 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 135 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 135 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 141 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 151 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 151 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 157 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 167 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 167 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 173 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 183 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 183 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 189 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 199 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 199 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 205 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 215 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 215 |
| Warning |
indentation | Indentation | 'method def modifier' have incorrect indentation level 3, expected level should be 4. | 218 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 7, expected level should be 8. | 220 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 7, expected level should be 8. | 221 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 141). | 221 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 7, expected level should be 8. | 222 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 7, expected level should be 8. | 223 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 7, expected level should be 8. | 225 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 7, expected level should be 8. | 226 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 7, expected level should be 8. | 227 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 7, expected level should be 8. | 228 |
| Warning |
indentation | Indentation | 'method def rcurly' have incorrect indentation level 3, expected level should be 4. | 229 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/L2RewriteService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.providers.ConfigInterface' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 14 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/L2RewriteServiceTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 13 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 13 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/L3ForwardingService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 56 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' declaration and its first usage is 9, 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). | 74 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 75 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/L3FowardingServiceTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Action' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 34 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Status' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 35 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.providers.NetvirtProvidersProvider' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 36 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.PipelineOrchestrator' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.Service'. | 37 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.powermock.api.support.membermodification.MemberModifier'. | 42 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 44 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 44 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 72 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 76 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 76 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/LoadBalancerService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 68 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 86 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 139 |
| Warning |
blocks | RightCurly | '}' at column 9 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else or try/catch/finally). | 150 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 161 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 161 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 171 |
| Warning |
whitespace | OperatorWrap | '||' should be on a new line. | 177 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 181 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' declaration and its first usage is 20, 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). | 192 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' declaration and its first usage is 19, 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). | 195 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 197 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '-' is not preceded with whitespace. | 209 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '-' is not followed by whitespace. | 209 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 252 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 253 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 253 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 266 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 266 |
| Warning |
whitespace | OperatorWrap | '||' should be on a new line. | 271 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 275 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' declaration and its first usage is 19, 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). | 287 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' declaration and its first usage is 14, 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). | 290 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 292 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 341 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 342 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 342 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 354 |
| Warning |
whitespace | OperatorWrap | '||' should be on a new line. | 361 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' declaration and its first usage is 15, 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). | 377 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' declaration and its first usage is 10, 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). | 380 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 382 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 433 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/LoadBalancerServiceTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | UnusedImports | Unused import - org.junit.Assert.assertEquals. | 11 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 44 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 47 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 47 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 84 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 95 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 172). | 96 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 98 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 198). | 100 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 211). | 102 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 103 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 201). | 107 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 108 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 113 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 113 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 182). | 118 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 193). | 120 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 183). | 122 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 123 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 186). | 127 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/OutboundNatService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
naming | ParameterName | Parameter name 'OutPort' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 66 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' declaration and its first usage is 27, 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). | 82 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' declaration and its first usage is 26, 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). | 83 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions_tmp' declaration and its first usage is 21, 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). | 84 |
| Warning |
naming | LocalVariableName | Local variable name 'instructions_tmp' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 84 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'ib' declaration and its first usage is 18, 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). | 85 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'aab' declaration and its first usage is 17, 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). | 87 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 89 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' 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). | 174 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 175 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/OutboundNatServiceTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | UnusedImports | Unused import - org.junit.Assert.assertEquals. | 11 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 40 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 42 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 42 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 72 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 72 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 144). | 73 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 84 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 97 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 97 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 211). | 102 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 103 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 214). | 107 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/ResubmitAclLearnService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | UnusedImports | Unused import - java.math.BigInteger. | 11 |
| Warning |
imports | UnusedImports | Unused import - java.net.InetAddress. | 12 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.net.Inet6Address' import. Should be before 'java.net.InetAddress'. | 13 |
| Warning |
imports | UnusedImports | Unused import - java.net.Inet6Address. | 13 |
| Warning |
imports | UnusedImports | Unused import - java.net.UnknownHostException. | 14 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'java.util.List'. | 16 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.InboundNatProvider' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.api.StatusCode'. | 19 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.netvirt.openstack.netvirt.api.InboundNatProvider. | 19 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Status' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.api.StatusCode'. | 20 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.ResubmitAclLearnProvider' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.api.StatusCode'. | 21 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.providers.ConfigInterface' import. Should be before 'org.opendaylight.netvirt.openstack.netvirt.providers.openflow13.AbstractServiceInstance'. | 23 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.netvirt.utils.mdsal.openflow.ActionUtils. | 25 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.netvirt.utils.mdsal.openflow.InstructionUtils. | 27 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.netvirt.utils.mdsal.openflow.MatchUtils. | 28 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.Ipv4Prefix. | 29 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder. | 34 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.tables.table.FlowBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder'. | 35 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.flow.InstructionsBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder'. | 36 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.flow.MatchBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder'. | 37 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.flow.MatchBuilder. | 37 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.list.Instruction' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder'. | 38 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.list.InstructionBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder'. | 39 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.list.InstructionKey' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder'. | 40 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.nodes.NodeBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder'. | 41 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.action.types.rev131112.action.list.ActionKey' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder'. | 42 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.instruction.ApplyActionsCase' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder'. | 43 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.instruction.ApplyActionsCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder'. | 44 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.instruction.apply.actions._case.ApplyActions' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder'. | 45 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.instruction.apply.actions._case.ApplyActionsBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder'. | 46 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.add.group.input.buckets.bucket.action.action.NxActionResubmitRpcAddGroupCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxRegCaseBuilder'. | 47 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 55 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeBuilder' declaration and its first usage is 18, 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). | 69 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'listAction' 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). | 73 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructionsBuilder' 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). | 93 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' 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). | 95 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/RoutingService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | EmptyLineSeparator | 'CTOR_DEF' should be separated from previous statement. | 50 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' declaration and its first usage is 23, 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). | 92 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' declaration and its first usage is 18, 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). | 93 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'ib' declaration and its first usage is 15, 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). | 94 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'aab' declaration and its first usage is 14, 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). | 95 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 101 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 140). | 104 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 139). | 112 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'isb' declaration and its first usage is 18, 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). | 175 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'instructions' declaration and its first usage is 13, 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). | 176 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'ib' declaration and its first usage is 10, 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). | 177 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'aab' declaration and its first usage is 9, 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). | 178 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 180 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/RoutingServiceTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 39 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 41 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 41 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 69 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 69 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 81 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 94 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 94 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/VlanResponderService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.HashSet' import. Should be before 'java.util.concurrent.ConcurrentHashMap'. | 12 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Iterator' import. Should be before 'java.util.concurrent.ConcurrentHashMap'. | 13 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'java.util.concurrent.ConcurrentHashMap'. | 14 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map' import. Should be before 'java.util.concurrent.ConcurrentHashMap'. | 15 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Set' import. Should be before 'java.util.concurrent.ConcurrentHashMap'. | 16 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.action.types.rev131112.action.action.OutputActionCase. | 36 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.action.types.rev131112.action.action.OutputActionCaseBuilder. | 37 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.action.types.rev131112.action.action.output.action._case.OutputActionBuilder. | 38 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.action.types.rev131112.action.list.Action. | 39 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.action.types.rev131112.action.list.ActionBuilder. | 40 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.action.types.rev131112.action.list.ActionKey. | 41 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.tables.table.Flow. | 43 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.instruction.ApplyActionsCase. | 47 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.instruction.ApplyActionsCaseBuilder. | 48 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.flow.types.rev131026.instruction.instruction.apply.actions._case.ApplyActionsBuilder. | 49 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.NodeConnectorId. | 53 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 132). | 104 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 132 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 163 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 168 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 186 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 187 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 139). | 199 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 231 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 236 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 250 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 253 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 263 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 263 |
| Warning |
whitespace | OperatorWrap | '&&' should be on a new line. | 278 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 290 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 295 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 297 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 298 |
| Warning |
indentation | Indentation | 'for' child have incorrect indentation level 21, expected level should be 20. | 310 |
| Warning |
indentation | Indentation | 'for' child have incorrect indentation level 21, expected level should be 20. | 312 |
| Warning |
indentation | Indentation | 'for' child have incorrect indentation level 21, expected level should be 20. | 313 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 313 |
| Warning |
indentation | Indentation | 'for' child have incorrect indentation level 21, expected level should be 20. | 314 |
| Warning |
indentation | Indentation | 'for' child have incorrect indentation level 21, expected level should be 20. | 316 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 142). | 316 |
| Warning |
indentation | Indentation | 'for' child have incorrect indentation level 21, expected level should be 20. | 317 |
| Warning |
indentation | Indentation | 'for' child have incorrect indentation level 21, expected level should be 20. | 318 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 327 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 335 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 342 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 141). | 343 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 345 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'flowBuilder' 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). | 350 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 366 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 375 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 382 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 389 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 130). | 393 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 400 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 401 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 402 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 136). | 409 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 439 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 441 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'flowBuilder' 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). | 455 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 472 |
| Warning |
indentation | Indentation | 'else' child have incorrect indentation level 17, expected level should be 16. | 478 |
| Warning |
indentation | Indentation | 'else rcurly' have incorrect indentation level 15, expected level should be 12. | 479 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 480 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 481 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 514 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 522 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 529 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 542 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 543 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 583 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 599 |
| Warning |
whitespace | OperatorWrap | '?' should be on a new line. | 605 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 141). | 613 |
| Warning |
blocks | LeftCurly | '{' at column 29 should be on the previous line. | 615 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 138). | 619 |
| Warning |
blocks | RightCurly | '}' at column 9 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else or try/catch/finally). | 625 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 626 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/arp/Arp.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | 'io.netty.buffer.Unpooled' should be separated from previous import group. | 12 |
| Warning |
naming | AbbreviationAsWordInName | Abbreviation in name 'ARP_FIELD_COORDINATES' must contain no more than '4' capital letters. | 45 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 45 |
| Warning |
naming | MemberName | Member name 'ARP_FIELD_COORDINATES' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 45 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 128 |
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 128 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 137 |
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 137 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/arp/ArpFlowFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 38 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 38 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 50 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 50 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 66 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 66 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/arp/ArpResolverMetadata.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 17 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 42 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 58 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 61 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 64 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 67 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 70 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 73 |
| Warning |
whitespace | OperatorWrap | '&&' should be on a new line. | 75 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 91 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 94 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 97 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 100 |
| Warning |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 108 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 116 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 163 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 165 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 167 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 171 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 173 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 176 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 178 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 181 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 183 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 186 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 188 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 192 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 194 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 196 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 198 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/arp/ArpSender.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.Futures' import. Should be before 'org.slf4j.LoggerFactory'. | 37 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.JdkFutureAdapters' import. Should be before 'org.slf4j.LoggerFactory'. | 38 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.ListenableFuture' import. Should be before 'org.slf4j.LoggerFactory'. | 39 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 141). | 87 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 135). | 91 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 118 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/arp/ArpUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.net.InetAddresses' import. Should be before 'org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.rev130715.MacAddress'. | 23 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 31 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 31 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 52 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 52 |
org/opendaylight/netvirt/openstack/netvirt/providers/openflow13/services/arp/GatewayMacResolverService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | UnusedImports | Unused import - java.util.concurrent.Callable. | 25 |
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 86 |
| Warning |
naming | ConstantName | Name 'threadFactoryArpWatcher' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 107 |
| Warning |
naming | ConstantName | Name 'threadFactoryMacRefresher' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 109 |
| Warning |
naming | ConstantName | Name 'threadFactoryRefreshReq' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 111 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 130). | 133 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 147). | 137 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 143 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 144 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 144 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 154 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 155 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 155 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 155 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 162 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 163 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 163 |
| Warning |
whitespace | ParenPad | ')' is preceded with whitespace. | 167 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 169 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 172). | 170 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 176). | 172 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 193 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 198 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 208 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 208 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 209 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 209 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'else' is not preceded with whitespace. | 212 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '}' is not followed by whitespace. | 212 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'else' is not followed by whitespace. | 212 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 212 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 237 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 297 |
| Warning |
indentation | Indentation | 'method def modifier' have incorrect indentation level 12, expected level should be one of the following: 16, 20. | 316 |
| Warning |
indentation | Indentation | 'if' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 318 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 319 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 146). | 319 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 320 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 321 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 322 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 324 |
| Warning |
indentation | Indentation | 'if' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 325 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 326 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 327 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 328 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 331 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 130). | 331 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 334 |
| Warning |
indentation | Indentation | 'for' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 337 |
| Warning |
indentation | Indentation | 'for' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 338 |
| Warning |
indentation | Indentation | 'for' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 340 |
| Warning |
indentation | Indentation | 'for' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 342 |
| Warning |
indentation | Indentation | 'for rcurly' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 343 |
| Warning |
indentation | Indentation | 'method def rcurly' have incorrect indentation level 12, expected level should be one of the following: 16, 20. | 344 |
| Warning |
indentation | Indentation | 'method def modifier' have incorrect indentation level 12, expected level should be one of the following: 16, 20. | 346 |
| Warning |
naming | ParameterName | Parameter name 't' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 347 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 348 |
| Warning |
indentation | Indentation | 'method def rcurly' have incorrect indentation level 12, expected level should be one of the following: 16, 20. | 349 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 354 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 357 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 357 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 362 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 362 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 363 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 363 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 374 |
| Warning |
naming | ParameterName | Parameter name 't' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 419 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 428 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 428 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 444 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 444 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 445 |