The following document contains the results of Checkstyle 7.6.1 with odl_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | IllegalCatch | 2 | |
| VariableDeclarationUsageDistance | 8 | ||
| indentation | Indentation
|
2 | |
| modifier | RedundantModifier | 27 | |
| naming | LocalVariableName
|
28 | |
| sizes | LineLength
|
4 | |
| whitespace | GenericWhitespace | 16 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| indentation | Indentation | 'if' child have incorrect indentation level 11, expected level should be 12. | 126 | |
| indentation | Indentation | 'else' child have incorrect indentation level 11, expected level should be 12. | 128 | |
| naming | LocalVariableName | Local variable name 'r' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 166 | |
| naming | LocalVariableName | Local variable name 'm' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 167 | |
| naming | LocalVariableName | Local variable name 'r' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 181 | |
| naming | LocalVariableName | Local variable name 'm' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 182 | |
| naming | LocalVariableName | Local variable name 'r' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 196 | |
| naming | LocalVariableName | Local variable name 'm' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 197 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'final' modifier. | 22 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 23 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 23 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 24 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 26 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 26 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 27 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 27 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 29 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 29 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 30 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 30 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 30 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 32 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 33 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 33 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 34 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 36 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 37 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 37 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 39 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 39 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 40 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 42 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 42 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 43 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 43 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | VariableDeclarationUsageDistance | Distance between variable 'headTpId' 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). | 305 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 354 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 363 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 365 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 371 | |
| naming | LocalVariableName | Local variable name 'aLink' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 405 | |
| naming | LocalVariableName | Local variable name 'aLink' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 462 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 466 | |
| naming | LocalVariableName | Local variable name 'iErrorBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 608 | |
| naming | LocalVariableName | Local variable name 'lAttribute' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 623 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 642 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 648 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 657 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 663 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 668 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 737 | |
| coding | IllegalCatch | Catching 'Exception' is not allowed. | 739 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 744 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 752 | |
| coding | VariableDeclarationUsageDistance | Distance between variable 'okBuilder' declaration and its first usage is 11, 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). | 766 | |
| naming | LocalVariableName | Local variable name 'iErrorBuilder' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 770 | |
| naming | LocalVariableName | Local variable name 'aLinkToExclude' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 777 | |
| coding | VariableDeclarationUsageDistance | Distance between variable 'directionalityInfo' 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). | 779 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 796 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 801 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 846 | |
| whitespace | GenericWhitespace | GenericWhitespace '>' is followed by whitespace. | 853 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | LocalVariableName | Local variable name 'lUnderlayTopology' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 163 | |
| naming | LocalVariableName | Local variable name 'rTx' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 231 | |
| naming | LocalVariableName | Local variable name 'rTx' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 250 | |
| naming | LocalVariableName | Local variable name 'rTx' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 270 | |
| naming | LocalVariableName | Local variable name 'rTx' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 282 | |
| naming | LocalVariableName | Local variable name 'rTx' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 302 | |
| naming | LocalVariableName | Local variable name 'rTx' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 322 | |
| naming | LocalVariableName | Local variable name 'rTx' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 340 | |
| naming | LocalVariableName | Local variable name 'rTx' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 366 | |
| naming | LocalVariableName | Local variable name 'rTx' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 392 | |
| naming | LocalVariableName | Local variable name 'rTx' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 414 | |
| naming | LocalVariableName | Local variable name 'rTx' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 504 | |
| naming | LocalVariableName | Local variable name 'rTx' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 524 | |
| naming | LocalVariableName | Local variable name 'rTx' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 542 | |
| coding | VariableDeclarationUsageDistance | Distance between variable 'topologyIid' declaration and its first usage is 21, 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). | 551 | |
| coding | VariableDeclarationUsageDistance | Distance between variable 'nodeId2' 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). | 554 | |
| coding | VariableDeclarationUsageDistance | Distance between variable 'tpId2' 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). | 555 | |
| coding | VariableDeclarationUsageDistance | Distance between variable 'forwardingAdjAnnounceInputBuilder' declaration and its first usage is 21, 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). | 560 | |
| naming | LocalVariableName | Local variable name 'lTpId' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 576 | |
| naming | LocalVariableName | Local variable name 'lAttribute' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'. | 610 | |
| coding | VariableDeclarationUsageDistance | Distance between variable 'networkTopologyRef' 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). | 613 | |
| sizes | LineLength | Line is longer than 120 characters (found 139). | 617 | |
| sizes | LineLength | Line is longer than 120 characters (found 170). | 618 | |
| sizes | LineLength | Line is longer than 120 characters (found 138). | 619 | |
| sizes | LineLength | Line is longer than 120 characters (found 168). | 620 |