Checkstyle Results
The following document contains the results of Checkstyle 6.16 with odl_checks.xml ruleset. 
Info
ErrorsFiles
Rules
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | RightCurly
|
1 |  Warning |
| RightCurly | 1 | Warning | |
| coding | VariableDeclarationUsageDistance | 24 | Warning |
| indentation | Indentation
|
3 | Warning |
| javadoc | JavadocStyle
|
11 | Warning |
| naming | LocalVariableName
|
4 | Warning |
| whitespace | GenericWhitespace | 1 | Warning |
Details
org/opendaylight/topoprocessing/nt/adapter/NTModelAdapter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 40 |
org/opendaylight/topoprocessing/nt/listener/NTUnderlayTopologyListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 19 |
org/opendaylight/topoprocessing/nt/listener/NTUnderlayTopologyListenerTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 49 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeYiid' 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). | 78 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeValueWithIp' declaration and its first usage is 5, 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). | 82 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'physicalNode' 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). | 96 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodePathArgument' 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). | 98 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeYiid' declaration and its first usage is 8, 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). | 153 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'physicalNode' 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). | 162 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodePathArgument' 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). | 165 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeYiid' 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). | 202 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'testNode' declaration and its first usage is 5, 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). | 207 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'physicalNode' 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). | 219 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodePathArgument' 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). | 221 |
| Warning |
whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 228 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeName' 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). | 239 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodeValue' 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). | 241 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'nodePathArgument' 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). | 251 |
org/opendaylight/topoprocessing/nt/provider/NTTopoProcessingProviderImplTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 49 |
| Warning |
naming | LocalVariableName | Local variable name 'NTProvider' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 110 |
org/opendaylight/topoprocessing/nt/request/NTTopologyRequestHandler.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 26 |
org/opendaylight/topoprocessing/nt/request/NTTopologyRequestHandlerTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
naming | LocalVariableName | Local variable name 'cBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 165 |
| Warning |
naming | LocalVariableName | Local variable name 'cBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 177 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'topoBuilder' declaration and its first usage is 9, 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). | 257 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'mappings' 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). | 260 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'topoBuilder' declaration and its first usage is 9, 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). | 289 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'mappings' 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). | 292 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'topoBuilder' declaration and its first usage is 9, 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). | 350 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'mappings' 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). | 353 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'topoBuilder' declaration and its first usage is 9, 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). | 393 |
| Warning |
naming | LocalVariableName | Local variable name 'fBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 394 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'filters' 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). | 396 |
org/opendaylight/topoprocessing/nt/request/NTTopologyRequestListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 30 |
org/opendaylight/topoprocessing/nt/request/NTTopologyRequestListenerTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 72 |
org/opendaylight/topoprocessing/nt/translator/NTLinkTranslator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 39 |
org/opendaylight/topoprocessing/nt/translator/NTNodeTranslator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 49 |
org/opendaylight/topoprocessing/nt/translator/NTOverlayItemTranslatorLinkTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 44 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'wrapperName' 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). | 81 |
org/opendaylight/topoprocessing/nt/translator/NTOverlayItemTranslatorNodeTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning |
javadoc | JavadocStyle | Javadoc has empty description section. | 44 |
| Warning |
coding | VariableDeclarationUsageDistance | Distance between variable 'wrapperName' 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). | 145 |
| Warning |
indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be one of the following: 16, 20. | 258 |
| Warning |
indentation | Indentation | 'block' child have incorrect indentation level 12, expected level should be one of the following: 16, 20. | 260 |
| Warning |
indentation | Indentation | 'block rcurly' have incorrect indentation level 8, expected level should be one of the following: 12, 16. | 262 |
| Warning |
blocks | RightCurly | '}' at column 9 should be alone on a line. | 262 |