The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
coding | AvoidInlineConditionals | 1 | Error |
HiddenField | 3 | Error | |
MagicNumber | 1 | Error | |
design | DesignForExtension | 5 | Error |
FinalClass | 2 | Error | |
VisibilityModifier | 1 | Error | |
javadoc | JavadocMethod | 37 | Error |
JavadocPackage | 5 | Error | |
JavadocStyle | 9 | Error | |
JavadocType | 3 | Error | |
JavadocVariable | 22 | Error | |
misc | FinalParameters | 59 | Error |
naming | ConstantName | 1 | Error |
StaticVariableName | 1 | Error | |
sizes | LineLength | 90 | Error |
whitespace | ParenPad | 14 | Error |
WhitespaceAfter | 1 | Error | |
WhitespaceAround | 9 | Error |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | javadoc | JavadocPackage | Missing package-info.java file. | |
Error | sizes | LineLength | Line is longer than 80 characters (found 87). | 34 |
Error | sizes | LineLength | Line is longer than 80 characters (found 126). | 37 |
Error | sizes | LineLength | Line is longer than 80 characters (found 119). | 40 |
Error | sizes | LineLength | Line is longer than 80 characters (found 83). | 43 |
Error | sizes | LineLength | Line is longer than 80 characters (found 127). | 46 |
Error | javadoc | JavadocType | Type Javadoc comment is missing an @param <T> tag. | 49 |
Error | sizes | LineLength | Line is longer than 80 characters (found 97). | 54 |
Error | javadoc | JavadocStyle | First sentence should end with a period. | 58 |
Error | sizes | LineLength | Line is longer than 80 characters (found 86). | 61 |
Error | javadoc | JavadocMethod | Expected @throws tag for 'ActivityConversionException'. | 71 |
Error | sizes | LineLength | Line is longer than 80 characters (found 83). | 78 |
Error | javadoc | JavadocMethod | Expected @throws tag for 'ActivityConversionException'. | 78 |
Error | sizes | LineLength | Line is longer than 80 characters (found 81). | 85 |
Error | javadoc | JavadocMethod | Expected @throws tag for 'ActivityConversionException'. | 85 |
Error | sizes | LineLength | Line is longer than 80 characters (found 81). | 92 |
Error | javadoc | JavadocMethod | Expected @throws tag for 'ActivityConversionException'. | 92 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | javadoc | JavadocType | Type Javadoc comment is missing an @param <T> tag. | 32 |
Error | sizes | LineLength | Line is longer than 80 characters (found 97). | 37 |
Error | sizes | LineLength | Line is longer than 80 characters (found 86). | 44 |
Error | sizes | LineLength | Line is longer than 80 characters (found 87). | 54 |
Error | javadoc | JavadocMethod | Expected @throws tag for 'ActivityConversionException'. | 54 |
Error | sizes | LineLength | Line is longer than 80 characters (found 83). | 61 |
Error | javadoc | JavadocMethod | Expected @throws tag for 'ActivityConversionException'. | 61 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | javadoc | JavadocStyle | First sentence should end with a period. | 26 |
Error | sizes | LineLength | Line is longer than 80 characters (found 90). | 30 |
Error | javadoc | JavadocType | Type Javadoc comment is missing an @param <T> tag. | 32 |
Error | sizes | LineLength | Line is longer than 80 characters (found 86). | 38 |
Error | javadoc | JavadocMethod | Expected @throws tag for 'ActivitySerializerException'. | 48 |
Error | javadoc | JavadocMethod | Expected @throws tag for 'ActivitySerializerException'. | 55 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | sizes | LineLength | Line is longer than 80 characters (found 95). | 25 |
Error | sizes | LineLength | Line is longer than 80 characters (found 88). | 31 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | javadoc | JavadocStyle | First sentence should end with a period. | 30 |
Error | design | FinalClass | Class ActivityUtil should be declared as final. | 36 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 39 |
Error | whitespace | WhitespaceAround | '{' is not followed by whitespace. | 39 |
Error | whitespace | WhitespaceAround | '}' is not preceded with whitespace. | 39 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 76 |
Error | naming | ConstantName | Name 'mapper' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 76 |
Error | misc | FinalParameters | Parameter activity should be final. | 85 |
Error | sizes | LineLength | Line is longer than 80 characters (found 101). | 86 |
Error | whitespace | WhitespaceAfter | 'typecast' is not followed by whitespace. | 86 |
Error | misc | FinalParameters | Parameter providerName should be final. | 100 |
Error | misc | FinalParameters | Parameter personId should be final. | 100 |
Error | misc | FinalParameters | Parameter providerName should be final. | 109 |
Error | sizes | LineLength | Line is longer than 80 characters (found 89). | 120 |
Error | misc | FinalParameters | Parameter provider should be final. | 120 |
Error | misc | FinalParameters | Parameter objectType should be final. | 120 |
Error | misc | FinalParameters | Parameter objectId should be final. | 120 |
Error | misc | FinalParameters | Parameter providerName should be final. | 130 |
Error | misc | FinalParameters | Parameter activityId should be final. | 130 |
Error | misc | FinalParameters | Parameter activity should be final. | 139 |
Error | misc | FinalParameters | Parameter activityObject should be final. | 152 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | javadoc | JavadocPackage | Missing package-info.java file. | |
Error | javadoc | JavadocStyle | First sentence should end with a period. | 32 |
Error | sizes | LineLength | Line is longer than 80 characters (found 105). | 33 |
Error | design | FinalClass | Class RFC3339Utils should be declared as final. | 35 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 37 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 39 |
Error | sizes | LineLength | Line is longer than 80 characters (found 98). | 43 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 43 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 44 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 45 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 46 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 49 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 50 |
Error | sizes | LineLength | Line is longer than 80 characters (found 89). | 51 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 51 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 52 |
Error | sizes | LineLength | Line is longer than 80 characters (found 90). | 53 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 53 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 55 |
Error | sizes | LineLength | Line is longer than 80 characters (found 117). | 56 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 56 |
Error | sizes | LineLength | Line is longer than 80 characters (found 123). | 57 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 57 |
Error | sizes | LineLength | Line is longer than 80 characters (found 117). | 58 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 58 |
Error | sizes | LineLength | Line is longer than 80 characters (found 123). | 59 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 59 |
Error | sizes | LineLength | Line is longer than 80 characters (found 114). | 62 |
Error | sizes | LineLength | Line is longer than 80 characters (found 94). | 67 |
Error | sizes | LineLength | Line is longer than 80 characters (found 120). | 68 |
Error | sizes | LineLength | Line is longer than 80 characters (found 103). | 69 |
Error | sizes | LineLength | Line is longer than 80 characters (found 90). | 75 |
Error | sizes | LineLength | Line is longer than 80 characters (found 85). | 77 |
Error | sizes | LineLength | Line is longer than 80 characters (found 81). | 89 |
Error | sizes | LineLength | Line is longer than 80 characters (found 82). | 90 |
Error | sizes | LineLength | Line is longer than 80 characters (found 83). | 93 |
Error | sizes | LineLength | Line is longer than 80 characters (found 83). | 94 |
Error | sizes | LineLength | Line is longer than 80 characters (found 83). | 95 |
Error | sizes | LineLength | Line is longer than 80 characters (found 84). | 96 |
Error | sizes | LineLength | Line is longer than 80 characters (found 83). | 97 |
Error | sizes | LineLength | Line is longer than 80 characters (found 83). | 98 |
Error | sizes | LineLength | Line is longer than 80 characters (found 83). | 99 |
Error | sizes | LineLength | Line is longer than 80 characters (found 83). | 100 |
Error | sizes | LineLength | Line is longer than 80 characters (found 83). | 101 |
Error | sizes | LineLength | Line is longer than 80 characters (found 83). | 114 |
Error | sizes | LineLength | Line is longer than 80 characters (found 83). | 115 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 124 |
Error | whitespace | WhitespaceAround | '{' is not followed by whitespace. | 124 |
Error | whitespace | WhitespaceAround | '}' is not preceded with whitespace. | 124 |
Error | javadoc | JavadocStyle | First sentence should end with a period. | 126 |
Error | misc | FinalParameters | Parameter toParse should be final. | 131 |
Error | sizes | LineLength | Line is longer than 80 characters (found 126). | 149 |
Error | sizes | LineLength | Line is longer than 80 characters (found 81). | 152 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 152 |
Error | misc | FinalParameters | Parameter formatter should be final. | 152 |
Error | misc | FinalParameters | Parameter toParse should be final. | 152 |
Error | sizes | LineLength | Line is longer than 80 characters (found 115). | 157 |
Error | sizes | LineLength | Line is longer than 80 characters (found 112). | 158 |
Error | sizes | LineLength | Line is longer than 80 characters (found 108). | 159 |
Error | misc | FinalParameters | Parameter dateString should be final. | 163 |
Error | misc | FinalParameters | Parameter dateString should be final. | 179 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 183 |
Error | misc | FinalParameters | Parameter toFormat should be final. | 183 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 187 |
Error | misc | FinalParameters | Parameter toFormat should be final. | 187 |
Error | misc | FinalParameters | Parameter tz should be final. | 187 |
Error | sizes | LineLength | Line is longer than 80 characters (found 99). | 188 |
Error | sizes | LineLength | Line is longer than 80 characters (found 82). | 191 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 191 |
Error | misc | FinalParameters | Parameter sub should be final. | 191 |
Error | misc | FinalParameters | Parameter suffix should be final. | 191 |
Error | sizes | LineLength | Line is longer than 80 characters (found 114). | 193 |
Error | coding | MagicNumber | '3' is a magic number. | 195 |
Error | coding | AvoidInlineConditionals | Avoid inline conditionals. | 196 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | javadoc | JavadocPackage | Missing package-info.java file. | |
Error | sizes | LineLength | Line is longer than 80 characters (found 85). | 22 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 27 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 30 |
Error | misc | FinalParameters | Parameter message should be final. | 30 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 34 |
Error | misc | FinalParameters | Parameter cause should be final. | 34 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 38 |
Error | misc | FinalParameters | Parameter message should be final. | 38 |
Error | misc | FinalParameters | Parameter cause should be final. | 38 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | sizes | LineLength | Line is longer than 80 characters (found 85). | 22 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 27 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 31 |
Error | misc | FinalParameters | Parameter message should be final. | 31 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 36 |
Error | misc | FinalParameters | Parameter cause should be final. | 36 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 41 |
Error | misc | FinalParameters | Parameter message should be final. | 41 |
Error | misc | FinalParameters | Parameter cause should be final. | 41 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | javadoc | JavadocStyle | First sentence should end with a period. | 38 |
Error | sizes | LineLength | Line is longer than 80 characters (found 84). | 43 |
Error | sizes | LineLength | Line is longer than 80 characters (found 100). | 46 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 48 |
Error | sizes | LineLength | Line is longer than 80 characters (found 98). | 50 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 50 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 52 |
Error | misc | FinalParameters | Parameter dateTimeClass should be final. | 52 |
Error | sizes | LineLength | Line is longer than 80 characters (found 94). | 56 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 56 |
Error | misc | FinalParameters | Parameter dateTimeClass should be final. | 56 |
Error | misc | FinalParameters | Parameter formats should be final. | 56 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 58 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 58 |
Error | javadoc | JavadocStyle | First sentence should end with a period. | 67 |
Error | sizes | LineLength | Line is longer than 80 characters (found 85). | 68 |
Error | sizes | LineLength | Line is longer than 80 characters (found 99). | 71 |
Error | misc | FinalParameters | Parameter jpar should be final. | 71 |
Error | misc | FinalParameters | Parameter context should be final. | 71 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 75 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 29 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | javadoc | JavadocStyle | First sentence should end with a period. | 31 |
Error | sizes | LineLength | Line is longer than 80 characters (found 96). | 35 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 37 |
Error | misc | FinalParameters | Parameter dateTimeClass should be final. | 37 |
Error | design | DesignForExtension | Class 'StreamsDateTimeSerializer' looks like designed for extension (can be subclassed), but the method 'serialize' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsDateTimeSerializer' final or making the method 'serialize' static/final/abstract/empty, or adding allowed annotation for the method. | 41 |
Error | sizes | LineLength | Line is longer than 80 characters (found 109). | 42 |
Error | misc | FinalParameters | Parameter value should be final. | 42 |
Error | misc | FinalParameters | Parameter jgen should be final. | 42 |
Error | misc | FinalParameters | Parameter provider should be final. | 42 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | sizes | LineLength | Line is longer than 80 characters (found 90). | 38 |
Error | sizes | LineLength | Line is longer than 80 characters (found 88). | 41 |
Error | sizes | LineLength | Line is longer than 80 characters (found 86). | 44 |
Error | sizes | LineLength | Line is longer than 80 characters (found 119). | 48 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 48 |
Error | naming | StaticVariableName | Name 'INSTANCE_MAP' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 48 |
Error | sizes | LineLength | Line is longer than 80 characters (found 100). | 50 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 50 |
Error | sizes | LineLength | Line is longer than 80 characters (found 99). | 65 |
Error | misc | FinalParameters | Parameter configuration should be final. | 65 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 66 |
Error | misc | FinalParameters | Parameter format should be final. | 82 |
Error | misc | FinalParameters | Parameter formats should be final. | 94 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 106 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 109 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 115 |
Error | misc | FinalParameters | Parameter format should be final. | 116 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 119 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 125 |
Error | misc | FinalParameters | Parameter formats should be final. | 126 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 129 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 135 |
Error | misc | FinalParameters | Parameter configuration should be final. | 135 |
Error | coding | HiddenField | 'configuration' hides a field. | 135 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 138 |
Error | design | DesignForExtension | Class 'StreamsJacksonMapper' looks like designed for extension (can be subclassed), but the method 'configure' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsJacksonMapper' final or making the method 'configure' static/final/abstract/empty, or adding allowed annotation for the method. | 144 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 144 |
Error | sizes | LineLength | Line is longer than 80 characters (found 91). | 145 |
Error | sizes | LineLength | Line is longer than 80 characters (found 81). | 148 |
Error | sizes | LineLength | Line is longer than 80 characters (found 87). | 149 |
Error | sizes | LineLength | Line is longer than 80 characters (found 111). | 152 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | javadoc | JavadocPackage | Missing package-info.java file. | |
Error | javadoc | JavadocStyle | First sentence should end with a period. | 35 |
Error | sizes | LineLength | Line is longer than 80 characters (found 91). | 44 |
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 44 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 46 |
Error | sizes | LineLength | Line is longer than 80 characters (found 127). | 53 |
Error | sizes | LineLength | Line is longer than 80 characters (found 103). | 58 |
Error | sizes | LineLength | Line is longer than 80 characters (found 81). | 64 |
Error | sizes | LineLength | Line is longer than 80 characters (found 102). | 65 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 71 |
Error | misc | FinalParameters | Parameter formats should be final. | 71 |
Error | sizes | LineLength | Line is longer than 80 characters (found 81). | 74 |
Error | sizes | LineLength | Line is longer than 80 characters (found 94). | 75 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | sizes | LineLength | Line is longer than 80 characters (found 96). | 32 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 34 |
Error | misc | FinalParameters | Parameter dateTimeClass should be final. | 34 |
Error | sizes | LineLength | Line is longer than 80 characters (found 97). | 38 |
Error | design | DesignForExtension | Class 'StreamsPeriodDeserializer' looks like designed for extension (can be subclassed), but the method 'deserialize' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsPeriodDeserializer' final or making the method 'deserialize' static/final/abstract/empty, or adding allowed annotation for the method. | 38 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 38 |
Error | misc | FinalParameters | Parameter jpar should be final. | 38 |
Error | misc | FinalParameters | Parameter context should be final. | 38 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | sizes | LineLength | Line is longer than 80 characters (found 92). | 32 |
Error | javadoc | JavadocMethod | Missing a Javadoc comment. | 34 |
Error | misc | FinalParameters | Parameter dateTimeClass should be final. | 34 |
Error | design | DesignForExtension | Class 'StreamsPeriodSerializer' looks like designed for extension (can be subclassed), but the method 'serialize' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsPeriodSerializer' final or making the method 'serialize' static/final/abstract/empty, or adding allowed annotation for the method. | 38 |
Error | sizes | LineLength | Line is longer than 80 characters (found 107). | 39 |
Error | misc | FinalParameters | Parameter value should be final. | 39 |
Error | misc | FinalParameters | Parameter jgen should be final. | 39 |
Error | misc | FinalParameters | Parameter provider should be final. | 39 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | javadoc | JavadocVariable | Missing a Javadoc comment. | 36 |
Error | design | VisibilityModifier | Variable 'dateFormatter' must be private and have accessor methods. | 36 |
Error | misc | FinalParameters | Parameter session should be final. | 46 |
Error | misc | FinalParameters | Parameter o should be final. | 46 |
Error | coding | HiddenField | 'dateFormatter' hides a field. | 47 |
Error | whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 48 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 48 |
Error | sizes | LineLength | Line is longer than 80 characters (found 85). | 49 |
Error | sizes | LineLength | Line is longer than 80 characters (found 98). | 55 |
Error | misc | FinalParameters | Parameter session should be final. | 55 |
Error | misc | FinalParameters | Parameter f should be final. | 55 |
Error | misc | FinalParameters | Parameter hint should be final. | 55 |
Error | coding | HiddenField | 'dateFormatter' hides a field. | 56 |
Error | whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 57 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 57 |
Error | sizes | LineLength | Line is longer than 80 characters (found 85). | 58 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Error | javadoc | JavadocPackage | Missing package-info.java file. | |
Error | sizes | LineLength | Line is longer than 80 characters (found 87). | 28 |
Error | design | DesignForExtension | Class 'ListSerializer' looks like designed for extension (can be subclassed), but the method 'serialize' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'ListSerializer' final or making the method 'serialize' static/final/abstract/empty, or adding allowed annotation for the method. | 32 |
Error | misc | FinalParameters | Parameter type should be final. | 33 |
Error | misc | FinalParameters | Parameter value should be final. | 33 |
Error | whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 35 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 35 |
Error | whitespace | ParenPad | ')' is preceded with whitespace. | 35 |
Error | whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 36 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 36 |
Error | whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 38 |
Error | whitespace | ParenPad | '(' is followed by whitespace. | 38 |