The following document contains the results of Checkstyle. 
| Files | Infos  | Warnings  | Errors  |
|---|---|---|---|
| 33 | 0 | 0 | 694 |
| Rules | Violations | Severity |
|---|---|---|
| PackageHtml | 3 | Error |
| NewlineAtEndOfFile | 0 | Error |
| Translation | 0 | Error |
| JavadocMethod | 18 | Error |
| JavadocType | 0 | Error |
| JavadocVariable | 0 | Error |
| JavadocStyle | 19 | Error |
| ConstantName | 1 | Error |
| LocalFinalVariableName | 0 | Error |
| LocalVariableName | 0 | Error |
| MemberName | 0 | Error |
| MethodName | 0 | Error |
| PackageName | 0 | Error |
| ParameterName | 0 | Error |
| StaticVariableName | 0 | Error |
| TypeName | 0 | Error |
| AvoidStarImport | 0 | Error |
| IllegalImport | 0 | Error |
| RedundantImport | 0 | Error |
| UnusedImports | 0 | Error |
| FileLength | 0 | Error |
| LineLength | 12 | Error |
| MethodLength | 0 | Error |
| ParameterNumber | 0 | Error |
| EmptyForIteratorPad | 0 | Error |
| MethodParamPad | 0 | Error |
| NoWhitespaceAfter | 1 | Error |
| NoWhitespaceBefore | 0 | Error |
| OperatorWrap | 0 | Error |
| ParenPad | 0 | Error |
| TypecastParenPad | 0 | Error |
| TabCharacter | 0 | Error |
| WhitespaceAfter | 0 | Error |
| WhitespaceAround | 0 | Error |
| ModifierOrder | 0 | Error |
| RedundantModifier | 37 | Error |
| AvoidNestedBlocks | 0 | Error |
| EmptyBlock | 0 | Error |
| LeftCurly | 262 | Error |
| NeedBraces | 0 | Error |
| RightCurly | 14 | Error |
| AvoidInlineConditionals | 2 | Error |
| DoubleCheckedLocking | 0 | Error |
| EmptyStatement | 2 | Error |
| EqualsHashCode | 0 | Error |
| HiddenField | 34 | Error |
| IllegalInstantiation | 0 | Error |
| InnerAssignment | 0 | Error |
| MagicNumber | 67 | Error |
| MissingSwitchDefault | 0 | Error |
| RedundantThrows | 0 | Error |
| SimplifyBooleanExpression | 0 | Error |
| SimplifyBooleanReturn | 1 | Error |
| DesignForExtension | 84 | Error |
| FinalClass | 2 | Error |
| HideUtilityClassConstructor | 0 | Error |
| InterfaceIsType | 0 | Error |
| VisibilityModifier | 0 | Error |
| ArrayTypeStyle | 0 | Error |
| FinalParameters | 135 | Error |
GenericIllegalRegexp
| 0 | Error |
| TodoComment | 0 | Error |
| UpperEll | 0 | Error |
| Violation | Message | Line |
|---|---|---|
| First sentence should end with a period. | 31 |
| '{' should be on the previous line. | 38 |
| Line is longer than 80 characters. | 40 |
| Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 53 |
| '{' should be on the previous line. | 54 |
| Method 'setup' is not designed for extension - needs to be abstract, final or empty. | 63 |
| Parameter config should be final. | 63 |
| 'config' hides a field. | 63 |
| '{' should be on the previous line. | 64 |
| Method 'tearDown' is not designed for extension - needs to be abstract, final or empty. | 73 |
| '{' should be on the previous line. | 74 |
| Empty statement. | 75 |
| Method 'getFailAndJoinTimes' is not designed for extension - needs to be abstract, final or empty. | 83 |
| '{' should be on the previous line. | 84 |
| '{' should be on the previous line. | 91 |
| '{' should be on the previous line. | 93 |
| '{' should be on the previous line. | 97 |
| '}' should be on the same line. | 100 |
| '{' should be on the previous line. | 102 |
| Method 'convert' is not designed for extension - needs to be abstract, final or empty. | 122 |
| Parameter arr should be final. | 122 |
| '{' should be on the previous line. | 123 |
| '{' should be on the previous line. | 127 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 29 |
| '1123' is a magic number. | 31 |
| '600' is a magic number. | 34 |
| '1000' is a magic number. | 34 |
| '1200' is a magic number. | 37 |
| '1000' is a magic number. | 37 |
| '3600' is a magic number. | 40 |
| '1000' is a magic number. | 40 |
| '600' is a magic number. | 43 |
| '1000' is a magic number. | 43 |
| Method 'getFQN' is not designed for extension - needs to be abstract, final or empty. | 50 |
| '{' should be on the previous line. | 51 |
| '1000' is a magic number. | 55 |
| '1000' is a magic number. | 57 |
| '1000' is a magic number. | 59 |
| '1000' is a magic number. | 61 |
| Method 'setSeed' is not designed for extension - needs to be abstract, final or empty. | 71 |
| Parameter seed should be final. | 71 |
| 'seed' hides a field. | 71 |
| '{' should be on the previous line. | 72 |
| Method 'getSeed' is not designed for extension - needs to be abstract, final or empty. | 81 |
| '{' should be on the previous line. | 82 |
| First sentence should end with a period. | 86 |
| Method 'setWarmupInterval' is not designed for extension - needs to be abstract, final or empty. | 91 |
| Parameter interval should be final. | 91 |
| '{' should be on the previous line. | 92 |
| First sentence should end with a period. | 96 |
| Method 'getWarmupInterval' is not designed for extension - needs to be abstract, final or empty. | 101 |
| '{' should be on the previous line. | 102 |
| Method 'setFailureInterval' is not designed for extension - needs to be abstract, final or empty. | 111 |
| Parameter interval should be final. | 111 |
| '{' should be on the previous line. | 112 |
| Method 'getFailureTime' is not designed for extension - needs to be abstract, final or empty. | 121 |
| '{' should be on the previous line. | 122 |
| Method 'setMttf' is not designed for extension - needs to be abstract, final or empty. | 131 |
| Parameter mttf should be final. | 131 |
| 'mttf' hides a field. | 131 |
| '{' should be on the previous line. | 132 |
| Method 'getMttf' is not designed for extension - needs to be abstract, final or empty. | 141 |
| '{' should be on the previous line. | 142 |
| Unused @param tag for 'mttf'. | 149 |
| Method 'setMttr' is not designed for extension - needs to be abstract, final or empty. | 151 |
| Parameter mttr should be final. | 151 |
| 'mttr' hides a field. | 151 |
| Expected @param tag for 'mttr'. | 151 |
| '{' should be on the previous line. | 152 |
| Method 'getMttr' is not designed for extension - needs to be abstract, final or empty. | 161 |
| '{' should be on the previous line. | 162 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 29 |
| Redundant 'public' modifier. | 37 |
| Violation | Message | Line |
|---|---|---|
| Class Identifier should be declared as final. | 36 |
| '{' should be on the previous line. | 37 |
| Parameter id should be final. | 49 |
| 'id' hides a field. | 49 |
| '{' should be on the previous line. | 50 |
| Expected an @return tag. | 63 |
| Parameter n should be final. | 63 |
| Parameter head should be final. | 63 |
| Parameter tail should be final. | 63 |
| '{' should be on the previous line. | 64 |
| '8' is a magic number. | 67 |
| '8' is a magic number. | 68 |
| '{' should be on the previous line. | 72 |
| '{' should be on the previous line. | 74 |
| '{' should be on the previous line. | 80 |
| '8' is a magic number. | 83 |
| '{' should be on the previous line. | 84 |
| '8' is a magic number. | 89 |
| '8' is a magic number. | 89 |
| '{' should be on the previous line. | 90 |
| Parameter base should be final. | 109 |
| Parameter prefix should be final. | 109 |
| '{' should be on the previous line. | 110 |
| '{' should be on the previous line. | 114 |
| '8' is a magic number. | 123 |
| '7' is a magic number. | 124 |
| '8' is a magic number. | 124 |
| '}' should be on the same line. | 133 |
| '{' should be on the previous line. | 135 |
| '{' is followed by whitespace. | 136 |
| Parameter addr should be final. | 150 |
| Parameter tsmpFactory should be final. | 151 |
| '{' should be on the previous line. | 152 |
| Parameter addr should be final. | 167 |
| Parameter tsmpFactory should be final. | 168 |
| Parameter seed should be final. | 168 |
| '{' should be on the previous line. | 169 |
| First sentence should end with a period. | 175 |
| Parameter id should be final. | 182 |
| '{' should be on the previous line. | 183 |
| '{' should be on the previous line. | 198 |
| Parameter id should be final. | 214 |
| 'id' hides a field. | 214 |
| '{' should be on the previous line. | 215 |
| '{' should be on the previous line. | 217 |
| Parameter id should be final. | 231 |
| 'id' hides a field. | 231 |
| '{' should be on the previous line. | 232 |
| '{' should be on the previous line. | 238 |
| '{' should be on the previous line. | 243 |
| '0x80' is a magic number. | 244 |
| '8' is a magic number. | 246 |
| '{' should be on the previous line. | 247 |
| '{' should be on the previous line. | 250 |
| Parameter id should be final. | 270 |
| 'id' hides a field. | 270 |
| Parameter base should be final. | 270 |
| '{' should be on the previous line. | 271 |
| Parameter pos should be final. | 283 |
| Parameter base should be final. | 283 |
| '{' should be on the previous line. | 284 |
| '8' is a magic number. | 285 |
| '8' is a magic number. | 288 |
| '{' should be on the previous line. | 289 |
| '0x80' is a magic number. | 290 |
| '8' is a magic number. | 292 |
| '{' should be on the previous line. | 293 |
| '{' should be on the previous line. | 296 |
| '{' should be on the previous line. | 303 |
| Line is longer than 80 characters. | 316 |
| Parameter id should be final. | 322 |
| 'id' hides a field. | 322 |
| '{' should be on the previous line. | 323 |
| '{' should be on the previous line. | 327 |
| '{' should be on the previous line. | 336 |
| '0x80' is a magic number. | 339 |
| '{' should be on the previous line. | 340 |
| '}' should be on the same line. | 343 |
| '{' should be on the previous line. | 345 |
| '0x80' is a magic number. | 348 |
| '{' should be on the previous line. | 349 |
| Parameter o should be final. | 367 |
| '{' should be on the previous line. | 368 |
| '{' should be on the previous line. | 370 |
| '{' should be on the previous line. | 375 |
| '{' should be on the previous line. | 383 |
| '{' should be on the previous line. | 387 |
| '{' should be on the previous line. | 401 |
| '{' should be on the previous line. | 405 |
| Unused @throws tag for 'ClassCastException'. | 456 |
| Parameter o should be final. | 459 |
| '{' should be on the previous line. | 460 |
| '{' should be on the previous line. | 462 |
| '{' should be on the previous line. | 467 |
| '{' should be on the previous line. | 476 |
| '{' should be on the previous line. | 478 |
| '0xff' is a magic number. | 479 |
| '0xff' is a magic number. | 480 |
| '{' should be on the previous line. | 483 |
| '}' should be on the same line. | 485 |
| '{' should be on the previous line. | 487 |
| '{' should be on the previous line. | 502 |
| '{' should be on the previous line. | 506 |
| '0xf0' is a magic number. | 507 |
| '{' should be on the previous line. | 508 |
| '0xff' is a magic number. | 512 |
| Parameter lhs should be final. | 526 |
| Parameter rhs should be final. | 526 |
| '{' should be on the previous line. | 527 |
| '0x80' is a magic number. | 530 |
| '{' should be on the previous line. | 531 |
| Parameter val should be final. | 545 |
| '{' should be on the previous line. | 546 |
| '{' should be on the previous line. | 550 |
| '0xff' is a magic number. | 551 |
| '0x100' is a magic number. | 553 |
| Avoid inline conditionals. | 553 |
| Parameter lhs should be final. | 570 |
| Parameter rhs should be final. | 570 |
| '{' should be on the previous line. | 571 |
| '{' should be on the previous line. | 578 |
| '0xff' is a magic number. | 579 |
| '0xff' is a magic number. | 579 |
| '{' should be on the previous line. | 584 |
| '0x100' is a magic number. | 586 |
| '}' should be on the same line. | 587 |
| '{' should be on the previous line. | 589 |
| '{' should be on the previous line. | 606 |
| '{' should be on the previous line. | 610 |
| '0xf0' is a magic number. | 611 |
| '{' should be on the previous line. | 612 |
| '0xff' is a magic number. | 616 |
| '{' should be on the previous line. | 628 |
| First sentence should end with a period. | 632 |
| Parameter lhs should be final. | 638 |
| Parameter rhs should be final. | 638 |
| '{' should be on the previous line. | 639 |
| Avoid inline conditionals. | 641 |
| Method 'getDifference' is not designed for extension - needs to be abstract, final or empty. | 650 |
| '{' should be on the previous line. | 651 |
| Unused @throws tag for 'ClassCastException'. | 699 |
| Method 'compareTo' is not designed for extension - needs to be abstract, final or empty. | 702 |
| Parameter o should be final. | 702 |
| '{' should be on the previous line. | 703 |
| '{' should be on the previous line. | 705 |
| '{' should be on the previous line. | 710 |
| '{' should be on the previous line. | 718 |
| '{' should be on the previous line. | 720 |
| '0xff' is a magic number. | 721 |
| '0xff' is a magic number. | 722 |
| '{' should be on the previous line. | 725 |
| '}' should be on the same line. | 727 |
| '{' should be on the previous line. | 729 |
| Line is longer than 80 characters. | 745 |
| Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 748 |
| Parameter o should be final. | 748 |
| '{' should be on the previous line. | 749 |
| '{' should be on the previous line. | 751 |
| '{' should be on the previous line. | 756 |
| Line is longer than 80 characters. | 757 |
| '{' should be on the previous line. | 763 |
| '{' should be on the previous line. | 765 |
| '0xff' is a magic number. | 766 |
| '0xff' is a magic number. | 767 |
| Conditional logic can be removed. | 769 |
| '{' should be on the previous line. | 770 |
| '}' should be on the same line. | 772 |
| '{' should be on the previous line. | 774 |
| Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 788 |
| '{' should be on the previous line. | 789 |
| '{' should be on the previous line. | 793 |
| Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 805 |
| '{' should be on the previous line. | 806 |
| '{' should be on the previous line. | 810 |
| '0xf0' is a magic number. | 811 |
| '{' should be on the previous line. | 812 |
| '0xff' is a magic number. | 816 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 27 |
| First sentence should end with a period. | 28 |
| Redundant 'public' modifier. | 33 |
| Redundant 'public' modifier. | 38 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 36 |
| Parameter id should be final. | 56 |
| 'id' hides a field. | 56 |
| Parameter address should be final. | 56 |
| 'address' hides a field. | 56 |
| Parameter port should be final. | 56 |
| 'port' hides a field. | 56 |
| '{' should be on the previous line. | 57 |
| Parameter id should be final. | 69 |
| 'id' hides a field. | 69 |
| Parameter addr should be final. | 69 |
| '{' should be on the previous line. | 70 |
| Parameter soaddr should be final. | 82 |
| Parameter tsmpFactory should be final. | 83 |
| Expected @param tag for 'tsmpFactory'. | 83 |
| '{' should be on the previous line. | 84 |
| '{' should be on the previous line. | 88 |
| Parameter addr should be final. | 104 |
| Parameter tsmpFactory should be final. | 105 |
| '{' should be on the previous line. | 106 |
| Parameter id should be final. | 120 |
| '{' should be on the previous line. | 121 |
| Parameter id should be final. | 133 |
| Parameter addr should be final. | 133 |
| '{' should be on the previous line. | 134 |
| Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 143 |
| '{' should be on the previous line. | 144 |
| Method 'getInetAddress' is not designed for extension - needs to be abstract, final or empty. | 153 |
| '{' should be on the previous line. | 154 |
| Method 'getPort' is not designed for extension - needs to be abstract, final or empty. | 163 |
| '{' should be on the previous line. | 164 |
| Method 'getSocketAddress' is not designed for extension - needs to be abstract, final or empty. | 173 |
| '{' should be on the previous line. | 174 |
| Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 188 |
| Parameter o should be final. | 188 |
| '{' should be on the previous line. | 189 |
| '{' should be on the previous line. | 191 |
| '{' should be on the previous line. | 196 |
| '{' should be on the previous line. | 204 |
| '}' should be on the same line. | 206 |
| '{' should be on the previous line. | 208 |
| Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 220 |
| '{' should be on the previous line. | 221 |
| '{' should be on the previous line. | 227 |
| Unused @throws tag for 'ClassCastException'. | 278 |
| Method 'compareTo' is not designed for extension - needs to be abstract, final or empty. | 281 |
| Parameter o should be final. | 281 |
| '{' should be on the previous line. | 282 |
| '{' should be on the previous line. | 284 |
| '{' should be on the previous line. | 289 |
| Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 303 |
| '{' should be on the previous line. | 304 |
| '{' should be on the previous line. | 310 |
| Method 'toFullString' is not designed for extension - needs to be abstract, final or empty. | 323 |
| '{' should be on the previous line. | 324 |
| '{' should be on the previous line. | 330 |
| Parameter id should be final. | 346 |
| '{' should be on the previous line. | 348 |
| '{' should be on the previous line. | 352 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 27 |
| Redundant 'public' modifier. | 33 |
| Redundant 'public' modifier. | 40 |
| Redundant 'public' modifier. | 47 |
| Redundant 'public' modifier. | 54 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 33 |
| Parameter socket should be final. | 42 |
| 'socket' hides a field. | 42 |
| '{' should be on the previous line. | 43 |
| Expected an @return tag. | 53 |
| Method 'getCost' is not designed for extension - needs to be abstract, final or empty. | 53 |
| Parameter node should be final. | 53 |
| '{' should be on the previous line. | 54 |
| '0.5' is a magic number. | 70 |
| Parameter percentage should be final. | 88 |
| '{' should be on the previous line. | 89 |
| '0.5' is a magic number. | 90 |
| '10.0' is a magic number. | 93 |
| Method 'getDefaultCost' is not designed for extension - needs to be abstract, final or empty. | 101 |
| '{' should be on the previous line. | 102 |
| '3000.0' is a magic number. | 103 |
| Line is longer than 80 characters. | 110 |
| Method 'getLatency' is not designed for extension - needs to be abstract, final or empty. | 115 |
| Parameter node should be final. | 115 |
| '{' should be on the previous line. | 116 |
| Line is longer than 80 characters. | 117 |
| Method 'getLossRate' is not designed for extension - needs to be abstract, final or empty. | 129 |
| Parameter node should be final. | 129 |
| '{' should be on the previous line. | 130 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 29 |
| Method 'getFQN' is not designed for extension - needs to be abstract, final or empty. | 44 |
| '{' should be on the previous line. | 45 |
| '{' should be on the previous line. | 49 |
| '}' should be on the same line. | 51 |
| '{' should be on the previous line. | 53 |
| '{' should be on the previous line. | 55 |
| '}' should be on the same line. | 57 |
| '{' should be on the previous line. | 59 |
| Method 'setShortName' is not designed for extension - needs to be abstract, final or empty. | 76 |
| Parameter name should be final. | 76 |
| '{' should be on the previous line. | 77 |
| Method 'getShortName' is not designed for extension - needs to be abstract, final or empty. | 86 |
| '{' should be on the previous line. | 87 |
| Method 'setClassName' is not designed for extension - needs to be abstract, final or empty. | 96 |
| Parameter serviceModuleClassName should be final. | 96 |
| '{' should be on the previous line. | 97 |
| Method 'getClassName' is not designed for extension - needs to be abstract, final or empty. | 106 |
| '{' should be on the previous line. | 107 |
| Method 'setConfiguration' is not designed for extension - needs to be abstract, final or empty. | 116 |
| Parameter serviceModuleConfiguration should be final. | 116 |
| Method 'getConfiguration' is not designed for extension - needs to be abstract, final or empty. | 126 |
| '{' should be on the previous line. | 127 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 33 |
| Line is longer than 80 characters. | 35 |
| Name 'instance' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 35 |
| Parameter timer should be final. | 45 |
| 'timer' hides a field. | 45 |
| '{' should be on the previous line. | 46 |
| '{' should be on the previous line. | 56 |
| Unused @param tag for 'delta'. | 63 |
| Method 'create' is not designed for extension - needs to be abstract, final or empty. | 67 |
| Parameter config should be final. | 67 |
| Expected @param tag for 'config'. | 67 |
| '{' should be on the previous line. | 68 |
| Method 'create' is not designed for extension - needs to be abstract, final or empty. | 83 |
| '{' should be on the previous line. | 84 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 29 |
| Redundant 'public' modifier. | 38 |
| Redundant 'public' modifier. | 45 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 28 |
| Redundant 'public' modifier. | 34 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 29 |
| First sentence should end with a period. | 30 |
| Redundant 'public' modifier. | 35 |
| Redundant 'public' modifier. | 42 |
| Redundant 'public' modifier. | 49 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 28 |
| Redundant 'public' modifier. | 34 |
| Redundant 'public' modifier. | 39 |
| Redundant 'public' modifier. | 46 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 37 |
| Parameter id should be final. | 43 |
| '{' should be on the previous line. | 44 |
| Parameter decoder should be final. | 58 |
| Parameter typeMapping should be final. | 58 |
| '{' should be on the previous line. | 60 |
| '4' is a magic number. | 67 |
| '{' should be on the previous line. | 74 |
| '}' should be on the same line. | 78 |
| '{' should be on the previous line. | 80 |
| Parameter encoder should be final. | 97 |
| Parameter typeMapping should be final. | 97 |
| Parameter instance should be final. | 98 |
| '{' should be on the previous line. | 99 |
| Line is longer than 80 characters. | 103 |
| '4' is a magic number. | 104 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 37 |
| Parameter id should be final. | 43 |
| '{' should be on the previous line. | 44 |
| Parameter decoder should be final. | 58 |
| Parameter typeMapping should be final. | 58 |
| '{' should be on the previous line. | 60 |
| Parameter encoder should be final. | 73 |
| Parameter typeMapping should be final. | 73 |
| Parameter instance should be final. | 74 |
| '{' should be on the previous line. | 75 |
| Parameter decoder should be final. | 89 |
| Parameter typeMapping should be final. | 90 |
| '{' should be on the previous line. | 92 |
| '4' is a magic number. | 99 |
| '{' should be on the previous line. | 106 |
| '}' should be on the same line. | 110 |
| '{' should be on the previous line. | 112 |
| Parameter encoder should be final. | 129 |
| Parameter typeMapping should be final. | 130 |
| Parameter instance should be final. | 130 |
| '{' should be on the previous line. | 132 |
| '4' is a magic number. | 138 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 27 |
| Redundant 'public' modifier. | 33 |
| Redundant 'public' modifier. | 40 |
| First sentence should end with a period. | 42 |
| Redundant 'public' modifier. | 47 |
| First sentence should end with a period. | 49 |
| Redundant 'public' modifier. | 54 |
| First sentence should end with a period. | 56 |
| Redundant 'public' modifier. | 61 |
| Redundant 'public' modifier. | 68 |
| Redundant 'public' modifier. | 75 |
| Redundant 'public' modifier. | 82 |
| Redundant 'public' modifier. | 89 |
| Redundant 'public' modifier. | 96 |
| Redundant 'public' modifier. | 103 |
| Redundant 'public' modifier. | 110 |
| Redundant 'public' modifier. | 117 |
| First sentence should end with a period. | 119 |
| Redundant 'public' modifier. | 124 |
| Redundant 'public' modifier. | 131 |
| Redundant 'public' modifier. | 138 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 28 |
| Redundant 'public' modifier. | 36 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 36 |
| Parameter id should be final. | 42 |
| '{' should be on the previous line. | 43 |
| Parameter decoder should be final. | 57 |
| Parameter typeMapping should be final. | 57 |
| '{' should be on the previous line. | 59 |
| Parameter encoder should be final. | 74 |
| Parameter typeMapping should be final. | 74 |
| Parameter instance should be final. | 75 |
| '{' should be on the previous line. | 76 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 32 |
| Redundant 'public' modifier. | 42 |
| Redundant 'public' modifier. | 50 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 34 |
| Parameter id should be final. | 40 |
| '{' should be on the previous line. | 41 |
| Parameter decoder should be final. | 55 |
| Parameter typeMapping should be final. | 55 |
| '{' should be on the previous line. | 57 |
| Parameter encoder should be final. | 78 |
| Parameter typeMapping should be final. | 78 |
| Parameter instance should be final. | 79 |
| '{' should be on the previous line. | 80 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 35 |
| Parameter lossThreshold should be final. | 62 |
| 'lossThreshold' hides a field. | 62 |
| Parameter unreportedInterval should be final. | 62 |
| 'unreportedInterval' hides a field. | 62 |
| Expected @param tag for 'unreportedInterval'. | 62 |
| Parameter probingInterval should be final. | 63 |
| 'probingInterval' hides a field. | 63 |
| Parameter sock should be final. | 63 |
| 'sock' hides a field. | 63 |
| Parameter tsmpFactory should be final. | 63 |
| 'tsmpFactory' hides a field. | 63 |
| Unused @param tag for 'addr'. | 76 |
| Method 'isUnreported' is not designed for extension - needs to be abstract, final or empty. | 80 |
| Parameter addr should be final. | 80 |
| Parameter unreportedInterval should be final. | 80 |
| 'unreportedInterval' hides a field. | 80 |
| Expected @param tag for 'unreportedInterval'. | 80 |
| '{' should be on the previous line. | 81 |
| Line is longer than 80 characters. | 87 |
| Method 'isUnreported' is not designed for extension - needs to be abstract, final or empty. | 101 |
| Parameter addr should be final. | 101 |
| '{' should be on the previous line. | 102 |
| Method 'isDead' is not designed for extension - needs to be abstract, final or empty. | 113 |
| Parameter addr should be final. | 113 |
| '{' should be on the previous line. | 114 |
| '{' should be on the previous line. | 119 |
| Method 'isProbingRequired' is not designed for extension - needs to be abstract, final or empty. | 133 |
| Parameter addr should be final. | 133 |
| '{' should be on the previous line. | 134 |
| Method 'getTimestampFactory' is not designed for extension - needs to be abstract, final or empty. | 146 |
| '{' should be on the previous line. | 147 |
| 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. | 38 |
| '97' is a magic number. | 40 |
| '300' is a magic number. | 46 |
| '1000' is a magic number. | 46 |
| '1800' is a magic number. | 49 |
| '1000' is a magic number. | 49 |
| '60' is a magic number. | 52 |
| '1000' is a magic number. | 52 |
| '256' is a magic number. | 58 |
| '32' is a magic number. | 64 |
| Parameter experiment should be final. | 91 |
| Parameter output should be final. | 91 |
| Expected @param tag for 'output'. | 91 |
| Parameter listener should be final. | 92 |
| Expected @param tag for 'listener'. | 92 |
| '{' should be on the previous line. | 94 |
| Line is longer than 80 characters. | 95 |
| Method 'setSeed' is not designed for extension - needs to be abstract, final or empty. | 110 |
| Parameter seed should be final. | 110 |
| 'seed' hides a field. | 110 |
| '{' should be on the previous line. | 111 |
| Method 'getSeed' is not designed for extension - needs to be abstract, final or empty. | 120 |
| '{' should be on the previous line. | 121 |
| Method 'setNumRuns' is not designed for extension - needs to be abstract, final or empty. | 130 |
| Parameter numRuns should be final. | 130 |
| 'numRuns' hides a field. | 130 |
| '{' should be on the previous line. | 131 |
| Method 'getNumRuns' is not designed for extension - needs to be abstract, final or empty. | 140 |
| '{' should be on the previous line. | 141 |
| First sentence should end with a period. | 145 |
| Method 'setStartupInterval' is not designed for extension - needs to be abstract, final or empty. | 150 |
| Parameter interval should be final. | 150 |
| '{' should be on the previous line. | 151 |
| First sentence should end with a period. | 155 |
| Method 'getStartupInterval' is not designed for extension - needs to be abstract, final or empty. | 160 |
| '{' should be on the previous line. | 161 |
| First sentence should end with a period. | 165 |
| Method 'setExperimentInterval' is not designed for extension - needs to be abstract, final or empty. | 170 |
| Parameter interval should be final. | 170 |
| '{' should be on the previous line. | 171 |
| First sentence should end with a period. | 175 |
| Method 'getExperimentInterval' is not designed for extension - needs to be abstract, final or empty. | 180 |
| '{' should be on the previous line. | 181 |
| First sentence should end with a period. | 185 |
| Unused Javadoc tag. | 188 |
| Method 'setTopologyDumpInterval' is not designed for extension - needs to be abstract, final or empty. | 190 |
| Parameter val should be final. | 190 |
| Expected @param tag for 'val'. | 190 |
| '{' should be on the previous line. | 191 |
| First sentence should end with a period. | 195 |
| Method 'getTopologyDumpInterval' is not designed for extension - needs to be abstract, final or empty. | 200 |
| '{' should be on the previous line. | 201 |
| Method 'setTopology' is not designed for extension - needs to be abstract, final or empty. | 210 |
| Parameter topology should be final. | 210 |
| 'topology' hides a field. | 210 |
| '{' should be on the previous line. | 211 |
| Method 'getTopology' is not designed for extension - needs to be abstract, final or empty. | 220 |
| '{' should be on the previous line. | 221 |
| Method 'setNumClients' is not designed for extension - needs to be abstract, final or empty. | 230 |
| Parameter numClients should be final. | 230 |
| 'numClients' hides a field. | 230 |
| '{' should be on the previous line. | 231 |
| Method 'getNumClients' is not designed for extension - needs to be abstract, final or empty. | 240 |
| '{' should be on the previous line. | 241 |
| Method 'setNumBootstrap' is not designed for extension - needs to be abstract, final or empty. | 250 |
| Parameter numBootstrap should be final. | 250 |
| 'numBootstrap' hides a field. | 250 |
| '{' should be on the previous line. | 251 |
| Method 'getNumBootstrap' is not designed for extension - needs to be abstract, final or empty. | 260 |
| '{' should be on the previous line. | 261 |
| Method 'setNumStartupClients' is not designed for extension - needs to be abstract, final or empty. | 270 |
| Parameter numStartupClients should be final. | 270 |
| 'numStartupClients' hides a field. | 270 |
| '{' should be on the previous line. | 271 |
| Method 'getNumStartupClients' is not designed for extension - needs to be abstract, final or empty. | 280 |
| '{' should be on the previous line. | 281 |
| Method 'getFQN' is not designed for extension - needs to be abstract, final or empty. | 290 |
| '{' should be on the previous line. | 291 |
| '{' should be on the previous line. | 295 |
| '}' should be on the same line. | 297 |
| '{' should be on the previous line. | 299 |
| '1000' is a magic number. | 311 |
| '1000' is a magic number. | 313 |
| '{' should be on the previous line. | 318 |
| '{' should be on the previous line. | 326 |
| Method 'setServiceClassName' is not designed for extension - needs to be abstract, final or empty. | 339 |
| Parameter serviceClassName should be final. | 339 |
| 'serviceClassName' hides a field. | 339 |
| '{' should be on the previous line. | 340 |
| Method 'getServiceClassName' is not designed for extension - needs to be abstract, final or empty. | 349 |
| '{' should be on the previous line. | 350 |
| Method 'setBootstrapClassName' is not designed for extension - needs to be abstract, final or empty. | 359 |
| Parameter bootstrapClassName should be final. | 359 |
| 'bootstrapClassName' hides a field. | 359 |
| '{' should be on the previous line. | 360 |
| Method 'getBootstrapClassName' is not designed for extension - needs to be abstract, final or empty. | 369 |
| '{' should be on the previous line. | 370 |
| Method 'setServiceConfiguration' is not designed for extension - needs to be abstract, final or empty. | 379 |
| Parameter serviceConfiguration should be final. | 379 |
| 'serviceConfiguration' hides a field. | 379 |
| Method 'getServiceConfiguration' is not designed for extension - needs to be abstract, final or empty. | 389 |
| '{' should be on the previous line. | 390 |
| Method 'setBootstrapConfiguration' is not designed for extension - needs to be abstract, final or empty. | 399 |
| Parameter bootstrapConfiguration should be final. | 400 |
| 'bootstrapConfiguration' hides a field. | 400 |
| '{' should be on the previous line. | 401 |
| First sentence should end with a period. | 405 |
| Method 'getBootstrapConfiguration' is not designed for extension - needs to be abstract, final or empty. | 410 |
| '{' should be on the previous line. | 411 |
| Method 'setWorkloads' is not designed for extension - needs to be abstract, final or empty. | 420 |
| Parameter workloads should be final. | 420 |
| 'workloads' hides a field. | 420 |
| '{' should be on the previous line. | 421 |
| Method 'getWorkloads' is not designed for extension - needs to be abstract, final or empty. | 430 |
| '{' should be on the previous line. | 431 |
| Method 'setFailureModels' is not designed for extension - needs to be abstract, final or empty. | 440 |
| Parameter failureModels should be final. | 440 |
| 'failureModels' hides a field. | 440 |
| '{' should be on the previous line. | 441 |
| Method 'getFailureModels' is not designed for extension - needs to be abstract, final or empty. | 450 |
| '{' should be on the previous line. | 451 |
| Violation | Message | Line |
|---|---|---|
| Class NodeId should be declared as final. | 36 |
| '{' should be on the previous line. | 37 |
| Parameter id should be final. | 51 |
| Parameter address should be final. | 51 |
| Parameter port should be final. | 51 |
| '{' should be on the previous line. | 52 |
| Parameter id should be final. | 62 |
| Parameter addr should be final. | 62 |
| '{' should be on the previous line. | 63 |
| Parameter nodes should be final. | 74 |
| '{' should be on the previous line. | 75 |
| Parameter soaddr should be final. | 91 |
| Parameter tsmpFactory should be final. | 92 |
| '{' should be on the previous line. | 93 |
| '{' should be on the previous line. | 97 |
| '{' should be on the previous line. | 110 |
| '{' should be on the previous line. | 112 |
| '{' should be on the previous line. | 114 |
| '4' is a magic number. | 116 |
| '}' should be on the same line. | 117 |
| '{' should be on the previous line. | 119 |
| Empty statement. | 120 |
| Parameter addr should be final. | 135 |
| Parameter tsmpFactory should be final. | 136 |
| '{' should be on the previous line. | 137 |
| Line is longer than 80 characters. | 138 |
| Parameter id should be final. | 149 |
| '{' should be on the previous line. | 150 |
| Parameter id should be final. | 162 |
| Parameter addr should be final. | 162 |
| '{' should be on the previous line. | 163 |
| Parameter id should be final. | 176 |
| '{' should be on the previous line. | 178 |
| '{' should be on the previous line. | 182 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 28 |
| Redundant 'public' modifier. | 34 |
| Redundant 'public' modifier. | 41 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 29 |
| Expected an @return tag. | 36 |
| Line is longer than 80 characters. | 49 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 35 |
| '{' should be on the previous line. | 46 |
| Method 'setTimerTask' is not designed for extension - needs to be abstract, final or empty. | 54 |
| Parameter timerTask should be final. | 54 |
| 'timerTask' hides a field. | 54 |
| '{' should be on the previous line. | 55 |
| First sentence should end with a period. | 59 |
| Method 'add' is not designed for extension - needs to be abstract, final or empty. | 64 |
| Parameter agent should be final. | 64 |
| '{' should be on the previous line. | 65 |
| '{' should be on the previous line. | 69 |
| Method 'run' is not designed for extension - needs to be abstract, final or empty. | 78 |
| '{' should be on the previous line. | 79 |
| '{' should be on the previous line. | 81 |
| Method 'cancel' is not designed for extension - needs to be abstract, final or empty. | 89 |
| '{' should be on the previous line. | 90 |
