Checkstyle Results
The following document contains the results of Checkstyle 7.6.1 with odl_checks.xml ruleset. 
Summary
| Files |
Info |
Warnings |
Errors |
| 20 |
0 |
107 |
0 |
Rules
| Category |
Rule |
Violations |
Severity |
| coding |
IllegalThrows |
8 |
Warning |
| imports |
CustomImportOrder
- thirdPartyPackageRegExp: ".*"
- sortImportsInGroupAlphabetically: "true"
- customImportOrderRules: "STATIC###THIRD_PARTY_PACKAGE"
|
3 |
Warning |
| javadoc |
JavadocParagraph
- allowNewlineParagraph: "false"
|
2 |
Warning |
|
JavadocStyle
- tokens: "ANNOTATION_DEF, ANNOTATION_FIELD_DEF, CLASS_DEF, CTOR_DEF, ENUM_CONSTANT_DEF, ENUM_DEF, INTERFACE_DEF, METHOD_DEF, VARIABLE_DEF"
- checkEmptyJavadoc: "true"
|
16 |
Warning |
|
JavadocTagContinuationIndentation |
2 |
Warning |
|
NonEmptyAtclauseDescription |
8 |
Warning |
| modifier |
RedundantModifier |
1 |
Warning |
| naming |
ConstantName |
2 |
Warning |
|
LocalVariableName
- format: "^[a-z][a-z0-9][a-zA-Z0-9]*$"
- allowOneCharVarInForLoop: "true"
- tokens: "VARIABLE_DEF"
|
1 |
Warning |
|
ParameterName
- format: "^[a-z][a-z0-9][a-zA-Z0-9]*$"
|
2 |
Warning |
| sizes |
LineLength
- max: "120"
- ignorePattern: "^package.*|^import.*|a href|href|http://|https://|ftp://"
|
24 |
Warning |
| whitespace |
ParenPad |
3 |
Warning |
|
WhitespaceAround
- allowEmptyConstructors: "true"
- allowEmptyMethods: "true"
- allowEmptyLoops: "true"
- allowEmptyTypes: "true"
|
19 |
Warning |
Details
org/opendaylight/protocol/util/BitArray.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
javadoc |
JavadocTagContinuationIndentation |
Line continuation have incorrect indentation level, expected level should be 4. |
34 |
Warning |
javadoc |
JavadocTagContinuationIndentation |
Line continuation have incorrect indentation level, expected level should be 4. |
54 |
Warning |
naming |
ParameterName |
Parameter name 'b' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. |
84 |
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 |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
169 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
169 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '-' is not followed by whitespace. |
176 |
org/opendaylight/protocol/util/ByteArray.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 201). |
98 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 134). |
120 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 127). |
139 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 181). |
158 |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
162 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
162 |
Warning |
modifier |
RedundantModifier |
Redundant 'final' modifier. |
177 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 153). |
198 |
org/opendaylight/protocol/util/ByteArrayTest.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 141). |
153 |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
163 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
163 |
Warning |
naming |
LocalVariableName |
Local variable name 'b' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. |
221 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not preceded with whitespace. |
237 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not followed by whitespace. |
237 |
Warning |
coding |
IllegalThrows |
Throwing 'Throwable' is not allowed. |
238 |
org/opendaylight/protocol/util/ByteBufWriteUtilTest.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 129). |
199 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not preceded with whitespace. |
223 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not followed by whitespace. |
223 |
Warning |
coding |
IllegalThrows |
Throwing 'Throwable' is not allowed. |
224 |
org/opendaylight/protocol/util/IPAddressesAndPrefixesTest.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
imports |
CustomImportOrder |
'com.google.common.collect.Lists' should be separated from previous import group by one line. |
16 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 134). |
64 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 131). |
93 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 185). |
105 |
Warning |
whitespace |
ParenPad |
')' is preceded with whitespace. |
105 |
Warning |
whitespace |
ParenPad |
')' is preceded with whitespace. |
112 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 170). |
113 |
Warning |
whitespace |
ParenPad |
')' is preceded with whitespace. |
113 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 124). |
120 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 135). |
121 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 136). |
129 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 136). |
131 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 132). |
162 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 139). |
171 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 122). |
174 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not preceded with whitespace. |
185 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not followed by whitespace. |
185 |
Warning |
coding |
IllegalThrows |
Throwing 'Throwable' is not allowed. |
186 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not preceded with whitespace. |
196 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not followed by whitespace. |
196 |
Warning |
coding |
IllegalThrows |
Throwing 'Throwable' is not allowed. |
197 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 134). |
220 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 138). |
224 |
org/opendaylight/protocol/util/Ipv4Util.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
javadoc |
NonEmptyAtclauseDescription |
At-clause should have a non-empty description. |
79 |
Warning |
javadoc |
NonEmptyAtclauseDescription |
At-clause should have a non-empty description. |
80 |
Warning |
javadoc |
NonEmptyAtclauseDescription |
At-clause should have a non-empty description. |
179 |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
186 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
186 |
Warning |
javadoc |
NonEmptyAtclauseDescription |
At-clause should have a non-empty description. |
189 |
Warning |
javadoc |
NonEmptyAtclauseDescription |
At-clause should have a non-empty description. |
190 |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
198 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
198 |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
208 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
208 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 127). |
220 |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
223 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
223 |
Warning |
javadoc |
NonEmptyAtclauseDescription |
At-clause should have a non-empty description. |
225 |
org/opendaylight/protocol/util/Ipv6Util.java
| Severity |
Category |
Rule |
Message |
Line |
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 |
org/opendaylight/protocol/util/MplsLabelUtil.java
| Severity |
Category |
Rule |
Message |
Line |
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 |
NonEmptyAtclauseDescription |
At-clause should have a non-empty description. |
39 |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
45 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
45 |
Warning |
javadoc |
NonEmptyAtclauseDescription |
At-clause should have a non-empty description. |
48 |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
54 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
54 |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
63 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
63 |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
73 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
73 |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
82 |
Warning |
javadoc |
JavadocStyle |
Javadoc has empty description section. |
82 |
org/opendaylight/protocol/util/MplsLabelUtilTest.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
imports |
CustomImportOrder |
'io.netty.buffer.Unpooled' should be separated from previous import group by one line. |
13 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 124). |
27 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 123). |
40 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not preceded with whitespace. |
43 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not followed by whitespace. |
43 |
Warning |
coding |
IllegalThrows |
Throwing 'Throwable' is not allowed. |
44 |
org/opendaylight/protocol/util/PCEPHexDumpParser.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
javadoc |
JavadocParagraph |
Empty line should be followed by <p> tag on the next line. |
27 |
Warning |
javadoc |
JavadocParagraph |
Empty line should be followed by <p> tag on the next line. |
29 |
Warning |
naming |
ParameterName |
Parameter name 'c' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. |
53 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 136). |
68 |
org/opendaylight/protocol/util/PCEPHexDumpParserTest.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
imports |
CustomImportOrder |
'java.io.File' should be separated from previous import group by one line. |
14 |
Warning |
naming |
ConstantName |
Name 'hexDumpFileName' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. |
23 |
Warning |
naming |
ConstantName |
Name 'expectedSize' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. |
24 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not preceded with whitespace. |
46 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not followed by whitespace. |
46 |
Warning |
coding |
IllegalThrows |
Throwing 'Throwable' is not allowed. |
47 |
org/opendaylight/protocol/util/StatisticsUtilTest.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not preceded with whitespace. |
29 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not followed by whitespace. |
29 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not preceded with whitespace. |
34 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not followed by whitespace. |
34 |
Warning |
coding |
IllegalThrows |
Throwing 'Throwable' is not allowed. |
35 |
org/opendaylight/protocol/util/Values.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
29 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
29 |
org/opendaylight/protocol/util/ValuesTest.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not preceded with whitespace. |
17 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '=' is not followed by whitespace. |
17 |
Warning |
coding |
IllegalThrows |
Throwing 'Throwable' is not allowed. |
18 |