Checkstyle Results
The following document contains the results of Checkstyle 6.2 with odl_checks.xml ruleset. 
Summary
| Files |
Info |
Warnings |
Errors |
| 7 |
0 |
40 |
0 |
Rules
| Category |
Rule |
Violations |
Severity |
| imports |
CustomImportOrder
- sortImportsInGroupAlphabetically: "true"
- customImportOrderRules: "STATIC###STANDARD_JAVA_PACKAGE###THIRD_PARTY_PACKAGE###SPECIAL_IMPORTS"
- thirdPartyPackageRegExp: ".*"
- specialImportsRegExp: "^com.*"
|
7 |
Warning |
| indentation |
Indentation
- lineWrappingIndentation: "4"
- caseIndent: "4"
- arrayInitIndent: "4"
- throwsIndent: "4"
- basicOffset: "4"
- braceAdjustment: "0"
|
11 |
Warning |
| javadoc |
JavadocMethod
- allowMissingParamTags: "true"
- allowedAnnotations: "Override, Test"
- allowThrowsTagsForSubclasses: "true"
- scope: "public"
- allowMissingReturnTag: "true"
- minLineCount: "2"
- allowMissingThrowsTags: "true"
|
2 |
Warning |
| naming |
AbbreviationAsWordInName
- allowedAbbreviationLength: "1"
- ignoreFinal: "false"
|
7 |
Warning |
| sizes |
LineLength
- max: "120"
- ignorePattern: "^package.*|^import.*|a href|href|http://|https://|ftp://"
|
1 |
Warning |
| whitespace |
EmptyLineSeparator
- allowNoEmptyLineBetweenFields: "true"
|
8 |
Warning |
|
GenericWhitespace |
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"
|
1 |
Warning |
|
WhitespaceAround
- allowEmptyLoops: "true"
- allowEmptyTypes: "true"
- allowEmptyMethods: "true"
- allowEmptyConstructors: "true"
|
2 |
Warning |
Details
org/opendaylight/topoprocessing/nt/adapter/NTModelAdapter.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
whitespace |
EmptyLineSeparator |
'package' should be separated from previous statement. |
8 |
Warning |
naming |
AbbreviationAsWordInName |
Abbreviation in name must contain no more than '1' capital letters. |
44 |
Warning |
sizes |
LineLength |
Line is longer than 120 characters (found 126). |
52 |
Warning |
indentation |
Indentation |
'case' child have incorrect indentation level 8, expected level should be 12. |
77 |
Warning |
indentation |
Indentation |
'case' child have incorrect indentation level 8, expected level should be 12. |
78 |
Warning |
indentation |
Indentation |
'block' child have incorrect indentation level 12, expected level should be 16. |
79 |
Warning |
indentation |
Indentation |
'method call' child have incorrect indentation level 12, expected level should be 16. |
79 |
Warning |
indentation |
Indentation |
'block' child have incorrect indentation level 12, expected level should be 16. |
80 |
Warning |
indentation |
Indentation |
'case' child have incorrect indentation level 8, expected level should be 12. |
81 |
Warning |
indentation |
Indentation |
'block' child have incorrect indentation level 12, expected level should be 16. |
82 |
Warning |
indentation |
Indentation |
'method call' child have incorrect indentation level 12, expected level should be 16. |
82 |
Warning |
indentation |
Indentation |
'block' child have incorrect indentation level 12, expected level should be 16. |
83 |
Warning |
indentation |
Indentation |
'case' child have incorrect indentation level 8, expected level should be 12. |
84 |
Warning |
indentation |
Indentation |
'block' child have incorrect indentation level 12, expected level should be 16. |
85 |
org/opendaylight/topoprocessing/nt/listener/NTUnderlayTopologyListener.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
whitespace |
EmptyLineSeparator |
'package' should be separated from previous statement. |
8 |
Warning |
naming |
AbbreviationAsWordInName |
Abbreviation in name must contain no more than '1' capital letters. |
24 |
Warning |
javadoc |
JavadocMethod |
Missing a Javadoc comment. |
26 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: '{' is not preceded with whitespace. |
44 |
Warning |
whitespace |
EmptyLineSeparator |
';' should be separated from previous statement. |
46 |
org/opendaylight/topoprocessing/nt/provider/TopoProcessingProviderNT.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
naming |
AbbreviationAsWordInName |
Abbreviation in name must contain no more than '1' capital letters. |
17 |
Warning |
whitespace |
EmptyLineSeparator |
'METHOD_DEF' should be separated from previous statement. |
20 |
org/opendaylight/topoprocessing/nt/request/NTTopologyRequestHandler.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
whitespace |
EmptyLineSeparator |
'package' should be separated from previous statement. |
8 |
Warning |
naming |
AbbreviationAsWordInName |
Abbreviation in name must contain no more than '1' capital letters. |
30 |
org/opendaylight/topoprocessing/nt/request/NTTopologyRequestListener.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
whitespace |
EmptyLineSeparator |
'package' should be separated from previous statement. |
8 |
Warning |
imports |
CustomImportOrder |
Wrong lexicographical order for 'org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode' import. |
27 |
Warning |
naming |
AbbreviationAsWordInName |
Abbreviation in name must contain no more than '1' capital letters. |
34 |
Warning |
javadoc |
JavadocMethod |
Missing a Javadoc comment. |
40 |
Warning |
whitespace |
GenericWhitespace |
GenericWhitespace '<' is preceded with whitespace. |
57 |
org/opendaylight/topoprocessing/nt/translator/NTLinkTranslator.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
whitespace |
EmptyLineSeparator |
'package' should be separated from previous statement. |
8 |
Warning |
imports |
CustomImportOrder |
Wrong lexicographical order for 'org.opendaylight.yangtools.yang.data.api.schema.ContainerNode' import. |
27 |
Warning |
naming |
AbbreviationAsWordInName |
Abbreviation in name must contain no more than '1' capital letters. |
43 |
Warning |
whitespace |
WhitespaceAround |
WhitespaceAround: 'if' is not followed by whitespace. |
76 |
org/opendaylight/topoprocessing/nt/translator/NTNodeTranslator.java
| Severity |
Category |
Rule |
Message |
Line |
Warning |
whitespace |
EmptyLineSeparator |
'package' should be separated from previous statement. |
8 |
Warning |
imports |
CustomImportOrder |
Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. |
11 |
Warning |
imports |
CustomImportOrder |
Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. |
12 |
Warning |
imports |
CustomImportOrder |
Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. |
13 |
Warning |
imports |
CustomImportOrder |
Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. |
14 |
Warning |
imports |
CustomImportOrder |
Import statement is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group. |
15 |
Warning |
naming |
AbbreviationAsWordInName |
Abbreviation in name must contain no more than '1' capital letters. |
55 |
Warning |
whitespace |
OperatorWrap |
'&&' should be on a new line. |
93 |