Checkstyle Results
The following document contains the results of Checkstyle 6.16 with odl_checks.xml ruleset. 
Summary
Files |
Info |
Warnings |
Errors |
15 |
0 |
40 |
0 |
Rules
Category |
Rule |
Violations |
Severity |
coding |
IllegalCatch |
5 |
Warning |
|
VariableDeclarationUsageDistance |
1 |
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"
|
5 |
Warning |
|
NonEmptyAtclauseDescription |
3 |
Warning |
misc |
UpperEll |
1 |
Warning |
modifier |
RedundantModifier |
2 |
Warning |
naming |
LocalVariableName
- format: "^[a-z][a-z0-9][a-zA-Z0-9]*$"
- allowOneCharVarInForLoop: "true"
- tokens: "VARIABLE_DEF"
|
1 |
Warning |
|
MemberName
- format: "^[a-z][a-z0-9][a-zA-Z0-9]*$"
|
1 |
Warning |
sizes |
LineLength
- max: "120"
- ignorePattern: "^package.*|^import.*|a href|href|http://|https://|ftp://"
|
3 |
Warning |
whitespace |
OperatorWrap
- tokens: "BAND, BOR, BSR, BXOR, DIV, EQUAL, GE, GT, LAND, LE, LITERAL_INSTANCEOF, LOR, LT, MINUS, MOD, NOT_EQUAL, PLUS, QUESTION, SL, SR, STAR "
- option: "NL"
|
3 |
Warning |
|
ParenPad |
6 |
Warning |
|
WhitespaceAround
- allowEmptyConstructors: "true"
- allowEmptyMethods: "true"
- allowEmptyLoops: "true"
- allowEmptyTypes: "true"
|
2 |
Warning |
Details
Severity |
Category |
Rule |
Message |
Line |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
16 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
16 |
Severity |
Category |
Rule |
Message |
Line |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 136). |
96 |
Severity |
Category |
Rule |
Message |
Line |
Warning |
modifier |
RedundantModifier |
Redundant 'public' modifier. |
24 |
Severity |
Category |
Rule |
Message |
Line |
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 123). |
57 |
Warning |
whitespace |
OperatorWrap |
'+' should be on a new line. |
171 |
Warning |
coding |
IllegalCatch |
Catching 'Exception' is not allowed. |
252 |
Warning |
javadoc |
JavadocParagraph |
Empty line should be followed by <p> tag on the next line. |
260 |
Warning |
javadoc |
JavadocParagraph |
Empty line should be followed by <p> tag on the next line. |
262 |
Warning |
javadoc |
NonEmptyAtclauseDescription |
At-clause should have a non-empty description. |
265 |
Warning |
javadoc |
NonEmptyAtclauseDescription |
At-clause should have a non-empty description. |
266 |
Warning |
javadoc |
NonEmptyAtclauseDescription |
At-clause should have a non-empty description. |
267 |
Warning |
coding |
IllegalCatch |
Catching 'Exception' is not allowed. |
275 |
Warning |
coding |
IllegalCatch |
Catching 'Exception' is not allowed. |
311 |
Severity |
Category |
Rule |
Message |
Line |
Warning |
modifier |
RedundantModifier |
Redundant 'public' modifier. |
20 |
Severity |
Category |
Rule |
Message |
Line |
Warning |
misc |
UpperEll |
Should use uppercase 'L'. |
26 |
Severity |
Category |
Rule |
Message |
Line |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 160). |
48 |
Warning |
whitespace |
ParenPad |
'(' is followed by whitespace. |
85 |
Warning |
whitespace |
ParenPad |
')' is preceded with whitespace. |
85 |
Warning |
whitespace |
ParenPad |
'(' is followed by whitespace. |
88 |
Warning |
whitespace |
ParenPad |
')' is preceded with whitespace. |
88 |
Warning |
whitespace |
ParenPad |
'(' is followed by whitespace. |
92 |
Warning |
whitespace |
ParenPad |
')' is preceded with whitespace. |
92 |
Warning |
naming |
MemberName |
Member name 'yang_skip' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. |
93 |
Severity |
Category |
Rule |
Message |
Line |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '{' is not preceded with whitespace. |
40 |
Severity |
Category |
Rule |
Message |
Line |
Warning |
coding |
IllegalCatch |
Catching 'Exception' is not allowed. |
220 |
Warning |
whitespace |
OperatorWrap |
'+' should be on a new line. |
223 |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
263 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
263 |
Warning |
coding |
IllegalCatch |
Catching 'Exception' is not allowed. |
276 |
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 |
coding |
VariableDeclarationUsageDistance |
Distance between variable 'g' 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). |
299 |
Warning |
naming |
LocalVariableName |
Local variable name 'g' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. |
299 |
Warning |
whitespace |
OperatorWrap |
'+' should be on a new line. |
305 |
Warning |
javadoc |
SummaryJavadoc |
First sentence of Javadoc is incomplete (period is missing) or not present. |
339 |
Warning |
javadoc |
JavadocStyle |
First sentence should end with a period. |
339 |
Severity |
Category |
Rule |
Message |
Line |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '{' is not preceded with whitespace. |
33 |