The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | LeftCurly | 1 |  Error |
| coding | HiddenField | 12 | Error |
| MagicNumber | 3 | Error | |
| SimplifyBooleanExpression | 1 | Error | |
| design | DesignForExtension | 25 | Error |
| VisibilityModifier | 7 | Error | |
| javadoc | JavadocMethod | 29 | Error |
| JavadocPackage | 2 | Error | |
| JavadocType | 2 | Error | |
| JavadocVariable | 19 | Error | |
| misc | FinalParameters | 34 | Error |
| modifier | ModifierOrder | 3 | Error |
| regexp | RegexpSingleline
|
4 | Error |
| sizes | LineLength | 22 | Error |
| whitespace | ParenPad | 4 | Error |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocPackage | Missing package-info.java file. | |
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 19 |
| Error |
blocks | LeftCurly | '{' at column 1 should be on the previous line. | 26 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 28 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 31 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 34 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 85). | 35 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 38 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 49 |
| Error |
misc | FinalParameters | Parameter userName should be final. | 55 |
| Error |
coding | HiddenField | 'userName' hides a field. | 55 |
| Error |
design | DesignForExtension | Class 'GMailConfiguration' looks like designed for extension (can be subclassed), but the method 'withUserName' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailConfiguration' final or making the method 'withUserName' static/final/abstract/empty, or adding allowed annotation for the method. | 59 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 59 |
| Error |
misc | FinalParameters | Parameter userName should be final. | 59 |
| Error |
coding | HiddenField | 'userName' hides a field. | 59 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 65 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 76 |
| Error |
misc | FinalParameters | Parameter password should be final. | 82 |
| Error |
coding | HiddenField | 'password' hides a field. | 82 |
| Error |
design | DesignForExtension | Class 'GMailConfiguration' looks like designed for extension (can be subclassed), but the method 'withPassword' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailConfiguration' final or making the method 'withPassword' static/final/abstract/empty, or adding allowed annotation for the method. | 86 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 86 |
| Error |
misc | FinalParameters | Parameter password should be final. | 86 |
| Error |
coding | HiddenField | 'password' hides a field. | 86 |
| Error |
design | DesignForExtension | Class 'GMailConfiguration' looks like designed for extension (can be subclassed), but the method 'toString' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailConfiguration' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method. | 91 |
| Error |
design | DesignForExtension | Class 'GMailConfiguration' looks like designed for extension (can be subclassed), but the method 'getAdditionalProperties' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailConfiguration' final or making the method 'getAdditionalProperties' static/final/abstract/empty, or adding allowed annotation for the method. | 96 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 96 |
| Error |
design | DesignForExtension | Class 'GMailConfiguration' looks like designed for extension (can be subclassed), but the method 'setAdditionalProperty' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailConfiguration' final or making the method 'setAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 102 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 102 |
| Error |
misc | FinalParameters | Parameter name should be final. | 104 |
| Error |
misc | FinalParameters | Parameter value should be final. | 104 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 81). | 108 |
| Error |
design | DesignForExtension | Class 'GMailConfiguration' looks like designed for extension (can be subclassed), but the method 'withAdditionalProperty' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailConfiguration' final or making the method 'withAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 108 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 108 |
| Error |
misc | FinalParameters | Parameter name should be final. | 108 |
| Error |
misc | FinalParameters | Parameter value should be final. | 108 |
| Error |
design | DesignForExtension | Class 'GMailConfiguration' looks like designed for extension (can be subclassed), but the method 'hashCode' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailConfiguration' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method. | 113 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 113). | 115 |
| Error |
design | DesignForExtension | Class 'GMailConfiguration' looks like designed for extension (can be subclassed), but the method 'equals' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailConfiguration' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method. | 118 |
| Error |
misc | FinalParameters | Parameter other should be final. | 119 |
| Error |
coding | SimplifyBooleanExpression | Expression can be simplified. | 123 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 163). | 127 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocPackage | Missing package-info.java file. | |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 92). | 36 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 36 |
| Error |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 36 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 38 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 40 |
| Error |
misc | FinalParameters | Parameter provider should be final. | 40 |
| Error |
coding | HiddenField | 'provider' hides a field. | 40 |
| Error |
design | DesignForExtension | Class 'GMailImapProviderTask' looks like designed for extension (can be subclassed), but the method 'run' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailImapProviderTask' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | 44 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 85). | 47 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 102). | 52 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 52 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 52 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
sizes | LineLength | Line is longer than 80 characters (found 89). | 57 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 101). | 59 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 59 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 61 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 63 |
| Error |
misc | FinalParameters | Parameter provider should be final. | 63 |
| Error |
coding | HiddenField | 'provider' hides a field. | 63 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 78 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 81 |
| Error |
misc | FinalParameters | Parameter event should be final. | 81 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 85 |
| Error |
misc | FinalParameters | Parameter event should be final. | 85 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 89 |
| Error |
misc | FinalParameters | Parameter event should be final. | 89 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 95 |
| Error |
misc | FinalParameters | Parameter event should be final. | 95 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 99 |
| Error |
misc | FinalParameters | Parameter event should be final. | 99 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 86). | 103 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 103 |
| Error |
misc | FinalParameters | Parameter activity should be final. | 103 |
| Error |
misc | FinalParameters | Parameter gmailMessage should be final. | 103 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 92). | 104 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 108 |
| Error |
misc | FinalParameters | Parameter idparts should be final. | 108 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 123). | 110 |
| Error |
design | DesignForExtension | Class 'GMailMessageActivitySerializer' looks like designed for extension (can be subclassed), but the method 'serializationFormat' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailMessageActivitySerializer' final or making the method 'serializationFormat' static/final/abstract/empty, or adding allowed annotation for the method. | 113 |
| Error |
design | DesignForExtension | Class 'GMailMessageActivitySerializer' 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 'GMailMessageActivitySerializer' final or making the method 'serialize' static/final/abstract/empty, or adding allowed annotation for the method. | 118 |
| Error |
misc | FinalParameters | Parameter activity should be final. | 119 |
| Error |
design | DesignForExtension | Class 'GMailMessageActivitySerializer' 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 'GMailMessageActivitySerializer' final or making the method 'deserialize' static/final/abstract/empty, or adding allowed annotation for the method. | 123 |
| Error |
misc | FinalParameters | Parameter gmailMessage should be final. | 124 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 119). | 127 |
| Error |
coding | HiddenField | 'provider' hides a field. | 129 |
| Error |
design | DesignForExtension | Class 'GMailMessageActivitySerializer' looks like designed for extension (can be subclassed), but the method 'deserializeAll' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailMessageActivitySerializer' final or making the method 'deserializeAll' static/final/abstract/empty, or adding allowed annotation for the method. | 155 |
| Error |
misc | FinalParameters | Parameter serializedList should be final. | 156 |
| Error |
design | DesignForExtension | Class 'GMailMessageActivitySerializer' looks like designed for extension (can be subclassed), but the method 'convert' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailMessageActivitySerializer' final or making the method 'convert' static/final/abstract/empty, or adding allowed annotation for the method. | 160 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 160 |
| Error |
misc | FinalParameters | Parameter event should be final. | 160 |
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 164 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 165 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 169 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 173 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 181 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 186 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 58 |
| Error |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 58 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 84). | 60 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 60 |
| Error |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 60 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 61 |
| Error |
design | VisibilityModifier | Variable 'inQueue' must be private and have accessor methods. | 61 |
| Error |
coding | MagicNumber | '10000' is a magic number. | 61 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 87). | 62 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 62 |
| Error |
design | VisibilityModifier | Variable 'providerQueue' must be private and have accessor methods. | 62 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 63 |
| Error |
design | VisibilityModifier | Variable 'task' must be private and have accessor methods. | 63 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 64 |
| Error |
design | VisibilityModifier | Variable 'rssClient' must be private and have accessor methods. | 64 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 65 |
| Error |
design | VisibilityModifier | Variable 'imapClient' must be private and have accessor methods. | 65 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 66 |
| Error |
design | VisibilityModifier | Variable 'countersTotal' must be private and have accessor methods. | 66 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 67 |
| Error |
design | VisibilityModifier | Variable 'countersCurrent' must be private and have accessor methods. | 67 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 68 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 69 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 70 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 72 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 81). | 74 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 77 |
| Error |
misc | FinalParameters | Parameter config should be final. | 77 |
| Error |
coding | HiddenField | 'config' hides a field. | 77 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 81 |
| Error |
misc | FinalParameters | Parameter klass should be final. | 81 |
| Error |
coding | HiddenField | 'klass' hides a field. | 81 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 81). | 83 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 87 |
| Error |
misc | FinalParameters | Parameter config should be final. | 87 |
| Error |
coding | HiddenField | 'config' hides a field. | 87 |
| Error |
misc | FinalParameters | Parameter klass should be final. | 87 |
| Error |
coding | HiddenField | 'klass' hides a field. | 87 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 95). | 92 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 92 |
| Error |
misc | FinalParameters | Parameter nThreads should be final. | 92 |
| Error |
misc | FinalParameters | Parameter queueSize should be final. | 92 |
| Error |
coding | MagicNumber | '5000L' is a magic number. | 94 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 94). | 95 |
| Error |
design | DesignForExtension | Class 'GMailProvider' looks like designed for extension (can be subclassed), but the method 'getConfig' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailProvider' final or making the method 'getConfig' static/final/abstract/empty, or adding allowed annotation for the method. | 98 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 98 |
| Error |
design | DesignForExtension | Class 'GMailProvider' looks like designed for extension (can be subclassed), but the method 'setConfig' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailProvider' final or making the method 'setConfig' static/final/abstract/empty, or adding allowed annotation for the method. | 102 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 102 |
| Error |
misc | FinalParameters | Parameter config should be final. | 102 |
| Error |
coding | HiddenField | 'config' hides a field. | 102 |
| Error |
design | DesignForExtension | Class 'GMailProvider' looks like designed for extension (can be subclassed), but the method 'getInQueue' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailProvider' final or making the method 'getInQueue' static/final/abstract/empty, or adding allowed annotation for the method. | 106 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 106 |
| Error |
design | DesignForExtension | Class 'GMailProvider' looks like designed for extension (can be subclassed), but the method 'getId' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailProvider' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 110 |
| Error |
design | DesignForExtension | Class 'GMailProvider' looks like designed for extension (can be subclassed), but the method 'startStream' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailProvider' final or making the method 'startStream' static/final/abstract/empty, or adding allowed annotation for the method. | 115 |
| Error |
design | DesignForExtension | Class 'GMailProvider' looks like designed for extension (can be subclassed), but the method 'readCurrent' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailProvider' final or making the method 'readCurrent' static/final/abstract/empty, or adding allowed annotation for the method. | 122 |
| Error |
whitespace | ParenPad | '(' is followed by whitespace. | 127 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 127 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 81). | 128 |
| Error |
design | DesignForExtension | Class 'GMailProvider' looks like designed for extension (can be subclassed), but the method 'readNew' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailProvider' final or making the method 'readNew' static/final/abstract/empty, or adding allowed annotation for the method. | 139 |
| Error |
misc | FinalParameters | Parameter sequence should be final. | 140 |
| Error |
design | DesignForExtension | Class 'GMailProvider' looks like designed for extension (can be subclassed), but the method 'readRange' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailProvider' final or making the method 'readRange' static/final/abstract/empty, or adding allowed annotation for the method. | 144 |
| Error |
misc | FinalParameters | Parameter start should be final. | 145 |
| Error |
misc | FinalParameters | Parameter end should be final. | 145 |
| Error |
design | DesignForExtension | Class 'GMailProvider' looks like designed for extension (can be subclassed), but the method 'isRunning' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailProvider' final or making the method 'isRunning' static/final/abstract/empty, or adding allowed annotation for the method. | 149 |
| Error |
design | DesignForExtension | Class 'GMailProvider' looks like designed for extension (can be subclassed), but the method 'prepare' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailProvider' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 154 |
| Error |
misc | FinalParameters | Parameter configurationObject should be final. | 155 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 118). | 162 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 97). | 167 |
| Error |
design | DesignForExtension | Class 'GMailProvider' looks like designed for extension (can be subclassed), but the method 'cleanUp' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GMailProvider' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 175 |
| Error |
coding | MagicNumber | '5' is a magic number. | 178 |
Info
Warnings