Checkstyle Results
The following document contains the results of Checkstyle 7.6.1 with odl_checks.xml ruleset. 
Summary
| Files |
Info |
Warnings |
Errors |
| 14 |
0 |
17 |
0 |
Rules
| Category |
Rule |
Violations |
Severity |
| coding |
IllegalCatch |
1 |
Warning |
| naming |
ParameterName
- format: "^[a-z][a-z0-9][a-zA-Z0-9]*$"
|
1 |
Warning |
| regexp |
RegexpSinglelineJava
- ignoreComments: "false"
- format: "\.printStackTrace?\("
- message: "Line contains printStacktrace()."
|
1 |
Warning |
|
RegexpSinglelineJava
- ignoreComments: "false"
- format: "System\.(out)|(err)\.print(ln)?\("
- message: "Line contains console output."
|
3 |
Warning |
| sizes |
LineLength
- max: "120"
- ignorePattern: "^package.*|^import.*|a href|href|http://|https://|ftp://"
|
5 |
Warning |
| whitespace |
EmptyLineSeparator
- allowNoEmptyLineBetweenFields: "true"
- tokens: "IMPORT, CLASS_DEF, INTERFACE_DEF, ENUM_DEF, STATIC_INIT, INSTANCE_INIT, METHOD_DEF, CTOR_DEF, VARIABLE_DEF"
|
1 |
Warning |
|
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"
|
2 |
Warning |
|
WhitespaceAround
- allowEmptyConstructors: "true"
- allowEmptyMethods: "true"
- allowEmptyLoops: "true"
- allowEmptyTypes: "true"
|
2 |
Warning |
Details
| Severity |
Category |
Rule |
Message |
Line |
Warning |
whitespace |
EmptyLineSeparator |
'METHOD_DEF' should be separated from previous statement. |
19 |
Warning |
coding |
IllegalCatch |
Catching 'Exception' is not allowed. |
22 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: 'catch' is not followed by whitespace. |
22 |
Warning |
regexp |
RegexpSinglelineJava |
Line contains printStacktrace(). |
23 |
Warning |
regexp |
RegexpSinglelineJava |
Line contains console output. |
24 |
Warning |
regexp |
RegexpSinglelineJava |
Line contains console output. |
25 |
Warning |
naming |
ParameterName |
Parameter name 'e' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. |
34 |
Warning |
whitespace |
OperatorWrap |
'+' should be on a new line. |
41 |
Warning |
whitespace |
OperatorWrap |
'+' should be on a new line. |
42 |
| Severity |
Category |
Rule |
Message |
Line |
Warning |
extension |
ForbidCertainImports |
Use 'java.nio.charset.StandardCharsets' instead of 'com.google.common.base.Charsets' |
14 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 140). |
78 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 152). |
79 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 138). |
82 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 212). |
83 |
Warning |
regexp |
RegexpSinglelineJava |
Line contains console output. |
87 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: 'for' is not followed by whitespace. |
92 |
| Severity |
Category |
Rule |
Message |
Line |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 124). |
25 |