The following document contains the results of Checkstyle. 
| Files | Infos | Warnings | Errors |
|---|---|---|---|
| 14 | 0 | 0 | 114 |
| Rules | Violations | Severity |
|---|---|---|
| PackageHtml | 2 | |
| NewlineAtEndOfFile | 0 | |
| Translation | 0 | |
| JavadocMethod | 1 | |
| JavadocType | 0 | |
| JavadocVariable | 0 | |
| JavadocStyle | 1 | |
| ConstantName | 0 | |
| LocalFinalVariableName | 0 | |
| LocalVariableName | 0 | |
| MemberName | 0 | |
| MethodName | 0 | |
| PackageName | 0 | |
| ParameterName | 0 | |
| StaticVariableName | 0 | |
| TypeName | 0 | |
| AvoidStarImport | 0 | |
| IllegalImport | 0 | |
| RedundantImport | 0 | |
| UnusedImports | 0 | |
| FileLength | 0 | |
| LineLength | 1 | |
| MethodLength | 0 | |
| ParameterNumber | 0 | |
| EmptyForIteratorPad | 0 | |
| MethodParamPad | 0 | |
| NoWhitespaceAfter | 0 | |
| NoWhitespaceBefore | 0 | |
| OperatorWrap | 0 | |
| ParenPad | 0 | |
| TypecastParenPad | 0 | |
| TabCharacter | 0 | |
| WhitespaceAfter | 0 | |
| WhitespaceAround | 0 | |
| ModifierOrder | 0 | |
| RedundantModifier | 34 | |
| AvoidNestedBlocks | 0 | |
| EmptyBlock | 0 | |
| LeftCurly | 42 | |
| NeedBraces | 0 | |
| RightCurly | 1 | |
| AvoidInlineConditionals | 0 | |
| DoubleCheckedLocking | 0 | |
| EmptyStatement | 1 | |
| EqualsHashCode | 0 | |
| HiddenField | 4 | |
| IllegalInstantiation | 0 | |
| InnerAssignment | 0 | |
| MagicNumber | 0 | |
| MissingSwitchDefault | 0 | |
| RedundantThrows | 0 | |
| SimplifyBooleanExpression | 0 | |
| SimplifyBooleanReturn | 0 | |
| DesignForExtension | 17 | |
| FinalClass | 0 | |
| HideUtilityClassConstructor | 0 | |
| InterfaceIsType | 0 | |
| VisibilityModifier | 1 | |
| ArrayTypeStyle | 0 | |
| FinalParameters | 9 | |
GenericIllegalRegexp
| 0 | |
| TodoComment | 0 | |
| UpperEll | 0 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 18 | |
| Redundant 'public' modifier. | 24 | |
| Redundant 'public' modifier. | 31 | |
| Redundant 'public' modifier. | 40 | |
| Redundant 'public' modifier. | 47 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 30 | |
| Redundant 'public' modifier. | 38 | |
| Redundant 'public' modifier. | 45 | |
| Redundant 'public' modifier. | 52 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 32 | |
| Redundant 'public' modifier. | 38 | |
| Redundant 'public' modifier. | 45 | |
| Redundant 'public' modifier. | 52 | |
| Redundant 'public' modifier. | 59 | |
| Redundant 'public' modifier. | 66 | |
| Redundant 'public' modifier. | 73 | |
| Redundant 'public' modifier. | 80 | |
| Redundant 'public' modifier. | 87 | |
| Redundant 'public' modifier. | 94 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 30 | |
| Redundant 'public' modifier. | 36 | |
| Redundant 'public' modifier. | 45 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 29 | |
| Redundant 'public' modifier. | 37 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 31 | |
| Redundant 'public' modifier. | 37 | |
| First sentence should end with a period. | 39 | |
| Redundant 'public' modifier. | 44 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 20 | |
| Redundant 'public' modifier. | 26 | |
| Redundant 'public' modifier. | 33 | |
| Line is longer than 80 characters. | 36 | |
| Redundant 'public' modifier. | 42 | |
| Redundant 'public' modifier. | 51 | |
| Expected @param tag for 'destination'. | 51 | |
| Redundant 'public' modifier. | 58 | |
| Redundant 'public' modifier. | 65 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 20 | |
| Redundant 'public' modifier. | 26 | |
| Redundant 'public' modifier. | 33 | |
| Redundant 'public' modifier. | 40 | |
| Redundant 'public' modifier. | 47 | |
| Redundant 'public' modifier. | 54 | |
| Redundant 'public' modifier. | 62 | |
| Redundant 'public' modifier. | 69 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 35 | |
| '{' should be on the previous line. | 43 | |
| Parameter numepochs should be final. | 52 | |
| '{' should be on the previous line. | 53 | |
| '{' should be on the previous line. | 55 | |
| '{' should be on the previous line. | 63 | |
| Method 'age' is not designed for extension - needs to be abstract, final or empty. | 71 | |
| '{' should be on the previous line. | 72 | |
| '{' should be on the previous line. | 74 | |
| Method 'insert' is not designed for extension - needs to be abstract, final or empty. | 86 | |
| Parameter node should be final. | 86 | |
| '{' should be on the previous line. | 87 | |
| '{' should be on the previous line. | 91 | |
| Method 'remove' is not designed for extension - needs to be abstract, final or empty. | 103 | |
| Parameter node should be final. | 103 | |
| '{' should be on the previous line. | 104 | |
| '{' should be on the previous line. | 108 | |
| Method 'contains' is not designed for extension - needs to be abstract, final or empty. | 122 | |
| Parameter node should be final. | 122 | |
| '{' should be on the previous line. | 123 | |
| '{' should be on the previous line. | 127 | |
| Method 'getAll' is not designed for extension - needs to be abstract, final or empty. | 139 | |
| '{' should be on the previous line. | 140 | |
| '{' should be on the previous line. | 144 | |
| Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 156 | |
| '{' should be on the previous line. | 157 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 33 | |
| Variable 'completeNotifier' must be private and have accessor methods. | 56 | |
| Parameter identifier should be final. | 64 | |
| 'identifier' hides a field. | 64 | |
| Parameter timestamp should be final. | 64 | |
| 'timestamp' hides a field. | 64 | |
| '{' should be on the previous line. | 65 | |
| Method 'getOneTimeCompleteNotifier' is not designed for extension - needs to be abstract, final or empty. | 75 | |
| '{' should be on the previous line. | 76 | |
| Method 'setNodeId' is not designed for extension - needs to be abstract, final or empty. | 87 | |
| Parameter res should be final. | 87 | |
| Parameter hopDistance should be final. | 87 | |
| 'hopDistance' hides a field. | 87 | |
| Parameter lookupTime should be final. | 88 | |
| 'lookupTime' hides a field. | 88 | |
| '{' should be on the previous line. | 89 | |
| Method 'isComplete' is not designed for extension - needs to be abstract, final or empty. | 106 | |
| '{' should be on the previous line. | 107 | |
| Method 'hasFailed' is not designed for extension - needs to be abstract, final or empty. | 116 | |
| '{' should be on the previous line. | 117 | |
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 126 | |
| '{' should be on the previous line. | 127 | |
| Method 'getNodeId' is not designed for extension - needs to be abstract, final or empty. | 136 | |
| '{' should be on the previous line. | 137 | |
| Method 'getNodeIdBlocking' is not designed for extension - needs to be abstract, final or empty. | 146 | |
| '{' should be on the previous line. | 147 | |
| '{' should be on the previous line. | 149 | |
| '{' should be on the previous line. | 151 | |
| '}' should be on the same line. | 153 | |
| '{' should be on the previous line. | 155 | |
| Empty statement. | 156 | |
| Method 'getHopDistance' is not designed for extension - needs to be abstract, final or empty. | 168 | |
| '{' should be on the previous line. | 169 | |
| Method 'getTimestamp' is not designed for extension - needs to be abstract, final or empty. | 178 | |
| '{' should be on the previous line. | 179 | |
| Method 'getLookupTime' is not designed for extension - needs to be abstract, final or empty. | 188 | |
| '{' should be on the previous line. | 189 | |
| Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 198 | |
| '{' should be on the previous line. | 199 |