The following document contains the results of Checkstyle 6.2 with odl_checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
blocks | LeftCurly
|
3 | ![]() |
NeedBraces | 17 | ![]() | |
RightCurly
|
2 | ![]() | |
RightCurly | 2 | ![]() | |
coding | VariableDeclarationUsageDistance | 76 | ![]() |
imports | AvoidStarImport | 1 | ![]() |
CustomImportOrder
|
97 | ![]() | |
indentation | Indentation
|
36 | ![]() |
javadoc | JavadocMethod
|
16 | ![]() |
JavadocParagraph | 2 | ![]() | |
JavadocTagContinuationIndentation | 1 | ![]() | |
modifier | ModifierOrder | 5 | ![]() |
naming | AbbreviationAsWordInName
|
22 | ![]() |
LocalVariableName
|
3 | ![]() | |
MemberName
|
1 | ![]() | |
ParameterName
|
7 | ![]() | |
sizes | LineLength
|
64 | ![]() |
whitespace | EmptyLineSeparator
|
27 | ![]() |
MethodParamPad | 11 | ![]() | |
OperatorWrap
|
6 | ![]() | |
WhitespaceAround
|
79 | ![]() |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Constants' import. | 46 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 59 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 60 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 61 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 62 |
![]() |
naming | LocalVariableName | Local variable name 'ConfigurationServiceTracker' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 184 |
![]() |
naming | LocalVariableName | Local variable name 'NodeCacheManagerTracker' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 200 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 242 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 16 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.Executors' import. | 12 |
![]() |
imports | AvoidStarImport | Using the '.*' form of import should be avoided - org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.providers.config.rev160109.*. | 22 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 40 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Constants' import. | 12 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.md.sal.binding.api.DataBroker' import. | 14 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.md.sal.common.api.clustering.CandidateAlreadyRegisteredException' import. | 16 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 29 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 47 |
![]() |
indentation | Indentation | 'ctor def' child have incorrect indentation level 12, expected level should be 8. | 50 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 66 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Constants' import. | 17 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 40 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 44 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 45 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 60 |
![]() |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 71 |
![]() |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 84 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 84 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 89 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 97 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 114 |
![]() |
indentation | Indentation | 'createNodePath' have incorrect indentation level 4, expected level should be 8. | 131 |
![]() |
javadoc | SummaryJavadoc | Forbidden summary fragment. | 137 |
![]() |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 140 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 191 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 199 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 199 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 207 |
![]() |
indentation | Indentation | 'getOpenFlowNode' have incorrect indentation level 4, expected level should be 8. | 208 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 217 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 217 |
![]() |
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). | 243 |
![]() |
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). | 244 |
![]() |
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). | 251 |
![]() |
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). | 254 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.translator.Neutron_IPs' import. | 27 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.MdsalHelper' import. | 28 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 98 |
![]() |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 162 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 166 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 171 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 171 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 176 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 176 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 176 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 182 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 182 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 182 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 182 |
![]() |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 185 |
![]() |
indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 205 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 294 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 128). | 302 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 335 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 155). | 335 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 359 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 155). | 359 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 126). | 370 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 373 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 154). | 373 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 127). | 384 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 388 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 121). | 435 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 145). | 565 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 145). | 635 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 145). | 669 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 708 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 716 |
![]() |
blocks | LeftCurly | '{' should be on the previous line. | 723 |
![]() |
whitespace | MethodParamPad | '(' should be on the previous line. | 725 |
![]() |
blocks | LeftCurly | '{' should be on the previous line. | 735 |
![]() |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 822 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 126). | 836 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 135). | 846 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 846 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 847 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 848 |
![]() |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 874 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 902 |
![]() |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 926 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 949 |
![]() |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 963 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 987 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 1060 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 140). | 1061 |
![]() |
blocks | LeftCurly | '{' should be on the previous line. | 1067 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 1077 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 1102 |
![]() |
blocks | RightCurly | '}' should be on the same line. | 1104 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'else' is not followed by whitespace. | 1105 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 1105 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 1119 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 1154 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 124). | 1174 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 126). | 1177 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 134). | 1272 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 125). | 1273 |
![]() |
whitespace | OperatorWrap | '&&' should be on a new line. | 1324 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 1342 |
![]() |
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). | 1353 |
![]() |
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). | 1362 |
![]() |
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). | 1365 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 1456 |
![]() |
naming | ParameterName | Parameter name 'OFPortOut' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 1456 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 1486 |
![]() |
naming | ParameterName | Parameter name 'OFPortOut' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 1486 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 125). | 1577 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 143). | 1611 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 121). | 1612 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 1620 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 1620 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 147). | 1631 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 125). | 1632 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 125). | 1634 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 1640 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 1640 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 147). | 1649 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 125). | 1650 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 1658 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 1658 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 1665 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 121). | 1668 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 1682 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 1682 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 1688 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 161). | 1703 |
![]() |
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). | 1779 |
![]() |
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). | 1780 |
![]() |
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). | 1799 |
![]() |
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). | 1800 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 1845 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 157). | 1859 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '<' is not preceded with whitespace. | 1918 |
![]() |
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). | 1941 |
![]() |
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). | 1942 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 1981 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | 'org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.Node'should be separated from previous import group. | 13 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 25 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 26 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 27 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 28 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 29 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 30 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 31 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 32 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.Executors' import. | 16 |
![]() |
imports | CustomImportOrder | 'com.google.common.util.concurrent.ThreadFactoryBuilder'should be separated from previous import group. | 19 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.NodeCacheListener' import. | 23 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. | 33 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 39 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 48 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 85 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 94 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 94 |
![]() |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 106 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 120 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 137 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Constants' import. | 22 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. | 43 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 121). | 77 |
![]() |
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). | 92 |
![]() |
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). | 93 |
![]() |
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). | 94 |
![]() |
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). | 95 |
![]() |
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). | 97 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 140). | 97 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.providers.ConfigInterface' import. | 17 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 38 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 39 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. | 42 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 47 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 48 |
![]() |
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). | 82 |
![]() |
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). | 85 |
![]() |
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 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 136). | 127 |
![]() |
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). | 147 |
![]() |
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). | 150 |
![]() |
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). | 197 |
![]() |
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). | 200 |
![]() |
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). | 240 |
![]() |
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). | 241 |
![]() |
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). | 244 |
![]() |
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). | 246 |
![]() |
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). | 302 |
![]() |
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). | 334 |
![]() |
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). | 345 |
![]() |
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). | 348 |
![]() |
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). | 377 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Constants' import. | 14 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.translator.Neutron_IPs' import. | 22 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 44 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 45 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 46 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 47 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 48 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 49 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.providers.ConfigInterface' import. | 17 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 35 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 36 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 37 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 45 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 46 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 47 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 48 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 134). | 67 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 69 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 130). | 74 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 146). | 88 |
![]() |
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). | 118 |
![]() |
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). | 119 |
![]() |
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). | 120 |
![]() |
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). | 121 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 140). | 123 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'java.net.Inet6Address' import. | 13 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Action' import. | 20 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. | 38 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 39 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 40 |
![]() |
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). | 75 |
![]() |
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). | 76 |
![]() |
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). | 139 |
![]() |
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). | 140 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Constants' import. | 15 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.translator.Neutron_IPs' import. | 22 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 43 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 44 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 45 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 46 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 47 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 48 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.providers.ConfigInterface' import. | 19 |
![]() |
whitespace | EmptyLineSeparator | 'CTOR_DEF' should be separated from previous statement. | 55 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 132). | 113 |
![]() |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 113 |
![]() |
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 |
![]() |
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 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 125). | 230 |
![]() |
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 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 492 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 492 |
![]() |
naming | ParameterName | Parameter name 'OFPortOut' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 492 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 577 |
![]() |
naming | ParameterName | Parameter name 'OFPortOut' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 577 |
![]() |
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). | 581 |
![]() |
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). | 582 |
![]() |
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). | 659 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 788 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 160). | 800 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.providers.ConfigInterface' import. | 14 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Action' import. | 17 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. | 37 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 124). | 58 |
![]() |
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). | 76 |
![]() |
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). | 77 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Constants' import. | 16 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.NodeId' import. | 44 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 48 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 49 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 50 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 52 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 53 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. | 59 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 124). | 141 |
![]() |
blocks | RightCurly | '}' should be on the same line. | 152 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 163 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 170 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 121). | 173 |
![]() |
whitespace | OperatorWrap | '||' should be on a new line. | 179 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 128). | 183 |
![]() |
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). | 194 |
![]() |
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). | 197 |
![]() |
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). | 199 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '-' is not preceded with whitespace. | 211 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '-' is not followed by whitespace. | 211 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 254 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 131). | 254 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 255 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 255 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 260 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 268 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 268 |
![]() |
whitespace | OperatorWrap | '||' should be on a new line. | 273 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 128). | 277 |
![]() |
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). | 289 |
![]() |
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). | 292 |
![]() |
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). | 294 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 125). | 343 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 344 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 344 |
![]() |
whitespace | OperatorWrap | '+' should be on a new line. | 356 |
![]() |
whitespace | OperatorWrap | '||' should be on a new line. | 363 |
![]() |
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). | 379 |
![]() |
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). | 382 |
![]() |
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). | 384 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 435 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'java.net.Inet6Address' import. | 13 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Action' import. | 21 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. | 43 |
![]() |
naming | ParameterName | Parameter name 'OutPort' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 68 |
![]() |
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). | 84 |
![]() |
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). | 85 |
![]() |
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). | 86 |
![]() |
naming | LocalVariableName | Local variable name 'instructions_tmp' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 86 |
![]() |
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). | 87 |
![]() |
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). | 89 |
![]() |
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). | 91 |
![]() |
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). | 176 |
![]() |
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). | 177 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Action' import. | 20 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.Status' import. | 24 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. | 42 |
![]() |
whitespace | EmptyLineSeparator | 'CTOR_DEF' should be separated from previous statement. | 52 |
![]() |
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). | 94 |
![]() |
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). | 95 |
![]() |
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). | 96 |
![]() |
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). | 97 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 103 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 140). | 106 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 139). | 114 |
![]() |
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). | 177 |
![]() |
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). | 178 |
![]() |
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). | 179 |
![]() |
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). | 180 |
![]() |
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). | 182 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'THIRD_PARTY_PACKAGE' group. | 12 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 45 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 131). | 45 |
![]() |
naming | MemberName | Member name 'ARP_FIELD_COORDINATES' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 45 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 62 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 72 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 77 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 82 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 87 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 128 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 136 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 38 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 48 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 61 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 27 |
![]() |
modifier | ModifierOrder | '@Nullable' annotation modifier does not preceed non-annotation modifiers. | 27 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 38 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 42 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 58 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 61 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 64 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 67 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 70 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 73 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 73 |
![]() |
whitespace | OperatorWrap | '&&' should be on a new line. | 75 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 91 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 94 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 97 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 100 |
![]() |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 108 |
![]() |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 116 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 163 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 165 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 167 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 171 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 173 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 176 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 178 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 181 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 183 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 186 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 188 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 192 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 194 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 196 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 198 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.liblldp.Ethernet' import. | 12 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.liblldp.Ethernet' import. | 16 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.Futures' import. | 37 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 139). | 87 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 135). | 90 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 124). | 117 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.liblldp.Ethernet' import. | 17 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.net.InetAddresses' import. | 23 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 31 |
![]() |
javadoc | SummaryJavadoc | First sentence should be present. | 50 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 75 |
![]() |
modifier | ModifierOrder | '@Nullable' annotation modifier does not preceed non-annotation modifiers. | 75 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 87 |
![]() |
modifier | ModifierOrder | '@Nullable' annotation modifier does not preceed non-annotation modifiers. | 87 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.netvirt.openstack.netvirt.api.ConfigurationService' import. | 23 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 72 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 73 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 74 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 75 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 76 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 77 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 78 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 79 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 80 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 81 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 82 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 83 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 84 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 85 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STATIC' group. | 87 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 134 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 138 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 142 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 143 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 143 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 147 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 160 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 138). | 161 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 161 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 161 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 137). | 171 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 172 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 172 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 127). | 173 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 133). | 178 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 180). | 179 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 184). | 181 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 209 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 219 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 219 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 220 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 220 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'else' is not preceded with whitespace. | 229 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '}' is not followed by whitespace. | 229 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'else' is not followed by whitespace. | 229 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 229 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 254 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 314 |
![]() |
indentation | Indentation | 'method def modifier' have incorrect indentation level 12, expected level should be one of the following: 16, 20. | 333 |
![]() |
indentation | Indentation | 'if' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 335 |
![]() |
indentation | Indentation | 'if' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 336 |
![]() |
indentation | Indentation | 'method call' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 336 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 146). | 336 |
![]() |
indentation | Indentation | 'if' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 337 |
![]() |
indentation | Indentation | 'if rcurly' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 338 |
![]() |
indentation | Indentation | 'method call' child have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 339 |
![]() |
indentation | Indentation | 'method def' child have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 339 |
![]() |
indentation | Indentation | 'member def type' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 341 |
![]() |
indentation | Indentation | 'method def' child have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 341 |
![]() |
indentation | Indentation | 'if' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 342 |
![]() |
indentation | Indentation | 'if' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 343 |
![]() |
indentation | Indentation | 'method call' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 343 |
![]() |
indentation | Indentation | 'if' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 344 |
![]() |
indentation | Indentation | 'if rcurly' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 345 |
![]() |
indentation | Indentation | 'method call' child have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 348 |
![]() |
indentation | Indentation | 'method def' child have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 348 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 130). | 348 |
![]() |
indentation | Indentation | 'member def type' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 351 |
![]() |
indentation | Indentation | 'method def' child have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 351 |
![]() |
indentation | Indentation | 'for' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 354 |
![]() |
indentation | Indentation | 'for' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 355 |
![]() |
indentation | Indentation | 'member def type' have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 355 |
![]() |
indentation | Indentation | 'for' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 357 |
![]() |
indentation | Indentation | 'member def type' have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 357 |
![]() |
indentation | Indentation | 'for' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 359 |
![]() |
indentation | Indentation | 'method call' child have incorrect indentation level 20, expected level should be one of the following: 24, 28. | 359 |
![]() |
indentation | Indentation | 'for rcurly' have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 360 |
![]() |
indentation | Indentation | 'method def rcurly' have incorrect indentation level 12, expected level should be one of the following: 16, 20. | 361 |
![]() |
indentation | Indentation | 'method def modifier' have incorrect indentation level 12, expected level should be one of the following: 16, 20. | 363 |
![]() |
naming | ParameterName | Parameter name 't' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 364 |
![]() |
indentation | Indentation | 'method call' child have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 365 |
![]() |
indentation | Indentation | 'method def' child have incorrect indentation level 16, expected level should be one of the following: 20, 24. | 365 |
![]() |
indentation | Indentation | 'method def rcurly' have incorrect indentation level 12, expected level should be one of the following: 16, 20. | 366 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 371 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 377 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 377 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 382 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 382 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 383 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 383 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 125). | 395 |
![]() |
naming | ParameterName | Parameter name 't' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 440 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 449 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 449 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 465 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 465 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 466 |