Checkstyle Results

The following document contains the results of Checkstyle 6.16 with odl_checks.xml ruleset. rss feed

Summary

Files  Info  Warnings  Errors
2 0 46 0

Rules

Category Rule Violations Severity
coding IllegalCatch 1  Warning
OverloadMethodsDeclarationOrder 4  Warning
VariableDeclarationUsageDistance 2  Warning
imports CustomImportOrder
  • thirdPartyPackageRegExp: ".*"
  • sortImportsInGroupAlphabetically: "true"
  • customImportOrderRules: "STATIC###THIRD_PARTY_PACKAGE"
2  Warning
javadoc JavadocStyle
  • checkEmptyJavadoc: "true"
3  Warning
NonEmptyAtclauseDescription 5  Warning
sizes LineLength
  • max: "120"
  • ignorePattern: "^package.*|^import.*|a href|href|http://|https://|ftp://"
1  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"
25  Warning

Details

org/opendaylight/ovsdb/utils/southbound/utils/SouthboundUtils.java

Severity Category Rule Message Line
 Warning imports CustomImportOrder Wrong lexicographical order for 'com.google.common.collect.ImmutableBiMap' import. Should be before 'org.slf4j.LoggerFactory'. 106
 Warning coding OverloadMethodsDeclarationOrder Overload methods should not be split. Previous overloaded method located at line '189'. 198
 Warning whitespace WhitespaceAround WhitespaceAround: '{' is not preceded with whitespace. 219
 Warning coding OverloadMethodsDeclarationOrder Overload methods should not be split. Previous overloaded method located at line '193'. 235
 Warning coding OverloadMethodsDeclarationOrder Overload methods should not be split. Previous overloaded method located at line '249'. 281
 Warning javadoc JavadocStyle First sentence should end with a period. 397
 Warning whitespace WhitespaceAround WhitespaceAround: 'else' is not preceded with whitespace. 469
 Warning whitespace WhitespaceAround WhitespaceAround: '}' is not followed by whitespace. 469
 Warning whitespace WhitespaceAround WhitespaceAround: 'else' is not followed by whitespace. 469
 Warning whitespace WhitespaceAround WhitespaceAround: '{' is not preceded with whitespace. 469
 Warning javadoc SummaryJavadoc First sentence of Javadoc is incomplete (period is missing) or not present. 583
 Warning javadoc JavadocStyle First sentence should end with a period. 583
 Warning javadoc SummaryJavadoc First sentence of Javadoc is incomplete (period is missing) or not present. 593
 Warning javadoc JavadocStyle First sentence should end with a period. 593
 Warning whitespace EmptyLineSeparator 'METHOD_DEF' should be separated from previous statement. 602
 Warning whitespace WhitespaceAround WhitespaceAround: 'if' is not followed by whitespace. 619
 Warning whitespace WhitespaceAround WhitespaceAround: 'if' is not followed by whitespace. 633
 Warning coding OverloadMethodsDeclarationOrder Overload methods should not be split. Previous overloaded method located at line '528'. 646
 Warning coding VariableDeclarationUsageDistance Distance between variable 'tpIid' 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). 715
 Warning coding VariableDeclarationUsageDistance Distance between variable 'tpIid' 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). 761
 Warning whitespace WhitespaceAround WhitespaceAround: 'try' is not followed by whitespace. 916
 Warning whitespace WhitespaceAround WhitespaceAround: '{' is not preceded with whitespace. 916
 Warning sizes LineLength Line is longer than 120 characters (found 124). 917
 Warning whitespace WhitespaceAround WhitespaceAround: '{' is not preceded with whitespace. 917
 Warning coding IllegalCatch Catching 'Exception' is not allowed. 928
 Warning whitespace WhitespaceAround WhitespaceAround: 'catch' is not preceded with whitespace. 928
 Warning whitespace WhitespaceAround WhitespaceAround: '}' is not followed by whitespace. 928
 Warning whitespace WhitespaceAround WhitespaceAround: '{' is not preceded with whitespace. 928
 Warning whitespace WhitespaceAround WhitespaceAround: 'if' is not followed by whitespace. 1061
 Warning whitespace WhitespaceAround WhitespaceAround: '{' is not preceded with whitespace. 1061
 Warning whitespace WhitespaceAround WhitespaceAround: 'for' is not followed by whitespace. 1062
 Warning whitespace WhitespaceAround WhitespaceAround: '{' is not preceded with whitespace. 1062
 Warning javadoc NonEmptyAtclauseDescription At-clause should have a non-empty description. 1077
 Warning javadoc NonEmptyAtclauseDescription At-clause should have a non-empty description. 1078
 Warning javadoc NonEmptyAtclauseDescription At-clause should have a non-empty description. 1079
 Warning whitespace WhitespaceAround WhitespaceAround: 'if' is not followed by whitespace. 1083
 Warning whitespace WhitespaceAround WhitespaceAround: '{' is not preceded with whitespace. 1083
 Warning whitespace WhitespaceAround WhitespaceAround: 'if' is not followed by whitespace. 1085
 Warning whitespace WhitespaceAround WhitespaceAround: '{' is not preceded with whitespace. 1085
 Warning whitespace WhitespaceAround WhitespaceAround: 'for' is not followed by whitespace. 1086
 Warning whitespace WhitespaceAround WhitespaceAround: '{' is not preceded with whitespace. 1086
 Warning whitespace WhitespaceAround WhitespaceAround: 'if' is not followed by whitespace. 1087
 Warning whitespace WhitespaceAround WhitespaceAround: '{' is not preceded with whitespace. 1087
 Warning javadoc NonEmptyAtclauseDescription At-clause should have a non-empty description. 1099
 Warning javadoc NonEmptyAtclauseDescription At-clause should have a non-empty description. 1100

org/opendaylight/ovsdb/utils/southbound/utils/SouthboundUtilsTest.java

Severity Category Rule Message Line
 Warning imports CustomImportOrder 'org.junit.Test' should be separated from previous import group. 12