Checkstyle Results
The following document contains the results of Checkstyle 6.16 with odl_checks.xml ruleset. 
Info
ErrorsFiles
Rules
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | LeftCurly
|
5 |  Warning |
RightCurly
|
4 | Warning | |
| RightCurly | 4 | Warning | |
| coding | OverloadMethodsDeclarationOrder | 1 | Warning |
| VariableDeclarationUsageDistance | 23 | Warning | |
| imports | CustomImportOrder
|
24 | Warning |
| indentation | Indentation
|
12 | Warning |
| javadoc | JavadocParagraph
|
1 | Warning |
JavadocStyle
|
56 | Warning | |
| JavadocTagContinuationIndentation | 4 | Warning | |
| NonEmptyAtclauseDescription | 4 | Warning | |
| modifier | ModifierOrder | 2 | Warning |
| naming | AbbreviationAsWordInName
|
1 | Warning |
LocalVariableName
|
12 | Warning | |
ParameterName
|
9 | Warning | |
| sizes | LineLength
|
20 | Warning |
| whitespace | EmptyLineSeparator
|
8 | Warning |
| MethodParamPad | 1 | Warning | |
OperatorWrap
|
1 | Warning | |
WhitespaceAround
|
11 | Warning |
Details
org/opendaylight/netvirt/utils/mdsal/openflow/ActionUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNsiCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNspCaseBuilder'. | 51 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNshc1CaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNspCaseBuilder'. | 52 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNshc2CaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNspCaseBuilder'. | 53 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNshc3CaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNspCaseBuilder'. | 54 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNshc4CaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNspCaseBuilder'. | 55 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNshMdtypeCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNspCaseBuilder'. | 56 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNshNpCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.dst.choice.grouping.dst.choice.DstNxNspCaseBuilder'. | 57 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nodes.node.table.flow.instructions.instruction.instruction.apply.actions._case.apply.actions.action.action.NxActionPushNshNodesNodeTableFlowApplyActionsCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nodes.node.table.flow.instructions.instruction.instruction.apply.actions._case.apply.actions.action.action.NxActionResubmitNodesNodeTableFlowApplyActionsCaseBuilder'. | 74 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nodes.node.table.flow.instructions.instruction.instruction.apply.actions._case.apply.actions.action.action.NxActionPopNshNodesNodeTableFlowApplyActionsCaseBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nodes.node.table.flow.instructions.instruction.instruction.apply.actions._case.apply.actions.action.action.NxActionResubmitNodesNodeTableFlowApplyActionsCaseBuilder'. | 75 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.push.nsh.grouping.NxPushNsh' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 90 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.push.nsh.grouping.NxPushNshBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 91 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.pop.nsh.grouping.NxPopNsh' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 92 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.pop.nsh.grouping.NxPopNshBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.nx.action.resubmit.grouping.NxResubmitBuilder'. | 93 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.math.BigInteger' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.src.choice.grouping.src.choice.SrcOfIpSrcCaseBuilder'. | 105 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'java.util.List' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.action.rev140714.src.choice.grouping.src.choice.SrcOfIpSrcCaseBuilder'. | 106 |
| Warning |
naming | LocalVariableName | Local variable name 'r' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 196 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 243 |
| Warning |
naming | LocalVariableName | Local variable name 'r' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 280 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 171). | 285 |
| Warning |
naming | LocalVariableName | Local variable name 'r' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 337 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 175). | 338 |
| Warning |
naming | LocalVariableName | Local variable name 'r' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 360 |
| Warning |
naming | LocalVariableName | Local variable name 'r' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 366 |
| Warning |
naming | LocalVariableName | Local variable name 'r' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 372 |
| Warning |
naming | LocalVariableName | Local variable name 'r' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 483 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 173). | 489 |
| Warning |
naming | LocalVariableName | Local variable name 'r' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 508 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 517 |
| Warning |
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 537 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 540 |
| Warning |
naming | LocalVariableName | Local variable name 't' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 543 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '-' is not preceded with whitespace. | 543 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '-' is not followed by whitespace. | 543 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '*' is not preceded with whitespace. | 543 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '*' is not followed by whitespace. | 543 |
| Warning |
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 565 |
| Warning |
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 566 |
| Warning |
naming | AbbreviationAsWordInName | Abbreviation in name 'HEXES' must contain no more than '4' capital letters. | 569 |
| Warning |
whitespace | OperatorWrap | '+' should be on a new line. | 574 |
org/opendaylight/netvirt/utils/mdsal/openflow/FlowUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 40 |
| Warning |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 41 |
| Warning |
coding | OverloadMethodsDeclarationOrder | Overload methods should not be split. Previous overloaded method located at line '51'. | 59 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 100 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 100 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not preceded with whitespace. | 115 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '|' is not followed by whitespace. | 115 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 156 |
org/opendaylight/netvirt/utils/mdsal/openflow/InstructionUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026.match.VlanMatchBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026.vlan.match.fields.VlanIdBuilder'. | 78 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 88 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 88 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 97 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 120 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 120 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 130 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 152 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 152 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 160 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 181 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 181 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 197 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 197 |
| Warning |
indentation | Indentation | 'for' child have incorrect indentation level 24, expected level should be 20. | 219 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'ncid' declaration and its first usage is 13, but allowed 3. Consider to make that variable as final if you still need to store its value in advance (before method calls that might do side effect on original value). | 308 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: 'if' is not followed by whitespace. | 360 |
| Warning |
indentation | Indentation | 'for rcurly' have incorrect indentation level 11, expected level should be 12. | 371 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 402 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 434 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 434 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 131). | 448 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 545 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 545 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' declaration and its first usage is 4, but allowed 3. Consider to make that variable as final if you still need to store its value in advance (before method calls that might do side effect on original value). | 554 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 581 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 581 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 607 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 607 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 642 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 642 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 651 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 673 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 673 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 684 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 712 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 712 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 741 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 741 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 759 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 759 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 768 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 790 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 790 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 799 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 821 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 821 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 830 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 852 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 852 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 861 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 883 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 883 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 892 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 914 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 914 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 924 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 946 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 946 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 954 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 976 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 976 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1004 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1004 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 1012 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'ab' 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). | 1013 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1034 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1034 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 1042 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1063 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1063 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 1071 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1091 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1091 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 1099 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1119 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1119 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 1127 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1153 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1153 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 1161 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1187 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1187 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 134). | 1194 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 1218 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 1221 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'actionList' 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). | 1243 |
| Warning |
whitespace | WhitespaceAround | WhitespaceAround: '=' is not preceded with whitespace. | 1246 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 125). | 1266 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 156). | 1292 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 1296 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1342 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 1345 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 121). | 1349 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 123). | 1353 |
org/opendaylight/netvirt/utils/mdsal/openflow/MatchUtils.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.match.rev140714.NxmNxNspKey' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.match.rev140714.nxm.nx.tun.id.grouping.NxmNxTunIdBuilder'. | 82 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.match.rev140714.nxm.nx.nsp.grouping.NxmNxNspBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.match.rev140714.nxm.nx.tun.id.grouping.NxmNxTunIdBuilder'. | 83 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.match.rev140714.NxmNxNsiKey' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.match.rev140714.nxm.nx.tun.id.grouping.NxmNxTunIdBuilder'. | 84 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.match.rev140714.nxm.nx.nsi.grouping.NxmNxNsiBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.match.rev140714.nxm.nx.tun.id.grouping.NxmNxTunIdBuilder'. | 85 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.match.rev140714.nxm.of.tcp.dst.grouping.NxmOfTcpDstBuilder' import. Should be before 'org.opendaylight.yang.gen.v1.urn.opendaylight.openflowplugin.extension.nicira.match.rev140714.nxm.of.tcp.src.grouping.NxmOfTcpSrcBuilder'. | 87 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.ImmutableList' import. Should be before 'org.slf4j.LoggerFactory'. | 93 |
| Warning |
imports | CustomImportOrder | Wrong lexicographical order for 'com.google.common.collect.Lists' import. Should be before 'org.slf4j.LoggerFactory'. | 94 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 125 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 125 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 154 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 154 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 193 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 193 |
| Warning |
naming | ParameterName | Parameter name 'sMacAddr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 200 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 211 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 211 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 229 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 229 |
| Warning |
naming | ParameterName | Parameter name 'dMacAddr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 236 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 249 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 249 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 140). | 258 |
| Warning |
naming | ParameterName | Parameter name 'sMacAddr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 258 |
| Warning |
naming | ParameterName | Parameter name 'dMacAddr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 258 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 275 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 275 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 291 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 291 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 319 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 319 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 345 |
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 345 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 367 |
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 367 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 404 |
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 404 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 425 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 425 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 452 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 452 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 478 |
| Warning |
javadoc | JavadocParagraph | <p> tag should be preceded with an empty line. | 485 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 486 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 487 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 488 |
| Warning |
javadoc | JavadocTagContinuationIndentation | Line continuation have incorrect indentation level, expected level should be 4. | 489 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 517 |
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 517 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 126). | 520 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 534 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 534 |
| 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). | 552 |
| 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). | 555 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 563 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 563 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 135). | 572 |
| Warning |
indentation | Indentation | '.' have incorrect indentation level 12, expected level should be 16. | 580 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 593 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 593 |
| 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). | 611 |
| 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). | 614 |
| Warning |
naming | ParameterName | Parameter name 'dMacAddr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 872 |
| Warning |
naming | ParameterName | Parameter name 'dMacAddr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 1003 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1083 |
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 1083 |
| Warning |
naming | ParameterName | Parameter name 'dMacAddr' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 1087 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 1259 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 1284 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 124). | 1309 |
| Warning |
blocks | LeftCurly | '{' at column 5 should be on the previous line. | 1310 |
| Warning |
indentation | Indentation | '.' have incorrect indentation level 12, expected level should be 16. | 1317 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 1335 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 145). | 1335 |
| Warning |
blocks | LeftCurly | '{' at column 5 should be on the previous line. | 1336 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 1352 |
| Warning |
sizes | LineLength | Line is longer than 120 characters (found 145). | 1352 |
| Warning |
blocks | LeftCurly | '{' at column 5 should be on the previous line. | 1353 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 1367 |
| Warning |
blocks | LeftCurly | '{' at column 5 should be on the previous line. | 1368 |
| Warning |
indentation | Indentation | '.' have incorrect indentation level 12, expected level should be 16. | 1373 |
| Warning |
blocks | LeftCurly | '{' at column 5 should be on the previous line. | 1391 |
| Warning |
indentation | Indentation | '.' have incorrect indentation level 12, expected level should be 16. | 1398 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1450 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1450 |
| Warning |
javadoc | NonEmptyAtclauseDescription | At-clause should have a non-empty description. | 1453 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1464 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1464 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1512 |
| Warning |
naming | ParameterName | Parameter name 'ct_state' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 1587 |
| Warning |
indentation | Indentation | '.' have incorrect indentation level 8, expected level should be 12. | 1592 |
| Warning |
indentation | Indentation | '.' have incorrect indentation level 8, expected level should be 12. | 1593 |
| Warning |
indentation | Indentation | '.' have incorrect indentation level 8, expected level should be 12. | 1595 |
| Warning |
naming | ParameterName | Parameter name 'ct_zone' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 1603 |
| Warning |
indentation | Indentation | '.' have incorrect indentation level 8, expected level should be 12. | 1607 |
| Warning |
indentation | Indentation | '.' have incorrect indentation level 8, expected level should be 12. | 1608 |
| Warning |
indentation | Indentation | '.' have incorrect indentation level 8, expected level should be 12. | 1610 |
| Warning |
whitespace | EmptyLineSeparator | 'CTOR_DEF' should be separated from previous statement. | 1621 |
| Warning |
whitespace | EmptyLineSeparator | 'METHOD_DEF' should be separated from previous statement. | 1626 |
| Warning |
naming | LocalVariableName | Local variable name 'm' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 1666 |
| Warning |
naming | LocalVariableName | Local variable name 'm' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 1679 |
| Warning |
naming | LocalVariableName | Local variable name 'm' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 1731 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1759 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1759 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1769 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1769 |
| Warning |
javadoc | SummaryJavadoc | First sentence of Javadoc is incomplete (period is missing) or not present. | 1830 |
| Warning |
javadoc | JavadocStyle | First sentence should end with a period. | 1830 |
org/opendaylight/netvirt/utils/mdsal/openflow/MatchUtilsTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
imports | CustomImportOrder | 'org.junit.Test' should be separated from previous import group. | 11 |
| Warning |
whitespace | MethodParamPad | '(' is preceded with whitespace. | 16 |