The following document contains the results of Checkstyle 6.2 with odl_checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
blocks | NeedBraces | 1 | ![]() |
RightCurly
|
1 | ![]() | |
RightCurly | 1 | ![]() | |
coding | OverloadMethodsDeclarationOrder | 10 | ![]() |
VariableDeclarationUsageDistance | 13 | ![]() | |
imports | AvoidStarImport | 1 | ![]() |
CustomImportOrder
|
62 | ![]() | |
indentation | Indentation
|
4 | ![]() |
javadoc | JavadocMethod
|
69 | ![]() |
JavadocParagraph | 8 | ![]() | |
naming | AbbreviationAsWordInName
|
54 | ![]() |
LocalVariableName
|
49 | ![]() | |
ParameterName
|
51 | ![]() | |
sizes | LineLength
|
90 | ![]() |
whitespace | EmptyLineSeparator
|
17 | ![]() |
MethodParamPad | 3 | ![]() | |
OperatorWrap
|
14 | ![]() | |
SeparatorWrap
|
7 | ![]() | |
SeparatorWrap
|
7 | ![]() | |
WhitespaceAround
|
105 | ![]() |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. | 53 |
![]() |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 70 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 79 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 85 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 87 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 107 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 112 |
![]() |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '79'. | 162 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 220 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 224 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 261 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 271 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 285 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.TimeoutException' import. | 22 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 66 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 86 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 117 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 162 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 168 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 181 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 181 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 192 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 232 |
![]() |
naming | LocalVariableName | Local variable name 'pSwitchNode' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 235 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 237 |
![]() |
blocks | RightCurly | '}' should be on the same line. | 244 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 245 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 245 |
![]() |
naming | ParameterName | Parameter name 'pNode' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 253 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 255 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 378 |
![]() |
indentation | Indentation | 'method def rcurly' have incorrect indentation level 8, expected level should be 4. | 386 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 388 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 405 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 405 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'switch' is not followed by whitespace. | 408 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 408 |
![]() |
whitespace | OperatorWrap | '+' should be on a new line. | 423 |
![]() |
whitespace | SeparatorWrap | ',' should be on the previous line. | 429 |
![]() |
naming | ParameterName | Parameter name 't' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 434 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 445 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 121). | 446 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 121). | 531 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 535 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 538 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 42 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 72 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 211 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 212 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 219 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 219 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'switch' is not followed by whitespace. | 227 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 232 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'switch' is not followed by whitespace. | 244 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 249 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '!=' is not preceded with whitespace. | 277 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '!=' is not followed by whitespace. | 277 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 279 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 282 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 285 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 291 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 43 |
![]() |
naming | ParameterName | Parameter name 'lSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 50 |
![]() |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 66 |
![]() |
naming | ParameterName | Parameter name 'pLocator' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 82 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 98 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'ENUM_DEF' should be separated from previous statement. | 13 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 13 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.ImmutableBiMap' import. | 17 |
![]() |
indentation | Indentation | '=' have incorrect indentation level 4, expected level should be 8. | 26 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.EncapsulationTypeBase' import. | 33 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Preconditions' import. | 54 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 62 |
![]() |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 69 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 91 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 110 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 119 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 126 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 136 |
![]() |
whitespace | OperatorWrap | '+' should be on a new line. | 137 |
![]() |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '74'. | 148 |
![]() |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 149 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 154 |
![]() |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '86'. | 162 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 162 |
![]() |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '154'. | 169 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 169 |
![]() |
naming | ParameterName | Parameter name 'lSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 170 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 184 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 194 |
![]() |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '184'. | 205 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 210 |
![]() |
naming | ParameterName | Parameter name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 210 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 212 |
![]() |
whitespace | OperatorWrap | '&&' should be on a new line. | 212 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 214 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 214 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 214 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 214 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 220 |
![]() |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '205'. | 224 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 224 |
![]() |
naming | ParameterName | Parameter name 'vBindings' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 225 |
![]() |
indentation | Indentation | 'method def' child have incorrect indentation level 16, expected level should be 8. | 226 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 36 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 55 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 57 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 99 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 99 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 103 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 107 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 111 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 155 |
![]() |
whitespace | SeparatorWrap | ',' should be on the previous line. | 159 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 182 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 27 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 55 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 65 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 76 |
![]() |
naming | ParameterName | Parameter name 'pNode' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 77 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 93 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 129 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 73 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 16 |
![]() |
imports | AvoidStarImport | Using the '.*' form of import should be avoided - java.util.concurrent.*. | 16 |
![]() |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 20 |
![]() |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 33 |
![]() |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 42 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 57 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 127). | 59 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 75 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
![]() |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 21 |
![]() |
naming | ParameterName | Parameter name 'o' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 102 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 103 |
![]() |
whitespace | OperatorWrap | '+' should be on a new line. | 122 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 122 |
![]() |
whitespace | OperatorWrap | '+' should be on a new line. | 123 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 13 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 14 |
![]() |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 18 |
![]() |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 24 |
![]() |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 31 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 43 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 44 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 44 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 46 |
![]() |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 54 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.ovsdb.hwvtepsouthbound.HwvtepConnectionManager' import. | 13 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 21 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 22 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 23 |
![]() |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 27 |
![]() |
naming | LocalVariableName | Local variable name 'nIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 49 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 72 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 47 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 56 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 125). | 68 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 129). | 69 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 126). | 73 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 129). | 82 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 98 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 107 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 116 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 125 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 125 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 145 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 165 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 172 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 181 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 128). | 188 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 198 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 218 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 238 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 258 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 129). | 278 |
![]() |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '181'. | 278 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 278 |
![]() |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '125'. | 286 |
![]() |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '145'. | 291 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 33 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 132). | 61 |
![]() |
whitespace | OperatorWrap | '&&' should be on a new line. | 63 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 90 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 104 |
![]() |
naming | LocalVariableName | Local variable name 'i' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 116 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 117 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 126). | 118 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 33 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 126). | 69 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 135). | 81 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 93 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 127 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 152 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 34 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 67 |
![]() |
whitespace | SeparatorWrap | '.' should be on a new line. | 69 |
![]() |
naming | LocalVariableName | Local variable name 'i' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 120 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 121 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 35 |
![]() |
coding | VariableDeclarationUsageDistance | Distance between variable 'operationalMacOptional' 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). | 69 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 129). | 71 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 81 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 141). | 100 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 105 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 121). | 108 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 135). | 116 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 34 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 67 |
![]() |
whitespace | SeparatorWrap | '.' should be on a new line. | 69 |
![]() |
naming | LocalVariableName | Local variable name 'i' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 120 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 121 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 35 |
![]() |
coding | VariableDeclarationUsageDistance | Distance between variable 'operationalMacOptional' 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). | 69 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 132). | 71 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 81 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 141). | 100 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 105 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 113 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 135). | 115 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 35 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 129). | 65 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 121). | 69 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 137). | 78 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 126). | 98 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 134). | 101 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 116 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 117 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 118 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 134). | 121 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 134). | 129 |
![]() |
naming | LocalVariableName | Local variable name 'i' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 138 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 139 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 40 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 83 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 133). | 96 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 124). | 103 |
![]() |
whitespace | OperatorWrap | '+' should be on a new line. | 105 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 126). | 165 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'else' is not preceded with whitespace. | 166 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '}' is not followed by whitespace. | 166 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'else' is not followed by whitespace. | 166 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 166 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 134). | 167 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 184 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 134). | 187 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 210 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 231 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 232 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 134). | 235 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 134). | 243 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 32 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 131). | 60 |
![]() |
whitespace | OperatorWrap | '&&' should be on a new line. | 61 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 63 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 130). | 68 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 148). | 72 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 89 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 47 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 125). | 85 |
![]() |
whitespace | OperatorWrap | '+' should be on a new line. | 100 |
![]() |
whitespace | OperatorWrap | '+' should be on a new line. | 110 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 120 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 134). | 124 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 138). | 138 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 121). | 149 |
![]() |
naming | ParameterName | Parameter name 'pSwitchExists' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 172 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 175 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 176 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 180 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 182 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 184 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 192 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 196 |
![]() |
naming | LocalVariableName | Local variable name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 201 |
![]() |
whitespace | SeparatorWrap | '.' should be on a new line. | 209 |
![]() |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 216 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 232 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 234 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 247 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 249 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 262 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 264 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 275 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 276 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 309 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 329 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 23 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.ListenableFuture' import. | 20 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | EmptyLineSeparator | 'package' should be separated from previous statement. | 8 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.md.sal.binding.api.DataObjectModification.ModificationType' import. | 22 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.hwvtep.global.attributes.LogicalSwitches' import. | 34 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 41 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 49 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 50 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 51 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 57 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 58 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 58 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 64 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'switch' is not followed by whitespace. | 66 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 71 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 81 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'switch' is not followed by whitespace. | 83 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 88 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 102 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 105 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 124 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 158). | 135 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 135 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 128). | 140 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 140 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 137). | 165 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 126). | 172 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 172 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 128). | 174 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 133). | 182 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 129). | 184 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 121). | 189 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 195 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 196 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 196 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 199 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 199 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 153). | 200 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 200 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 200 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 33 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 66 |
![]() |
whitespace | SeparatorWrap | '.' should be on a new line. | 68 |
![]() |
naming | LocalVariableName | Local variable name 'i' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 119 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 120 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 38 |
![]() |
coding | VariableDeclarationUsageDistance | Distance between variable 'operationalMacOptional' 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). | 72 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 129). | 74 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 84 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 141). | 103 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 108 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 111 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 130). | 122 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 33 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 66 |
![]() |
whitespace | SeparatorWrap | '.' should be on a new line. | 68 |
![]() |
naming | LocalVariableName | Local variable name 'i' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 119 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 120 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.Node' import. | 31 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 36 |
![]() |
coding | VariableDeclarationUsageDistance | Distance between variable 'operationalMacOptional' 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). | 70 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 132). | 72 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 82 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 141). | 101 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 106 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 130). | 119 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 38 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 24 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 28 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 34 |
![]() |
naming | LocalVariableName | Local variable name 'lSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 36 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 36 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 40 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 122). | 42 |
![]() |
naming | ParameterName | Parameter name 'lSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 54 |
![]() |
naming | ParameterName | Parameter name 'lSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 65 |
![]() |
coding | VariableDeclarationUsageDistance | Distance between variable 'hgAugmentationBuilder' 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). | 69 |
![]() |
coding | VariableDeclarationUsageDistance | Distance between variable 'lSwitches' 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). | 70 |
![]() |
naming | LocalVariableName | Local variable name 'lSwitches' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 70 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | 'org.opendaylight.controller.md.sal.binding.api.ReadWriteTransaction'should be separated from previous import group. | 12 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 60 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 63 |
![]() |
whitespace | OperatorWrap | '&&' should be on a new line. | 63 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 77 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 80 |
![]() |
whitespace | OperatorWrap | '&&' should be on a new line. | 80 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 94 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 97 |
![]() |
whitespace | OperatorWrap | '&&' should be on a new line. | 97 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 112 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 115 |
![]() |
whitespace | OperatorWrap | '&&' should be on a new line. | 115 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 129 |
![]() |
naming | LocalVariableName | Local variable name 'lSwitchIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 132 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | 'org.opendaylight.controller.md.sal.binding.api.ReadWriteTransaction'should be separated from previous import group. | 12 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 131). | 38 |
![]() |
indentation | Indentation | 'for' child have incorrect indentation level 20, expected level should be 12. | 44 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 12 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 13 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 14 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 15 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 45 |
![]() |
coding | VariableDeclarationUsageDistance | Distance between variable 'hgAugmentationBuilder' 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). | 73 |
![]() |
coding | VariableDeclarationUsageDistance | Distance between variable 'mList' 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 |
![]() |
naming | LocalVariableName | Local variable name 'mList' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 74 |
![]() |
naming | LocalVariableName | Local variable name 'mBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 75 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 13 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 14 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 15 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 53 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 53 |
![]() |
naming | ParameterName | Parameter name 'mMacLocal' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 67 |
![]() |
naming | ParameterName | Parameter name 'mMacLocal' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 78 |
![]() |
coding | VariableDeclarationUsageDistance | Distance between variable 'hgAugmentationBuilder' 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). | 81 |
![]() |
naming | LocalVariableName | Local variable name 'mMacLocalBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 82 |
![]() |
naming | LocalVariableName | Local variable name 'mMacLocalList' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 93 |
![]() |
naming | ParameterName | Parameter name 'mMacLocalBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 100 |
![]() |
naming | ParameterName | Parameter name 'mMacLocal' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 100 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 102 |
![]() |
naming | LocalVariableName | Local variable name 'lSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 103 |
![]() |
naming | LocalVariableName | Local variable name 'lSwitchIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 105 |
![]() |
naming | ParameterName | Parameter name 'mMacLocalBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 112 |
![]() |
naming | ParameterName | Parameter name 'mMacLocal' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 112 |
![]() |
naming | ParameterName | Parameter name 'mMacLocalBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 118 |
![]() |
naming | ParameterName | Parameter name 'mMacLocal' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 118 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 120 |
![]() |
naming | LocalVariableName | Local variable name 'pLocSetUUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 120 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 126 |
![]() |
naming | LocalVariableName | Local variable name 'pLocUUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 126 |
![]() |
naming | LocalVariableName | Local variable name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 127 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 13 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 14 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 15 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 16 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 124). | 55 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 55 |
![]() |
naming | ParameterName | Parameter name 'mMacRemote' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 69 |
![]() |
naming | ParameterName | Parameter name 'mMacRemote' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 80 |
![]() |
coding | VariableDeclarationUsageDistance | Distance between variable 'hgAugmentationBuilder' 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). | 83 |
![]() |
naming | LocalVariableName | Local variable name 'mMacRemoteBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 84 |
![]() |
naming | LocalVariableName | Local variable name 'mMacRemoteList' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 95 |
![]() |
naming | ParameterName | Parameter name 'mMacRemoteBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 102 |
![]() |
naming | ParameterName | Parameter name 'mMacRemote' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 102 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 104 |
![]() |
naming | LocalVariableName | Local variable name 'lSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 105 |
![]() |
naming | LocalVariableName | Local variable name 'lSwitchIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 107 |
![]() |
naming | ParameterName | Parameter name 'mMacRemoteBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 114 |
![]() |
naming | ParameterName | Parameter name 'mMacRemote' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 114 |
![]() |
naming | ParameterName | Parameter name 'mMacRemoteBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 120 |
![]() |
naming | ParameterName | Parameter name 'mMacRemote' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 120 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 122 |
![]() |
naming | LocalVariableName | Local variable name 'pLocSetUUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 122 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 128 |
![]() |
naming | LocalVariableName | Local variable name 'pLocUUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 128 |
![]() |
naming | LocalVariableName | Local variable name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 129 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 23 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | LineLength | Line is longer than 120 characters (found 124). | 27 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 33 |
![]() |
naming | LocalVariableName | Local variable name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 35 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 34 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 124). | 41 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 41 |
![]() |
naming | LocalVariableName | Local variable name 'pLocUpdate' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 60 |
![]() |
naming | LocalVariableName | Local variable name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 61 |
![]() |
naming | ParameterName | Parameter name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 88 |
![]() |
naming | ParameterName | Parameter name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 95 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | LineLength | Line is longer than 120 characters (found 121). | 35 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 43 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 45 |
![]() |
naming | LocalVariableName | Local variable name 'pPort' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 47 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 50 |
![]() |
naming | LocalVariableName | Local variable name 'pSwitchUUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 50 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 53 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 59 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 60 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 121). | 63 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 63 |
![]() |
naming | LocalVariableName | Local variable name 'pPortUpdateEntry' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 85 |
![]() |
naming | LocalVariableName | Local variable name 'pPortUpdate' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 86 |
![]() |
naming | LocalVariableName | Local variable name 'vBIiList' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 111 |
![]() |
naming | LocalVariableName | Local variable name 'vBindings' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 117 |
![]() |
naming | LocalVariableName | Local variable name 'vBid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 118 |
![]() |
naming | ParameterName | Parameter name 'vBindings' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 138 |
![]() |
naming | LocalVariableName | Local variable name 'lSwitchRef' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 182 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 187 |
![]() |
naming | LocalVariableName | Local variable name 'lSwitchIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 190 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 200 |
![]() |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '137'. | 229 |
![]() |
naming | ParameterName | Parameter name 'pPort' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 230 |
![]() |
naming | ParameterName | Parameter name 'pPort' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 248 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | 'org.opendaylight.controller.md.sal.binding.api.ReadWriteTransaction'should be separated from previous import group. | 12 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 29 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 35 |
![]() |
naming | LocalVariableName | Local variable name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 37 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 55 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 61 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 63 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 65 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 65 |
![]() |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 83 |
![]() |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 106 |
![]() |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 125 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 129 |
![]() |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 140 |
![]() |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 149 |
![]() |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 162 |
![]() |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 174 |
![]() |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 193 |
![]() |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 197 |
![]() |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 210 |
![]() |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 228 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 36 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 50 |
![]() |
naming | LocalVariableName | Local variable name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 64 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 65 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 65 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 72 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 82 |
![]() |
naming | LocalVariableName | Local variable name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 83 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 84 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 46 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 54 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 65 |
![]() |
naming | LocalVariableName | Local variable name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 81 |
![]() |
naming | LocalVariableName | Local variable name 'tBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 87 |
![]() |
naming | ParameterName | Parameter name 'tBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 106 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 108 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 112 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 114 |
![]() |
naming | ParameterName | Parameter name 'tBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 125 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 127 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 131 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 133 |
![]() |
naming | ParameterName | Parameter name 'tBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 145 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 147 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 151 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 153 |
![]() |
naming | ParameterName | Parameter name 'tBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 164 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 166 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 170 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 172 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 191 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 197 |
![]() |
naming | LocalVariableName | Local variable name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 199 |
![]() |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 200 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 13 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 14 |
![]() |
imports | CustomImportOrder | Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. | 15 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 123). | 49 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 49 |
![]() |
coding | VariableDeclarationUsageDistance | Distance between variable 'hgAugmentationBuilder' 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). | 78 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 86 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 95 |
![]() |
naming | LocalVariableName | Local variable name 'lSwitchIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 98 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. | 42 |
![]() |
sizes | LineLength | Line is longer than 120 characters (found 124). | 49 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 49 |
![]() |
naming | ParameterName | Parameter name 'uMacRemote' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 72 |
![]() |
coding | VariableDeclarationUsageDistance | Distance between variable 'hgAugmentationBuilder' 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). | 76 |
![]() |
coding | VariableDeclarationUsageDistance | Distance between variable 'remoteUMacs' 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). | 77 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 86 |
![]() |
naming | LocalVariableName | Local variable name 'pLocUUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 86 |
![]() |
naming | AbbreviationAsWordInName | Abbreviation in name must contain no more than '1' capital letters. | 96 |
![]() |
naming | LocalVariableName | Local variable name 'lSwitchIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 99 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.Executors' import. | 17 |
![]() |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.FutureCallback' import. | 30 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 53 |