Checkstyle Results
The following document contains the results of Checkstyle 6.16 with odl_checks.xml ruleset. 
InfoFiles
Rules
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | EmptyBlock
|
1 |  Warning |
| NeedBraces | 1 | Warning | |
RightCurly
|
1 | Warning | |
| RightCurly | 1 | Warning | |
| coding | IllegalCatch | 9 | Warning |
| MissingSwitchDefault | 2 | Warning | |
| OverloadMethodsDeclarationOrder | 13 | Warning | |
| VariableDeclarationUsageDistance | 14 | Warning | |
| imports | AvoidStarImport | 1 | Warning |
CustomImportOrder
|
168 | Warning | |
| UnusedImports | 5 | Warning | |
| indentation | Indentation
|
49 | Warning |
| javadoc | JavadocParagraph
|
8 | Warning |
JavadocStyle
|
11 | Warning | |
| NonEmptyAtclauseDescription | 1 | Warning | |
| misc | ArrayTypeStyle | 7 | Warning |
| modifier | RedundantModifier | 2 | Warning |
| naming | AbbreviationAsWordInName
|
3 | Warning |
ClassTypeParameterName
|
1 | Warning | |
| ConstantName | 2 | Warning | |
LocalVariableName
|
46 | Warning | |
MemberName
|
2 | Warning | |
MethodTypeParameterName
|
4 | Warning | |
ParameterName
|
63 | Warning | |
| regexp | RegexpSingleline
|
6 |  Error |
| sizes | LineLength
|
109 | Warning |
| whitespace | EmptyLineSeparator
|
10 | Warning |
| MethodParamPad | 3 | Warning | |
OperatorWrap
|
15 | Warning | |
SeparatorWrap
|
28 | Warning | |
SeparatorWrap
|
28 | Warning | |
WhitespaceAround
|
113 | Warning |
Details
org/opendaylight/ovsdb/hwvtepsouthbound/DataChangeListenerTestBase.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.io.InputStream' import. Should be before 'org.slf4j.LoggerFactory'. | 59 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.net.InetAddress' import. Should be before 'org.slf4j.LoggerFactory'. | 60 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 61 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Matchers.any' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 63 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Matchers.anyString' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 64 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Mockito.doReturn' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 65 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Mockito.mock' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 66 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Mockito.when' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 67 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType.CONFIGURATION' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 68 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType.OPERATIONAL' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 69 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.powermock.api.support.membermodification.MemberMatcher.field' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 70 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.powermock.api.support.membermodification.MemberModifier.suppress' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 71 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 78 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 111 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 113 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 133 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 140 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 149 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 132). | 150 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 153 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 155 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 169 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 184 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 184 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'update' 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). | 189 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 249 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 260 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 268 |
org/opendaylight/ovsdb/hwvtepsouthbound/DependencyQueueTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.powermock.reflect.Whitebox'. | 35 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.powermock.reflect.Whitebox'. | 36 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.CountDownLatch' import. Should be before 'org.powermock.reflect.Whitebox'. | 37 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.junit.Assert.assertEquals' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 39 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType.CONFIGURATION' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 40 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 43 |
| Warning |
naming | AbbreviationAsWordInName | Abbreviation in name 'MCAST_MAC_DATA_VALIDATOR' must contain no more than '4' capital letters. | 51 |
| Warning |
naming | MemberName | Member name 'MCAST_MAC_DATA_VALIDATOR' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 51 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 59 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 63 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 65 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 79 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 96 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 98 |
org/opendaylight/ovsdb/hwvtepsouthbound/HwvtepConnectionInstance.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. Should be before 'org.slf4j.LoggerFactory'. | 54 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.ListenableFuture' import. Should be before 'org.slf4j.LoggerFactory'. | 55 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 73 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 92 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 117 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '84'. | 178 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 302 |
org/opendaylight/ovsdb/hwvtepsouthbound/HwvtepConnectionManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'javax.annotation.Nonnull' import. Should be before 'org.slf4j.LoggerFactory'. | 55 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'javax.annotation.Nullable' import. Should be before 'org.slf4j.LoggerFactory'. | 56 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.net.ConnectException' import. Should be before 'org.slf4j.LoggerFactory'. | 57 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.net.InetAddress' import. Should be before 'org.slf4j.LoggerFactory'. | 58 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.net.UnknownHostException' import. Should be before 'org.slf4j.LoggerFactory'. | 59 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.slf4j.LoggerFactory'. | 60 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 61 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.slf4j.LoggerFactory'. | 62 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.ConcurrentHashMap' import. Should be before 'org.slf4j.LoggerFactory'. | 63 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.ExecutionException' import. Should be before 'org.slf4j.LoggerFactory'. | 64 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.TimeUnit' import. Should be before 'org.slf4j.LoggerFactory'. | 65 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.TimeoutException' import. Should be before 'org.slf4j.LoggerFactory'. | 66 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.opendaylight.ovsdb.lib.operations.Operations.op' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 68 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 70 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 118 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 170 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 183 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '229'. | 249 |
| Warning |
naming | LocalVariableName | Local variable name 'pSwitchNode' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 252 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 254 |
| Warning |
blocks | RightCurly | '}' at column 9 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else or try/catch/finally). | 261 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 262 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 262 |
| Warning |
naming | ParameterName | Parameter name 'pNode' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 270 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 272 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 410 |
| Warning |
indentation | Indentation | 'method def rcurly' have incorrect indentation level 8, expected level should be 4. | 418 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 437 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 437 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'switch' is not followed by whitespace. | 440 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 440 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 455 |
| Warning |
whitespace | SeparatorWrap | ',' should be on the previous line. | 461 |
| Warning |
naming | ParameterName | Parameter name 't' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 466 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 478 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '223'. | 554 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '294'. | 559 |
| Warning |
indentation | Indentation | 'if' have incorrect indentation level 9, expected level should be 8. | 560 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 13, expected level should be 12. | 561 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 9, expected level should be 8. | 562 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 569 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 573 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 576 |
org/opendaylight/ovsdb/hwvtepsouthbound/HwvtepDataChangeListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | UnusedImports | Unused import - com.google.common.base.Optional. | 11 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.controller.md.sal.binding.api.ReadWriteTransaction. | 27 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 63 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 71 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 85 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 87 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 89 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 211 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 212 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 219 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 219 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'switch' is not followed by whitespace. | 227 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 232 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'switch' is not followed by whitespace. | 244 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 249 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '!=' is not preceded with whitespace. | 276 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '!=' is not followed by whitespace. | 276 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 281 |
org/opendaylight/ovsdb/hwvtepsouthbound/HwvtepDataChangeListenerTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.lang.reflect.Field' import. Should be before 'org.slf4j.LoggerFactory'. | 37 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.lang.reflect.Modifier' import. Should be before 'org.slf4j.LoggerFactory'. | 38 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Iterator' import. Should be before 'org.slf4j.LoggerFactory'. | 39 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 40 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.junit.Assert.assertNotNull' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 42 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.junit.Assert.assertTrue' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 43 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Matchers.any' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 44 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Mockito.times' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 45 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.mockito.Mockito.verify' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 46 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType.CONFIGURATION' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 47 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType.OPERATIONAL' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 48 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 98 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 105 |
| Warning |
blocks | EmptyBlock | Empty catch block. | 105 |
| Warning |
misc | ArrayTypeStyle | Array brackets at illegal position. | 110 |
org/opendaylight/ovsdb/hwvtepsouthbound/HwvtepDeviceInfo.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.HashMap' import. Should be before 'org.slf4j.LoggerFactory'. | 25 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Iterator' import. Should be before 'org.slf4j.LoggerFactory'. | 26 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.slf4j.LoggerFactory'. | 27 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.ConcurrentHashMap' import. Should be before 'org.slf4j.LoggerFactory'. | 28 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 92 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 93 |
| Warning |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 121 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 203 |
org/opendaylight/ovsdb/hwvtepsouthbound/HwvtepOperationalDataChangeListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.NetworkTopology' import. Should be before 'org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.node.TerminationPoint'. | 24 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.Topology' import. Should be before 'org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.node.TerminationPoint'. | 25 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.TopologyKey' import. Should be before 'org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.node.TerminationPoint'. | 26 |
| Warning |
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. Should be before 'org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.node.TerminationPoint'. | 27 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Collection' import. Should be before 'org.slf4j.LoggerFactory'. | 36 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 130). | 47 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 57 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 70 |
| Warning |
coding | MissingSwitchDefault | switch without "default" clause. | 87 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'switch' is not followed by whitespace. | 87 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 114 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 172). | 115 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 120 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 123 |
org/opendaylight/ovsdb/hwvtepsouthbound/HwvtepOperationalStateTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 37 |
org/opendaylight/ovsdb/hwvtepsouthbound/HwvtepSchemaConstants.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | EmptyLineSeparator | 'ENUM_DEF' should be separated from previous statement. | 13 |
| Warning |
naming | AbbreviationAsWordInName | Abbreviation in name 'HWVTEPSCHEMATABLES' must contain no more than '4' capital letters. | 13 |
org/opendaylight/ovsdb/hwvtepsouthbound/HwvtepSouthboundConstants.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.ImmutableBiMap' import. Should be before 'org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.TopologyId'. | 20 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.ImmutableMap' import. Should be before 'org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.TopologyId'. | 21 |
| Warning |
indentation | Indentation | '=' have incorrect indentation level 4, expected level should be 8. | 30 |
org/opendaylight/ovsdb/hwvtepsouthbound/HwvtepSouthboundMapper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.EncapsulationTypeBase' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.hwvtep.physical.port.attributes.VlanBindingsKey'. | 33 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.EncapsulationTypeVxlanOverIpv4' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.hwvtep.physical.port.attributes.VlanBindingsKey'. | 34 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.HwvtepGlobalAugmentation' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.hwvtep.physical.port.attributes.VlanBindingsKey'. | 35 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.HwvtepNodeName' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.hwvtep.physical.port.attributes.VlanBindingsKey'. | 36 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.HwvtepPhysicalLocatorRef' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.hwvtep.physical.port.attributes.VlanBindingsKey'. | 37 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.HwvtepPhysicalPortAugmentation' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.hwvtep.physical.port.attributes.VlanBindingsKey'. | 38 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.PhysicalSwitchAugmentation' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.hwvtep.physical.port.attributes.VlanBindingsKey'. | 39 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.hwvtep.global.attributes.ConnectionInfoBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.hwvtep.physical.port.attributes.VlanBindingsKey'. | 40 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Preconditions' import. Should be before 'org.slf4j.LoggerFactory'. | 54 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.ImmutableBiMap' import. Should be before 'org.slf4j.LoggerFactory'. | 55 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.net.InetAddresses' import. Should be before 'org.slf4j.LoggerFactory'. | 56 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 69 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 137 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '74'. | 148 |
| Warning |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 149 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '86'. | 162 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '154'. | 169 |
| Warning |
naming | ParameterName | Parameter name 'lSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 170 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '184'. | 205 |
| Warning |
naming | ParameterName | Parameter name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 210 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 212 |
| Warning |
whitespace | OperatorWrap | '&&' should be on a new line. | 212 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 214 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 214 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 214 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 214 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '205'. | 224 |
| Warning |
naming | ParameterName | Parameter name 'vBindings' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 225 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 16, expected level should be 8. | 226 |
org/opendaylight/ovsdb/hwvtepsouthbound/HwvtepSouthboundProvider.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 36 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.slf4j.LoggerFactory'. | 37 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 107 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 107 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 111 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 115 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 119 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 140 |
| Warning |
whitespace | SeparatorWrap | ',' should be on the previous line. | 149 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 172 |
org/opendaylight/ovsdb/hwvtepsouthbound/HwvtepSouthboundUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yangtools.yang.binding.DataObject. | 31 |
| Warning |
naming | ParameterName | Parameter name 'pNode' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 84 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 131 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 136 |
| Warning |
naming | MethodTypeParameterName | Method type name 'KeyType' must match pattern '(^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)'. | 146 |
| Warning |
naming | MethodTypeParameterName | Method type name 'KeyType' must match pattern '(^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)'. | 157 |
| Warning |
naming | MethodTypeParameterName | Method type name 'KeyType' must match pattern '(^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)'. | 168 |
| Warning |
naming | MethodTypeParameterName | Method type name 'KeyType' must match pattern '(^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)'. | 179 |
org/opendaylight/ovsdb/hwvtepsouthbound/TestBuilders.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Arrays' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 39 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 40 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.UUID' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 41 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 47 |
| Warning |
misc | ArrayTypeStyle | Array brackets at illegal position. | 49 |
| Warning |
misc | ArrayTypeStyle | Array brackets at illegal position. | 59 |
| Warning |
misc | ArrayTypeStyle | Array brackets at illegal position. | 60 |
| Warning |
misc | ArrayTypeStyle | Array brackets at illegal position. | 71 |
| Warning |
misc | ArrayTypeStyle | Array brackets at illegal position. | 81 |
| Warning |
naming | LocalVariableName | Local variable name 'lSwitchIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 89 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 106 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 106 |
| Warning |
misc | ArrayTypeStyle | Array brackets at illegal position. | 134 |
| Warning |
naming | LocalVariableName | Local variable name 'mMacLocalBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 136 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 163 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 163 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 163 |
org/opendaylight/ovsdb/hwvtepsouthbound/UnMetDependencyGetterTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.powermock.reflect.Whitebox'. | 30 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.powermock.reflect.Whitebox'. | 31 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.junit.Assert.assertEquals' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 33 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.junit.Assert.assertNull' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 34 |
| Warning |
naming | AbbreviationAsWordInName | Abbreviation in name 'MCAST_MAC_DATA_VALIDATOR' must contain no more than '4' capital letters. | 40 |
| Warning |
naming | MemberName | Member name 'MCAST_MAC_DATA_VALIDATOR' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 40 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 47 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 51 |
org/opendaylight/ovsdb/hwvtepsouthbound/reconciliation/ReconciliationManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | AvoidStarImport | Using the '.*' form of import should be avoided - java.util.concurrent.*. | 16 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.*' import. Should be before 'org.slf4j.LoggerFactory'. | 16 |
| Warning |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 20 |
| Warning |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 33 |
| Warning |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 42 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 59 |
org/opendaylight/ovsdb/hwvtepsouthbound/reconciliation/ReconciliationTask.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 21 |
| Warning |
naming | ParameterName | Parameter name 'o' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 102 |
| Warning |
blocks | NeedBraces | 'if' construct must use '{}'s. | 103 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 122 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 122 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 123 |
org/opendaylight/ovsdb/hwvtepsouthbound/reconciliation/ReconciliationTaskManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.ConcurrentHashMap' import. Should be before 'org.slf4j.LoggerFactory'. | 13 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.Future' import. Should be before 'org.slf4j.LoggerFactory'. | 14 |
| Warning |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 18 |
| Warning |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 24 |
| Warning |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 31 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 43 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 44 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 44 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 46 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 54 |
org/opendaylight/ovsdb/hwvtepsouthbound/reconciliation/configuration/DataObjectModificationImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yangtools.yang.binding.Identifier' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 13 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yangtools.yang.binding.Identifiable' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 14 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yangtools.yang.binding.ChildOf' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 15 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yangtools.yang.binding.Augmentation' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 16 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 18 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Collection' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 19 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 94 |
org/opendaylight/ovsdb/hwvtepsouthbound/reconciliation/configuration/GlobalConfigOperationalChangeGetter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 23 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.HashSet' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 24 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 25 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Set' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 26 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 48 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 63 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 79 |
org/opendaylight/ovsdb/hwvtepsouthbound/reconciliation/configuration/HwvtepReconciliationManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.md.sal.binding.api.DataObjectModification' import. Should be before 'org.opendaylight.controller.md.sal.binding.api.DataTreeModification'. | 14 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.md.sal.binding.api.DataObjectModification.ModificationType' import. Should be before 'org.opendaylight.controller.md.sal.binding.api.DataTreeModification'. | 15 |
| Warning |
imports | UnusedImports | Unused import - org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.HwvtepPhysicalSwitchAttributes. | 21 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Collection' import. Should be before 'org.slf4j.LoggerFactory'. | 29 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 53 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 99 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 100 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 107 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 107 |
org/opendaylight/ovsdb/hwvtepsouthbound/reconciliation/configuration/HwvtepReconciliationTask.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.slf4j.LoggerFactory'. | 28 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Collection' import. Should be before 'org.slf4j.LoggerFactory'. | 29 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType.CONFIGURATION' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 31 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType.OPERATIONAL' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 32 |
org/opendaylight/ovsdb/hwvtepsouthbound/reconciliation/configuration/SwitchConfigOperationalChangeGetter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.slf4j.LoggerFactory'. | 23 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 24 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 80 |
org/opendaylight/ovsdb/hwvtepsouthbound/reconciliation/connection/ConnectionReconciliationTask.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.ovsdb.hwvtepsouthbound.HwvtepConnectionManager' import. Should be before 'org.opendaylight.ovsdb.lib.OvsdbClient'. | 13 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.net.ConnectException' import. Should be before 'org.slf4j.LoggerFactory'. | 21 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.net.UnknownHostException' import. Should be before 'org.slf4j.LoggerFactory'. | 22 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.atomic.AtomicInteger' import. Should be before 'org.slf4j.LoggerFactory'. | 23 |
| Warning |
javadoc | JavadocParagraph | Empty line should be followed by <p> tag on the next line. | 27 |
| Warning |
naming | LocalVariableName | Local variable name 'nIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 49 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 72 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/AbstractTransactCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. Should be before 'java.util.Set'. | 23 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 133). | 37 |
| Warning |
naming | ClassTypeParameterName | Class type name 'Aug' must match pattern '(^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)'. | 37 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 90 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 138). | 139 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 127). | 207 |
| Warning |
naming | ParameterName | Parameter name 'a' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 235 |
| Warning |
naming | ParameterName | Parameter name 'b' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 235 |
| Warning |
naming | ParameterName | Parameter name 'a' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 246 |
| Warning |
naming | ParameterName | Parameter name 'b' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 246 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'while' is not followed by whitespace. | 261 |
| Warning |
naming | ParameterName | Parameter name 'a' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 287 |
| Warning |
naming | ParameterName | Parameter name 'b' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 287 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 295 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 295 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/DependencyQueue.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.slf4j.LoggerFactory'. | 21 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Iterator' import. Should be before 'org.slf4j.LoggerFactory'. | 22 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 23 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.ExecutorService' import. Should be before 'org.slf4j.LoggerFactory'. | 24 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.Executors' import. Should be before 'org.slf4j.LoggerFactory'. | 25 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.LinkedBlockingQueue' import. Should be before 'org.slf4j.LoggerFactory'. | 26 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.ThreadFactory' import. Should be before 'org.slf4j.LoggerFactory'. | 27 |
| Warning |
naming | ConstantName | Name 'threadFact' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 32 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 32 |
| Warning |
naming | ConstantName | Name 'executorService' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 34 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 46 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 46 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 66 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 66 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 74 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 74 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'while' is not followed by whitespace. | 111 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/DependentJob.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 18 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 19 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 36 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 36 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 44 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 44 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 58 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 58 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/HwvtepOperationalState.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Collection' import. Should be before 'org.slf4j.LoggerFactory'. | 49 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Collections' import. Should be before 'org.slf4j.LoggerFactory'. | 50 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.HashMap' import. Should be before 'org.slf4j.LoggerFactory'. | 51 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 52 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.slf4j.LoggerFactory'. | 53 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map.Entry' import. Should be before 'org.slf4j.LoggerFactory'. | 54 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Set' import. Should be before 'org.slf4j.LoggerFactory'. | 55 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.concurrent.ConcurrentHashMap' import. Should be before 'org.slf4j.LoggerFactory'. | 56 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 68 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 130). | 69 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 125 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 126 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 130 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 139 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 182 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 229 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 245 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 335 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '238'. | 335 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '182'. | 343 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '202'. | 348 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/LogicalSwitchRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 33 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 69 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 132). | 83 |
| Warning |
whitespace | OperatorWrap | '&&' should be on a new line. | 85 |
| Warning |
naming | ParameterName | Parameter name 'a' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 104 |
| Warning |
naming | ParameterName | Parameter name 'b' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 104 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/LogicalSwitchUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 33 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 58 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 63 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 135). | 77 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 89 |
| Warning |
naming | ParameterName | Parameter name 'a' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 117 |
| Warning |
naming | ParameterName | Parameter name 'b' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 117 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/McastMacsLocalRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. Should be before 'java.util.Map.Entry'. | 19 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 32 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 67 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/McastMacsLocalUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 33 |
| Warning |
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). | 59 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 61 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 141). | 90 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 98 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 135). | 106 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/McastMacsRemoteRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 33 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 86 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 93 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 93 |
| Warning |
naming | ParameterName | Parameter name 'a' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 105 |
| Warning |
naming | ParameterName | Parameter name 'b' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 105 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/McastMacsRemoteUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. Should be before 'java.util.Objects'. | 21 |
| Warning |
imports | CustomImportOrder | Import statement for 'org.opendaylight.ovsdb.lib.operations.Operations.op' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line. | 39 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 78 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 89 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 90 |
| Warning |
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). | 90 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 92 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 132). | 92 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 93 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 94 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 95 |
| Warning |
indentation | Indentation | 'if' have incorrect indentation level 12, expected level should be 8. | 96 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 97 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 98 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 99 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 100 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 12, expected level should be 8. | 101 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 102 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 103 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 105 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 106 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 107 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 110 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 12, expected level should be 8. | 111 |
| Warning |
indentation | Indentation | 'else' child have incorrect indentation level 16, expected level should be 12. | 112 |
| Warning |
indentation | Indentation | 'else rcurly' have incorrect indentation level 12, expected level should be 8. | 114 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 141). | 120 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 133 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 135). | 135 |
| Warning |
naming | ParameterName | Parameter name 'a' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 165 |
| Warning |
naming | ParameterName | Parameter name 'b' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 165 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/PhysicalPortRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 35 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Sets' import. Should be before 'org.slf4j.LoggerFactory'. | 36 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 66 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 70 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 79 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 101 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 121 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 129 |
| Warning |
naming | LocalVariableName | Local variable name 'i' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 138 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 139 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/PhysicalPortUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 40 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Sets' import. Should be before 'org.slf4j.LoggerFactory'. | 41 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 83 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 133). | 96 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 103 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 105 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 165 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'else' is not preceded with whitespace. | 166 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '}' is not followed by whitespace. | 166 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'else' is not followed by whitespace. | 166 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 166 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 167 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 186 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 208 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 232 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 240 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/PhysicalSwitchRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 32 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Sets' import. Should be before 'org.slf4j.LoggerFactory'. | 33 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 60 |
| Warning |
whitespace | OperatorWrap | '&&' should be on a new line. | 61 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 63 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 130). | 68 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 148). | 72 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 88 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/PhysicalSwitchUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 47 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.ImmutableMap' import. Should be before 'org.slf4j.LoggerFactory'. | 48 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Sets' import. Should be before 'org.slf4j.LoggerFactory'. | 49 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 85 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 100 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 110 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 117 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 120 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 124 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 138). | 138 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 149 |
| Warning |
naming | ParameterName | Parameter name 'pSwitchExists' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 172 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 175 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 176 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 184 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 192 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 196 |
| Warning |
naming | LocalVariableName | Local variable name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 201 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 209 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 216 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 232 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 234 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 247 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 249 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 262 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 264 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 308 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 327 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/TransactCommandAggregator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.slf4j.LoggerFactory'. | 25 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Collection' import. Should be before 'org.slf4j.LoggerFactory'. | 26 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.HashMap' import. Should be before 'org.slf4j.LoggerFactory'. | 27 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.slf4j.LoggerFactory'. | 28 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.slf4j.LoggerFactory'. | 29 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Objects' import. Should be before 'org.slf4j.LoggerFactory'. | 30 |
| Warning |
coding | MissingSwitchDefault | switch without "default" clause. | 148 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/TransactInvokerImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Strings' import. Should be before 'java.util.Map'. | 14 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.ListenableFuture' import. Should be before 'org.slf4j.LoggerFactory'. | 26 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 62 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/TransactUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.controller.md.sal.binding.api.DataObjectModification.ModificationType' import. Should be before 'org.opendaylight.controller.md.sal.binding.api.ReadWriteTransaction'. | 22 |
| Warning |
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. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.hwvtep.physical.locator.set.attributes.LocatorSet'. | 38 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 45 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 54 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 55 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 62 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 62 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'switch' is not followed by whitespace. | 70 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 75 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'switch' is not followed by whitespace. | 87 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 92 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 109 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 158). | 139 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 144 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not followed by whitespace. | 144 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 137). | 162 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not preceded with whitespace. | 180 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '+' is not followed by whitespace. | 180 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 190 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 128). | 192 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 133). | 200 |
| Warning |
modifier | RedundantModifier | Redundant 'final' modifier. | 200 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 202 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 207 |
| Warning |
modifier | RedundantModifier | Redundant 'final' modifier. | 207 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 218 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 222 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 223 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/UcastMacsLocalRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. Should be before 'java.util.Map.Entry'. | 19 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 32 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 67 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/UcastMacsLocalUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 34 |
| Warning |
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). | 60 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 129). | 62 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 141). | 91 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 99 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 130). | 110 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/UcastMacsRemoteRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 55 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 55 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 66 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/UcastMacsRemoteUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | UnusedImports | Unused import - com.google.common.base.Optional. | 13 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 73 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 84 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 85 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'deviceData' 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). | 85 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 88 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 132). | 88 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 89 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 90 |
| Warning |
indentation | Indentation | 'method def' child have incorrect indentation level 12, expected level should be 8. | 91 |
| Warning |
indentation | Indentation | 'if' have incorrect indentation level 12, expected level should be 8. | 92 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 93 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 94 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 95 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 96 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 12, expected level should be 8. | 97 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 98 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 99 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 101 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 102 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 103 |
| Warning |
indentation | Indentation | 'if' child have incorrect indentation level 16, expected level should be 12. | 106 |
| Warning |
indentation | Indentation | 'if rcurly' have incorrect indentation level 12, expected level should be 8. | 107 |
| Warning |
indentation | Indentation | 'else' child have incorrect indentation level 16, expected level should be 12. | 108 |
| Warning |
indentation | Indentation | 'else rcurly' have incorrect indentation level 12, expected level should be 8. | 110 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 130). | 133 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 138). | 135 |
org/opendaylight/ovsdb/hwvtepsouthbound/transact/UnMetDependencyGetter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 21 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.HashMap' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 22 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 23 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 24 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 26 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 26 |
| Warning |
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 28 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 35 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 35 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 47 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 47 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 94 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/AbstractTransactionCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 17 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepGlobalRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 24 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.CheckedFuture' import. Should be before 'org.slf4j.LoggerFactory'. | 25 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 66 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepLogicalSwitchRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 28 |
| Warning |
naming | LocalVariableName | Local variable name 'lSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 36 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepLogicalSwitchUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 36 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 122). | 42 |
| Warning |
naming | ParameterName | Parameter name 'lSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 54 |
| Warning |
naming | ParameterName | Parameter name 'lSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 69 |
| Warning |
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). | 73 |
| Warning |
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). | 74 |
| Warning |
naming | LocalVariableName | Local variable name 'lSwitches' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 74 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepMacEntriesRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 63 |
| Warning |
whitespace | OperatorWrap | '&&' should be on a new line. | 63 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 80 |
| Warning |
whitespace | OperatorWrap | '&&' should be on a new line. | 80 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 98 |
| Warning |
whitespace | OperatorWrap | '&&' should be on a new line. | 98 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 116 |
| Warning |
whitespace | OperatorWrap | '&&' should be on a new line. | 116 |
| Warning |
naming | LocalVariableName | Local variable name 'lSwitchIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 134 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepManagerRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 38 |
| Warning |
indentation | Indentation | 'for' child have incorrect indentation level 20, expected level should be 12. | 44 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepManagerUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
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 |
| Warning |
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 |
| Warning |
naming | LocalVariableName | Local variable name 'mList' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 74 |
| Warning |
naming | LocalVariableName | Local variable name 'mBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 75 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepMcastMacsLocalUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 53 |
| Warning |
naming | ParameterName | Parameter name 'mMacLocal' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 67 |
| Warning |
naming | ParameterName | Parameter name 'mMacLocal' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 78 |
| Warning |
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 |
| Warning |
naming | LocalVariableName | Local variable name 'mMacLocalBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 82 |
| Warning |
naming | LocalVariableName | Local variable name 'mMacLocalList' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 93 |
| Warning |
naming | ParameterName | Parameter name 'mMacLocalBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 100 |
| Warning |
naming | ParameterName | Parameter name 'mMacLocal' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 100 |
| Warning |
naming | LocalVariableName | Local variable name 'lSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 103 |
| Warning |
naming | LocalVariableName | Local variable name 'lSwitchIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 105 |
| Warning |
naming | ParameterName | Parameter name 'mMacLocalBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 112 |
| Warning |
naming | ParameterName | Parameter name 'mMacLocal' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 112 |
| Warning |
naming | ParameterName | Parameter name 'mMacLocalBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 118 |
| Warning |
naming | ParameterName | Parameter name 'mMacLocal' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 118 |
| Warning |
naming | LocalVariableName | Local variable name 'pLocSetUUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 120 |
| Warning |
naming | LocalVariableName | Local variable name 'pLocUUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 126 |
| Warning |
naming | LocalVariableName | Local variable name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 127 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepMcastMacsRemoteUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 56 |
| Warning |
naming | ParameterName | Parameter name 'mMacRemote' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 70 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 87 |
| Warning |
naming | ParameterName | Parameter name 'mMacRemote' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 92 |
| Warning |
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). | 95 |
| Warning |
naming | LocalVariableName | Local variable name 'mMacRemoteBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 96 |
| Warning |
naming | LocalVariableName | Local variable name 'mMacRemoteList' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 107 |
| Warning |
naming | ParameterName | Parameter name 'mMacRemoteBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 114 |
| Warning |
naming | ParameterName | Parameter name 'mMacRemote' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 114 |
| Warning |
naming | LocalVariableName | Local variable name 'lSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 117 |
| Warning |
naming | LocalVariableName | Local variable name 'lSwitchIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 119 |
| Warning |
naming | ParameterName | Parameter name 'mMacRemoteBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 126 |
| Warning |
naming | ParameterName | Parameter name 'mMacRemote' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 126 |
| Warning |
naming | ParameterName | Parameter name 'mMacRemoteBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 132 |
| Warning |
naming | ParameterName | Parameter name 'mMacRemote' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 132 |
| Warning |
naming | LocalVariableName | Local variable name 'pLocSetUUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 134 |
| Warning |
naming | LocalVariableName | Local variable name 'pLocUUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 140 |
| Warning |
naming | LocalVariableName | Local variable name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 141 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepPhysicalLocatorRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 27 |
| Warning |
naming | LocalVariableName | Local variable name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 35 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepPhysicalLocatorUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 32 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map.Entry' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 33 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 40 |
| Warning |
naming | LocalVariableName | Local variable name 'pLocUpdate' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 59 |
| Warning |
naming | LocalVariableName | Local variable name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 60 |
| Warning |
naming | ParameterName | Parameter name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 88 |
| Warning |
naming | ParameterName | Parameter name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 95 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepPhysicalPortRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 35 |
| Warning |
naming | LocalVariableName | Local variable name 'pPort' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 47 |
| Warning |
naming | LocalVariableName | Local variable name 'pSwitchUUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 50 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepPhysicalPortUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 53 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Preconditions' import. Should be before 'org.slf4j.LoggerFactory'. | 54 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 63 |
| Warning |
naming | LocalVariableName | Local variable name 'pPortUpdateEntry' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 85 |
| Warning |
naming | LocalVariableName | Local variable name 'pPortUpdate' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 86 |
| Warning |
naming | LocalVariableName | Local variable name 'vBIiList' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 111 |
| Warning |
naming | LocalVariableName | Local variable name 'vBindings' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 117 |
| Warning |
naming | LocalVariableName | Local variable name 'vBid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 118 |
| Warning |
naming | ParameterName | Parameter name 'vBindings' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 138 |
| Warning |
naming | LocalVariableName | Local variable name 'lSwitchRef' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 182 |
| Warning |
naming | LocalVariableName | Local variable name 'lSwitchIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 190 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '137'. | 230 |
| Warning |
naming | ParameterName | Parameter name 'pPort' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 231 |
| Warning |
naming | ParameterName | Parameter name 'pPort' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 249 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepPhysicalSwitchRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 29 |
| Warning |
naming | LocalVariableName | Local variable name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 37 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepPhysicalSwitchUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 68 |
| Warning |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 86 |
| Warning |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 142 |
| Warning |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 160 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 164 |
| Warning |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 175 |
| Warning |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 184 |
| Warning |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 197 |
| Warning |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 209 |
| Warning |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 228 |
| Warning |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 232 |
| Warning |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 245 |
| Warning |
naming | ParameterName | Parameter name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 263 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepTunnelRemoveCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 50 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 55 |
| Warning |
naming | LocalVariableName | Local variable name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 64 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 65 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '{' is not preceded with whitespace. | 65 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 72 |
| Warning |
naming | LocalVariableName | Local variable name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 83 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 84 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepTunnelUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Optional' import. Should be before 'org.slf4j.LoggerFactory'. | 46 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.base.Preconditions' import. Should be before 'org.slf4j.LoggerFactory'. | 47 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 65 |
| Warning |
coding | IllegalCatch | Catching 'Exception' is not allowed. | 69 |
| Warning |
naming | LocalVariableName | Local variable name 'pSwitch' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 81 |
| Warning |
naming | LocalVariableName | Local variable name 'tBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 87 |
| Warning |
naming | ParameterName | Parameter name 'tBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 106 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 108 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 112 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 114 |
| Warning |
naming | ParameterName | Parameter name 'tBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 125 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 127 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 131 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 133 |
| Warning |
naming | ParameterName | Parameter name 'tBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 145 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 147 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 151 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 153 |
| Warning |
naming | ParameterName | Parameter name 'tBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 164 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 166 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'for' is not followed by whitespace. | 170 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 172 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 191 |
| Warning |
naming | LocalVariableName | Local variable name 'pLoc' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 199 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 200 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepUcastMacsLocalUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 49 |
| Warning |
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 |
| Warning |
naming | LocalVariableName | Local variable name 'lSwitchIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 98 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/HwvtepUcastMacsRemoteUpdateCommand.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.ArrayList' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 40 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 41 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.opendaylight.yangtools.yang.binding.InstanceIdentifier'. | 42 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 49 |
| Warning |
whitespace | SeparatorWrap | '.' should be on a new line. | 79 |
| Warning |
naming | ParameterName | Parameter name 'uMacRemote' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 84 |
| Warning |
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). | 88 |
| Warning |
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). | 89 |
| Warning |
naming | LocalVariableName | Local variable name 'pLocUUID' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 98 |
| Warning |
naming | LocalVariableName | Local variable name 'lSwitchIid' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 111 |
org/opendaylight/ovsdb/hwvtepsouthbound/transactions/md/TransactionInvokerImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.FutureCallback' import. Should be before 'org.slf4j.LoggerFactory'. | 30 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.Futures' import. Should be before 'org.slf4j.LoggerFactory'. | 31 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.util.concurrent.ThreadFactoryBuilder' import. Should be before 'org.slf4j.LoggerFactory'. | 32 |
| Warning |
regexp | RegexpSingleline | Line has trailing spaces. | 36 |