Checkstyle Results
The following document contains the results of Checkstyle 7.6.1 with odl_checks.xml ruleset. 
Summary
Files |
Info |
Warnings |
Errors |
5 |
0 |
26 |
0 |
Rules
Category |
Rule |
Violations |
Severity |
coding |
VariableDeclarationUsageDistance |
4 |
Warning |
modifier |
RedundantModifier |
1 |
Warning |
sizes |
LineLength
- max: "120"
- ignorePattern: "^package.*|^import.*|a href|href|http://|https://|ftp://"
|
19 |
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 |
|
WhitespaceAround
- allowEmptyConstructors: "true"
- allowEmptyMethods: "true"
- allowEmptyLoops: "true"
- allowEmptyTypes: "true"
|
1 |
Warning |
Details
org/opendaylight/protocol/bgp/cli/AbstractStatsCommandProvider.java
Severity |
Category |
Rule |
Message |
Line |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 135). |
25 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 147). |
36 |
Warning |
modifier |
RedundantModifier |
Redundant 'final' modifier. |
42 |
org/opendaylight/protocol/bgp/cli/utils/BgpCliUtils.java
Severity |
Category |
Rule |
Message |
Line |
Warning |
whitespace |
EmptyLineSeparator |
'METHOD_DEF' should be separated from previous statement. |
27 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 134). |
27 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 128). |
44 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 136). |
45 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 135). |
46 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 143). |
47 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 127). |
48 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 135). |
49 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 129). |
50 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 137). |
51 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 124). |
57 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 146). |
59 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 132). |
61 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 134). |
64 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 130). |
70 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 152). |
72 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 138). |
74 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 140). |
77 |
org/opendaylight/protocol/bgp/cli/utils/BgpCliUtilsTests.java
Severity |
Category |
Rule |
Message |
Line |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '{' is not preceded with whitespace. |
31 |
Warning |
coding |
VariableDeclarationUsageDistance |
Distance between variable 'objectName' declaration and its first usage is 10, 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). |
36 |
Warning |
coding |
VariableDeclarationUsageDistance |
Distance between variable 'printStream' declaration and its first usage is 10, 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). |
38 |
Warning |
coding |
VariableDeclarationUsageDistance |
Distance between variable 'peerPref' 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). |
41 |
Warning |
coding |
VariableDeclarationUsageDistance |
Distance between variable 'errorMsgs' 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). |
42 |