Checkstyle Results

The following document contains the results of Checkstyle.

Summary

Files	Infos	Warnings	Errors
42	0	0	1175

Rules

Rules	Violations	Severity
PackageHtml	4	Error
NewlineAtEndOfFile	0	Error
Translation	0	Error
JavadocMethod	21	Error
JavadocType	0	Error
JavadocVariable	1	Error
JavadocStyle	8	Error
ConstantName	0	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	48	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	22	Error
AvoidNestedBlocks	0	Error
EmptyBlock	0	Error
LeftCurly	487	Error
NeedBraces	0	Error
RightCurly	39	Error
AvoidInlineConditionals	1	Error
DoubleCheckedLocking	0	Error
EmptyStatement	2	Error
EqualsHashCode	0	Error
HiddenField	81	Error
IllegalInstantiation	0	Error
InnerAssignment	4	Error
MagicNumber	12	Error
MissingSwitchDefault	0	Error
RedundantThrows	0	Error
SimplifyBooleanExpression	0	Error
SimplifyBooleanReturn	0	Error
DesignForExtension	192	Error
FinalClass	0	Error
HideUtilityClassConstructor	0	Error
InterfaceIsType	0	Error
VisibilityModifier	0	Error
ArrayTypeStyle	0	Error
FinalParameters	248	Error
GenericIllegalRegexp message: `"Line has trailing spaces."` format: `"\s+$"`	3	Error
TodoComment	1	Error
UpperEll	0	Error

Files

Files	I	W	E
edu/nwu/scribe/bll/DefaultSessionPolicy.java	0	0	34
edu/nwu/scribe/dll/JoinRequest.java	0	0	23
edu/nwu/scribe/bll/ISessionPolicy.java	0	0	4
edu/nwu/scribe/bll/package.html	0	0	1
edu/nwu/scribe/dll/DataPacket.java	0	0	39
edu/nwu/scribe/dll/JoinRejected.java	0	0	32
edu/nwu/scribe/bll/IPreemptionProtocol.java	0	0	2
edu/nwu/scribe/bll/FatTreeJoinProtocol.java	0	0	24
edu/nwu/freepastry/bll/ScribeFreePastryService.java	0	0	47
edu/nwu/scribe/dll/ScribePacketMapping.java	0	0	2
edu/nwu/scribe/bll/ISessionState.java	0	0	16
edu/nwu/splitstream/bll/Configuration.java	0	0	11
edu/nwu/splitstream/bll/SplitStreamMulticastPublisherSession.java	0	0	14
edu/nwu/freepastry/bll/package.html	0	0	1
edu/nwu/scribe/bll/SessionState.java	0	0	70
edu/nwu/freepastry/bll/SplitStreamFreePastryService.java	0	0	46
edu/nwu/scribe/dll/CostQueryReply.java	0	0	23
edu/nwu/scribe/bll/Configuration.java	0	0	53
edu/nwu/scribe/bll/JoinProtocol.java	0	0	173
edu/nwu/freepastry/bll/ScribeMulticastPublisher.java	0	0	24
edu/nwu/freepastry/bll/SplitStreamMulticast.java	0	0	68
edu/nwu/scribe/dll/package.html	0	0	1
edu/nwu/scribe/dll/PublishRequest.java	0	0	37
edu/nwu/scribe/bll/IScribeClient.java	0	0	1
edu/nwu/scribe/bll/MulticastProtocol.java	0	0	67
edu/nwu/scribe/dll/JoinRedirect.java	0	0	19
edu/nwu/scribe/bll/IJoinProtocol.java	0	0	3
edu/nwu/scribe/bll/ScribePastryService.java	0	0	87
edu/nwu/scribe/dll/EScribePacketType.java	0	0	14
edu/nwu/scribe/dll/JoinAccepted.java	0	0	19
edu/nwu/freepastry/bll/ScribeFreePastryConfiguration.java	0	0	1
edu/nwu/freepastry/bll/ScribeMulticastClient.java	0	0	50
edu/nwu/splitstream/bll/SplitStreamConfiguration.java	0	0	1
edu/nwu/scribe/bll/MaintenanceProtocol.java	0	0	60
edu/nwu/scribe/bll/SessionStateManager.java	0	0	16
edu/nwu/scribe/bll/ScribeConfiguration.java	0	0	1
edu/nwu/scribe/bll/ISessionStateManager.java	0	0	3
edu/nwu/splitstream/bll/SplitStreamSessionPolicy.java	0	0	36
edu/nwu/freepastry/bll/SplitStreamFreePastryConfiguration.java	0	0	1
edu/nwu/splitstream/bll/SplitStreamPastryService.java	0	0	31
edu/nwu/scribe/dll/CostQueryRequest.java	0	0	19
edu/nwu/splitstream/bll/package.html	0	0	1

Details

edu/nwu/scribe/bll/DefaultSessionPolicy.java

Violation	Message	Line
	'{' should be on the previous line.	29
	Parameter outdegreeLimit should be final.	42
	'outdegreeLimit' hides a field.	42
	Parameter usePreemption should be final.	42
	'usePreemption' hides a field.	42
	'{' should be on the previous line.	43
	Method 'getOutdegreeLimit' is not designed for extension - needs to be abstract, final or empty.	55
	Parameter state should be final.	55
	'{' should be on the previous line.	56
	Method 'redirectSubscribe' is not designed for extension - needs to be abstract, final or empty.	69
	Parameter state should be final.	69
	Parameter subscriber should be final.	69
	Parameter outdegree should be final.	70
	'{' should be on the previous line.	71
	Method 'allowSubscribe' is not designed for extension - needs to be abstract, final or empty.	85
	Parameter state should be final.	85
	Parameter preemption should be final.	86
	Parameter subscriber should be final.	86
	Parameter outdegree should be final.	86
	'{' should be on the previous line.	87
	'outdegreeLimit' hides a field.	88
	'{' should be on the previous line.	91
	'{' should be on the previous line.	96
	'}' should be on the same line.	98
	'{' should be on the previous line.	100
	Method 'doPreemption' is not designed for extension - needs to be abstract, final or empty.	117
	Parameter state should be final.	117
	Parameter preemption should be final.	118
	Parameter subscriber should be final.	118
	Parameter outdegree should be final.	118
	'{' should be on the previous line.	119
	'{' should be on the previous line.	125
	'{' should be on the previous line.	127
	'{' should be on the previous line.	134

edu/nwu/scribe/dll/JoinRequest.java

Violation	Message	Line
	'{' should be on the previous line.	35
	'{' should be on the previous line.	63
	Parameter session should be final.	75
	'session' hides a field.	75
	Parameter outdegree should be final.	75
	'outdegree' hides a field.	75
	Parameter force should be final.	75
	'force' hides a field.	75
	'{' should be on the previous line.	76
	Method 'getSessionId' is not designed for extension - needs to be abstract, final or empty.	88
	'{' should be on the previous line.	89
	Method 'getOutdegree' is not designed for extension - needs to be abstract, final or empty.	98
	'{' should be on the previous line.	99
	Method 'getForce' is not designed for extension - needs to be abstract, final or empty.	108
	'{' should be on the previous line.	109
	Unused @throws tag for 'InvalidPacketException'.	117
	Method 'decode' is not designed for extension - needs to be abstract, final or empty.	119
	Parameter in should be final.	119
	Expected @throws tag for 'edu.nwu.net.api.InvalidPacketException'.	120
	'{' should be on the previous line.	121
	Method 'encode' is not designed for extension - needs to be abstract, final or empty.	134
	Parameter out should be final.	134
	'{' should be on the previous line.	135

edu/nwu/scribe/bll/ISessionPolicy.java

Violation	Message	Line
	'{' should be on the previous line.	29
	Redundant 'public' modifier.	37
	Redundant 'public' modifier.	48
	Redundant 'public' modifier.	61

edu/nwu/scribe/bll/package.html

Violation	Message	Line
	Missing package documentation file.	0

edu/nwu/scribe/dll/DataPacket.java

Violation	Message	Line
	'{' should be on the previous line.	38
	'{' should be on the previous line.	81
	Unused @param tag for 'date'.	94
	Parameter session should be final.	96
	'session' hides a field.	96
	Parameter source should be final.	96
	'source' hides a field.	96
	Parameter seqNr should be final.	96
	'seqNr' hides a field.	96
	Parameter timestamp should be final.	97
	'timestamp' hides a field.	97
	Parameter hops should be final.	97
	'hops' hides a field.	97
	Parameter data should be final.	97
	'data' hides a field.	97
	Expected @param tag for 'data'.	97
	'{' should be on the previous line.	98
	Method 'isData' is not designed for extension - needs to be abstract, final or empty.	113
	'{' should be on the previous line.	114
	Method 'getSessionId' is not designed for extension - needs to be abstract, final or empty.	123
	'{' should be on the previous line.	124
	Method 'getSource' is not designed for extension - needs to be abstract, final or empty.	133
	'{' should be on the previous line.	134
	Method 'getSeqNr' is not designed for extension - needs to be abstract, final or empty.	143
	'{' should be on the previous line.	144
	Method 'getSendTimestamp' is not designed for extension - needs to be abstract, final or empty.	153
	'{' should be on the previous line.	154
	Method 'getHops' is not designed for extension - needs to be abstract, final or empty.	163
	'{' should be on the previous line.	164
	Method 'getData' is not designed for extension - needs to be abstract, final or empty.	173
	'{' should be on the previous line.	174
	Unused @throws tag for 'InvalidPacketException'.	182
	Method 'decode' is not designed for extension - needs to be abstract, final or empty.	184
	Parameter in should be final.	184
	Expected @throws tag for 'edu.nwu.net.api.InvalidPacketException'.	185
	'{' should be on the previous line.	186
	Method 'encode' is not designed for extension - needs to be abstract, final or empty.	202
	Parameter out should be final.	202
	'{' should be on the previous line.	203

edu/nwu/scribe/dll/JoinRejected.java

Violation	Message	Line
	'{' should be on the previous line.	37
	'{' should be on the previous line.	77
	Parameter session should be final.	91
	'session' hides a field.	91
	Parameter pathToRoot should be final.	91
	'pathToRoot' hides a field.	91
	Parameter children should be final.	92
	'children' hides a field.	92
	Parameter distances should be final.	92
	'distances' hides a field.	92
	Parameter outdegree should be final.	92
	'outdegree' hides a field.	92
	'{' should be on the previous line.	93
	'{' should be on the previous line.	97
	Method 'getSessionId' is not designed for extension - needs to be abstract, final or empty.	114
	'{' should be on the previous line.	115
	Method 'getPathToRoot' is not designed for extension - needs to be abstract, final or empty.	124
	'{' should be on the previous line.	125
	Method 'getChildren' is not designed for extension - needs to be abstract, final or empty.	134
	'{' should be on the previous line.	135
	Method 'getDistances' is not designed for extension - needs to be abstract, final or empty.	144
	'{' should be on the previous line.	145
	Method 'getOutdegree' is not designed for extension - needs to be abstract, final or empty.	154
	'{' should be on the previous line.	155
	Unused @throws tag for 'InvalidPacketException'.	163
	Method 'decode' is not designed for extension - needs to be abstract, final or empty.	165
	Parameter in should be final.	165
	Expected @throws tag for 'edu.nwu.net.api.InvalidPacketException'.	166
	'{' should be on the previous line.	167
	Method 'encode' is not designed for extension - needs to be abstract, final or empty.	182
	Parameter out should be final.	182
	'{' should be on the previous line.	183

edu/nwu/scribe/bll/IPreemptionProtocol.java

Violation	Message	Line
	'{' should be on the previous line.	29
	Redundant 'public' modifier.	36

edu/nwu/scribe/bll/FatTreeJoinProtocol.java

Violation	Message	Line
	'{' should be on the previous line.	36
	Parameter config should be final.	49
	Parameter sock should be final.	49
	Parameter tsmpFactory should be final.	50
	Parameter agent should be final.	50
	Parameter sessionStateMgr should be final.	51
	'{' should be on the previous line.	52
	Method 'handle' is not designed for extension - needs to be abstract, final or empty.	61
	Parameter req should be final.	61
	'{' should be on the previous line.	62
	'{' should be on the previous line.	66
	'{' should be on the previous line.	76
	'{' should be on the previous line.	80
	'}' should be on the same line.	90
	'{' should be on the previous line.	92
	Parameter session should be final.	104
	'{' should be on the previous line.	105
	'10' is a magic number.	108
	'{' should be on the previous line.	114
	'{' should be on the previous line.	118
	Parameter outdegree should be final.	133
	'{' should be on the previous line.	134
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	143
	'{' should be on the previous line.	144

edu/nwu/freepastry/bll/ScribeFreePastryService.java

Violation	Message	Line
	'{' should be on the previous line.	54
	Line is longer than 80 characters.	56
	Line is longer than 80 characters.	62
	Method 'getAgentId' is not designed for extension - needs to be abstract, final or empty.	81
	'{' should be on the previous line.	82
	Method 'joinSession' is not designed for extension - needs to be abstract, final or empty.	93
	Parameter session should be final.	93
	'{' should be on the previous line.	94
	'{' should be on the previous line.	99
	Inner assignments should be avoided.	101
	Method 'setup' is not designed for extension - needs to be abstract, final or empty.	117
	Parameter socketFactory should be final.	117
	Parameter tsmpFactory should be final.	118
	Parameter epochServiceFactory should be final.	119
	Parameter config should be final.	119
	'config' hides a field.	119
	Parameter bootstrap should be final.	120
	'{' should be on the previous line.	121
	Line is longer than 80 characters.	130
	Method 'tearDown' is not designed for extension - needs to be abstract, final or empty.	136
	'{' should be on the previous line.	137
	'{' should be on the previous line.	149
	'{' should be on the previous line.	153
	'{' should be on the previous line.	159
	'{' should be on the previous line.	167
	Method 'getStateChangedNotifier' is not designed for extension - needs to be abstract, final or empty.	180
	'{' should be on the previous line.	181
	Method 'notifyOnline' is not designed for extension - needs to be abstract, final or empty.	188
	'{' should be on the previous line.	189
	Method 'notifyOffline' is not designed for extension - needs to be abstract, final or empty.	198
	'{' should be on the previous line.	199
	Method 'addSubscriber' is not designed for extension - needs to be abstract, final or empty.	213
	Parameter session should be final.	213
	Parameter subscriber should be final.	213
	'subscriber' hides a field.	213
	'{' should be on the previous line.	214
	'{' should be on the previous line.	218
	'{' should be on the previous line.	226
	Method 'removeSubscriber' is not designed for extension - needs to be abstract, final or empty.	242
	Parameter session should be final.	242
	Parameter subscriber should be final.	242
	'subscriber' hides a field.	242
	Line is longer than 80 characters.	246
	'{' should be on the previous line.	253
	'{' should be on the previous line.	267
	Parameter arg should be final.	273
	'{' should be on the previous line.	274

edu/nwu/scribe/dll/ScribePacketMapping.java

Violation	Message	Line
	'{' should be on the previous line.	27
	'{' should be on the previous line.	32

edu/nwu/scribe/bll/ISessionState.java

Violation	Message	Line
	'{' should be on the previous line.	30
	Redundant 'public' modifier.	36
	Redundant 'public' modifier.	43
	Redundant 'public' modifier.	50
	Redundant 'public' modifier.	57
	First sentence should end with a period.	59
	Redundant 'public' modifier.	66
	Redundant 'public' modifier.	76
	Redundant 'public' modifier.	86
	Redundant 'public' modifier.	95
	Redundant 'public' modifier.	104
	Redundant 'public' modifier.	111
	Redundant 'public' modifier.	119
	Redundant 'public' modifier.	129
	Redundant 'public' modifier.	136
	Redundant 'public' modifier.	143

edu/nwu/splitstream/bll/Configuration.java

Violation	Message	Line
	'{' should be on the previous line.	33
	'4' is a magic number.	38
	Method 'getNumTrees' is not designed for extension - needs to be abstract, final or empty.	45
	'{' should be on the previous line.	46
	Method 'getFQN' is not designed for extension - needs to be abstract, final or empty.	55
	'{' should be on the previous line.	56
	Method 'getCommonPrefix' is not designed for extension - needs to be abstract, final or empty.	74
	'{' should be on the previous line.	75
	Method 'setCommonPrefix' is not designed for extension - needs to be abstract, final or empty.	84
	Parameter prefix should be final.	84
	'{' should be on the previous line.	85

edu/nwu/splitstream/bll/SplitStreamMulticastPublisherSession.java

Violation	Message	Line
	'{' should be on the previous line.	32
	Line is longer than 80 characters.	34
	Parameter sessions should be final.	48
	'sessions' hides a field.	48
	'{' should be on the previous line.	49
	Method 'leaveSession' is not designed for extension - needs to be abstract, final or empty.	58
	'{' should be on the previous line.	59
	'{' should be on the previous line.	61
	Method 'publish' is not designed for extension - needs to be abstract, final or empty.	71
	Parameter data should be final.	71
	'{' should be on the previous line.	72
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	83
	'{' should be on the previous line.	84
	Avoid inline conditionals.	87

edu/nwu/freepastry/bll/package.html

Violation	Message	Line
	Missing package documentation file.	0

edu/nwu/scribe/bll/SessionState.java

Violation	Message	Line
	'{' should be on the previous line.	40
	Parameter session should be final.	78
	'session' hides a field.	78
	Parameter tsmpFactory should be final.	78
	'tsmpFactory' hides a field.	78
	Parameter preemptionProtocol should be final.	79
	'preemptionProtocol' hides a field.	79
	'{' should be on the previous line.	80
	Method 'getSessionId' is not designed for extension - needs to be abstract, final or empty.	91
	'{' should be on the previous line.	92
	Method 'getOutdegreeLimit' is not designed for extension - needs to be abstract, final or empty.	101
	'{' should be on the previous line.	102
	'{' should be on the previous line.	104
	Method 'redirectSubscribe' is not designed for extension - needs to be abstract, final or empty.	119
	Parameter subscriber should be final.	119
	Parameter outdegree should be final.	119
	'{' should be on the previous line.	120
	'{' should be on the previous line.	122
	'}' should be on the same line.	124
	'{' should be on the previous line.	126
	Method 'allowSubscribe' is not designed for extension - needs to be abstract, final or empty.	142
	Parameter subscriber should be final.	142
	Parameter outdegree should be final.	142
	'{' should be on the previous line.	143
	'{' should be on the previous line.	145
	'}' should be on the same line.	148
	'{' should be on the previous line.	150
	Method 'setPolicy' is not designed for extension - needs to be abstract, final or empty.	162
	Parameter policy should be final.	162
	'policy' hides a field.	162
	'{' should be on the previous line.	163
	'{' should be on the previous line.	167
	Method 'getParent' is not designed for extension - needs to be abstract, final or empty.	178
	'{' should be on the previous line.	179
	Method 'getPathToRoot' is not designed for extension - needs to be abstract, final or empty.	188
	'{' should be on the previous line.	189
	Method 'update' is not designed for extension - needs to be abstract, final or empty.	199
	Parameter parent should be final.	199
	'parent' hides a field.	199
	Parameter path should be final.	199
	'{' should be on the previous line.	200
	'{' should be on the previous line.	202
	'}' should be on the same line.	205
	'{' should be on the previous line.	207
	Method 'addChild' is not designed for extension - needs to be abstract, final or empty.	224
	Parameter child should be final.	224
	Parameter outdegree should be final.	224
	'{' should be on the previous line.	225
	'{' should be on the previous line.	227
	'{' should be on the previous line.	232
	'{' should be on the previous line.	242
	Method 'removeChild' is not designed for extension - needs to be abstract, final or empty.	254
	Parameter child should be final.	254
	'{' should be on the previous line.	255
	'{' should be on the previous line.	259
	First sentence should end with a period.	266
	Method 'isChild' is not designed for extension - needs to be abstract, final or empty.	273
	Parameter node should be final.	273
	'{' should be on the previous line.	274
	Method 'getChildren' is not designed for extension - needs to be abstract, final or empty.	283
	'{' should be on the previous line.	284
	Method 'getOutdegree' is not designed for extension - needs to be abstract, final or empty.	295
	Parameter child should be final.	295
	'{' should be on the previous line.	296
	'{' should be on the previous line.	301
	Method 'getTimestamp' is not designed for extension - needs to be abstract, final or empty.	315
	Parameter child should be final.	315
	'{' should be on the previous line.	316
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	325
	'{' should be on the previous line.	326

edu/nwu/freepastry/bll/SplitStreamFreePastryService.java

Violation	Message	Line
	'{' should be on the previous line.	53
	Line is longer than 80 characters.	55
	Line is longer than 80 characters.	61
	Method 'getAgentId' is not designed for extension - needs to be abstract, final or empty.	77
	'{' should be on the previous line.	78
	Method 'joinSession' is not designed for extension - needs to be abstract, final or empty.	89
	Parameter session should be final.	89
	'{' should be on the previous line.	90
	'{' should be on the previous line.	94
	Inner assignments should be avoided.	96
	Method 'setup' is not designed for extension - needs to be abstract, final or empty.	114
	Parameter socketFactory should be final.	114
	Parameter tsmpFactory should be final.	115
	Parameter epochServiceFactory should be final.	116
	Parameter config should be final.	116
	'config' hides a field.	116
	Parameter bootstrap should be final.	117
	'{' should be on the previous line.	118
	Line is longer than 80 characters.	127
	Method 'tearDown' is not designed for extension - needs to be abstract, final or empty.	133
	'{' should be on the previous line.	134
	'{' should be on the previous line.	146
	'{' should be on the previous line.	150
	'{' should be on the previous line.	156
	Method 'getStateChangedNotifier' is not designed for extension - needs to be abstract, final or empty.	167
	'{' should be on the previous line.	168
	Method 'notifyOnline' is not designed for extension - needs to be abstract, final or empty.	175
	'{' should be on the previous line.	176
	Method 'notifyOffline' is not designed for extension - needs to be abstract, final or empty.	185
	'{' should be on the previous line.	186
	Method 'addSubscriber' is not designed for extension - needs to be abstract, final or empty.	200
	Parameter session should be final.	200
	Parameter subscriber should be final.	200
	'subscriber' hides a field.	200
	'{' should be on the previous line.	201
	'{' should be on the previous line.	205
	'{' should be on the previous line.	214
	Method 'removeSubscriber' is not designed for extension - needs to be abstract, final or empty.	230
	Parameter session should be final.	230
	Parameter subscriber should be final.	230
	'subscriber' hides a field.	230
	Line is longer than 80 characters.	234
	'{' should be on the previous line.	237
	'{' should be on the previous line.	250
	Parameter arg should be final.	256
	'{' should be on the previous line.	257

edu/nwu/scribe/dll/CostQueryReply.java

Violation	Message	Line
	'{' should be on the previous line.	36
	'{' should be on the previous line.	62
	Parameter session should be final.	74
	'session' hides a field.	74
	Parameter node should be final.	74
	'node' hides a field.	74
	Parameter cost should be final.	74
	'cost' hides a field.	74
	'{' should be on the previous line.	75
	Method 'getSessionId' is not designed for extension - needs to be abstract, final or empty.	87
	'{' should be on the previous line.	88
	Method 'getNode' is not designed for extension - needs to be abstract, final or empty.	97
	'{' should be on the previous line.	98
	Method 'getCost' is not designed for extension - needs to be abstract, final or empty.	107
	'{' should be on the previous line.	108
	Unused @throws tag for 'InvalidPacketException'.	116
	Method 'decode' is not designed for extension - needs to be abstract, final or empty.	118
	Parameter in should be final.	118
	Expected @throws tag for 'edu.nwu.net.api.InvalidPacketException'.	119
	'{' should be on the previous line.	120
	Method 'encode' is not designed for extension - needs to be abstract, final or empty.	133
	Parameter out should be final.	133
	'{' should be on the previous line.	134

edu/nwu/scribe/bll/Configuration.java

Violation	Message	Line
	'{' should be on the previous line.	33
	'15000' is a magic number.	38
	'60000' is a magic number.	41
	'30' is a magic number.	44
	'15000' is a magic number.	47
	'16' is a magic number.	50
	'10000' is a magic number.	59
	Method 'getFQN' is not designed for extension - needs to be abstract, final or empty.	66
	'{' should be on the previous line.	67
	'{' should be on the previous line.	73
	'{' should be on the previous line.	78
	'}' should be on the same line.	80
	'{' should be on the previous line.	82
	Method 'getTreeHeartbeatInterval' is not designed for extension - needs to be abstract, final or empty.	102
	'{' should be on the previous line.	103
	Method 'setTreeHeartbeatInterval' is not designed for extension - needs to be abstract, final or empty.	112
	Parameter val should be final.	112
	'{' should be on the previous line.	113
	Method 'getTreeMaintenanceInterval' is not designed for extension - needs to be abstract, final or empty.	122
	'{' should be on the previous line.	123
	Method 'setTreeMaintenanceInterval' is not designed for extension - needs to be abstract, final or empty.	132
	Parameter val should be final.	132
	'{' should be on the previous line.	133
	Method 'getMaxHops' is not designed for extension - needs to be abstract, final or empty.	142
	'{' should be on the previous line.	143
	Method 'setMaxHops' is not designed for extension - needs to be abstract, final or empty.	152
	Parameter val should be final.	152
	'{' should be on the previous line.	153
	Method 'getEvaluationInterval' is not designed for extension - needs to be abstract, final or empty.	162
	'{' should be on the previous line.	163
	Method 'setEvaluationInterval' is not designed for extension - needs to be abstract, final or empty.	172
	Parameter val should be final.	172
	'{' should be on the previous line.	173
	Method 'getOutdegreeLimit' is not designed for extension - needs to be abstract, final or empty.	182
	'{' should be on the previous line.	183
	Method 'setOutdegreeLimit' is not designed for extension - needs to be abstract, final or empty.	192
	Parameter val should be final.	192
	'{' should be on the previous line.	193
	Method 'getFatTree' is not designed for extension - needs to be abstract, final or empty.	202
	'{' should be on the previous line.	203
	Method 'setFatTree' is not designed for extension - needs to be abstract, final or empty.	212
	Parameter val should be final.	212
	'{' should be on the previous line.	213
	Method 'getUseOutdegreeEstimation' is not designed for extension - needs to be abstract, final or empty.	222
	'{' should be on the previous line.	223
	Method 'setUseOutdegreeEstimation' is not designed for extension - needs to be abstract, final or empty.	232
	Parameter val should be final.	232
	'{' should be on the previous line.	233
	Method 'getEstimatedDataRateBps' is not designed for extension - needs to be abstract, final or empty.	242
	'{' should be on the previous line.	243
	Method 'setEstimatedDataRateBps' is not designed for extension - needs to be abstract, final or empty.	252
	Parameter val should be final.	252
	'{' should be on the previous line.	253

edu/nwu/scribe/bll/JoinProtocol.java

Violation	Message	Line
	'{' should be on the previous line.	55
	Line has trailing spaces.	60
	Parameter config should be final.	95
	'config' hides a field.	95
	Parameter sock should be final.	95
	'sock' hides a field.	95
	Parameter tsmpFactory should be final.	96
	'tsmpFactory' hides a field.	96
	Parameter agent should be final.	96
	Parameter sessionStateMgr should be final.	97
	'sessionStateMgr' hides a field.	97
	'{' should be on the previous line.	98
	Method 'getSessionState' is not designed for extension - needs to be abstract, final or empty.	116
	Parameter session should be final.	116
	'{' should be on the previous line.	117
	Method 'subscribe' is not designed for extension - needs to be abstract, final or empty.	127
	Parameter session should be final.	127
	Parameter policy should be final.	127
	'{' should be on the previous line.	128
	'{' should be on the previous line.	134
	'{' should be on the previous line.	137
	'}' should be on the same line.	142
	'{' should be on the previous line.	144
	'{' should be on the previous line.	146
	Method 'join' is not designed for extension - needs to be abstract, final or empty.	158
	Parameter session should be final.	158
	Parameter randomized should be final.	158
	'{' should be on the previous line.	159
	'{' should be on the previous line.	165
	'{' should be on the previous line.	168
	'{' should be on the previous line.	178
	Line is longer than 80 characters.	180
	'}' should be on the same line.	185
	'{' should be on the previous line.	187
	Method 'preempt' is not designed for extension - needs to be abstract, final or empty.	198
	Parameter session should be final.	198
	Parameter child should be final.	198
	'{' should be on the previous line.	199
	Parameter session should be final.	215
	Parameter parent should be final.	215
	Parameter force should be final.	215
	'{' should be on the previous line.	216
	'{' should be on the previous line.	220
	'{' is followed by whitespace.	224
	'}' should be on the same line.	227
	'{' should be on the previous line.	229
	Parameter session should be final.	240
	Parameter node should be final.	240
	'{' should be on the previous line.	241
	Method 'redirect' is not designed for extension - needs to be abstract, final or empty.	262
	Parameter session should be final.	262
	Parameter node should be final.	262
	Parameter redirect should be final.	262
	'{' should be on the previous line.	263
	Parameter session should be final.	276
	Parameter node should be final.	276
	'{' should be on the previous line.	277
	Method 'unsubscribe' is not designed for extension - needs to be abstract, final or empty.	294
	Parameter session should be final.	294
	'{' should be on the previous line.	295
	Empty statement.	296
	Method 'getConfig' is not designed for extension - needs to be abstract, final or empty.	304
	'{' should be on the previous line.	305
	Method 'getTimestampFactory' is not designed for extension - needs to be abstract, final or empty.	314
	'{' should be on the previous line.	315
	Method 'epoch' is not designed for extension - needs to be abstract, final or empty.	322
	'{' should be on the previous line.	323
	'{' should be on the previous line.	325
	'{' should be on the previous line.	329
	'{' should be on the previous line.	343
	'{' should be on the previous line.	349
	'{' should be on the previous line.	355
	'{' should be on the previous line.	361
	'{' should be on the previous line.	375
	'{' should be on the previous line.	379
	'{' should be on the previous line.	383
	'{' should be on the previous line.	385
	'{' should be on the previous line.	389
	'}' should be on the same line.	396
	Line is longer than 80 characters.	400
	'{' should be on the previous line.	400
	'{' should be on the previous line.	403
	'}' should be on the same line.	410
	'{' should be on the previous line.	412
	First sentence should end with a period.	419
	Method 'receive' is not designed for extension - needs to be abstract, final or empty.	424
	Parameter msg should be final.	424
	'{' should be on the previous line.	425
	'{' should be on the previous line.	427
	'}' should be on the same line.	429
	'{' should be on the previous line.	431
	'}' should be on the same line.	433
	'{' should be on the previous line.	435
	'}' should be on the same line.	437
	'{' should be on the previous line.	439
	'}' should be on the same line.	441
	'{' should be on the previous line.	443
	'}' should be on the same line.	445
	'{' should be on the previous line.	447
	'}' should be on the same line.	449
	'{' should be on the previous line.	451
	Method 'handle' is not designed for extension - needs to be abstract, final or empty.	461
	Parameter req should be final.	461
	'{' should be on the previous line.	462
	'{' should be on the previous line.	466
	'{' should be on the previous line.	471
	'{' should be on the previous line.	476
	Line is longer than 80 characters.	483
	'}' should be on the same line.	484
	'{' should be on the previous line.	486
	'{' should be on the previous line.	489
	'}' should be on the same line.	496
	'{' should be on the previous line.	498
	'{' should be on the previous line.	501
	'}' should be on the same line.	503
	'{' should be on the previous line.	505
	'{' should be on the previous line.	507
	'}' should be on the same line.	514
	'{' should be on the previous line.	516
	'{' should be on the previous line.	518
	Parameter acc should be final.	532
	'{' should be on the previous line.	533
	'{' should be on the previous line.	538
	'{' should be on the previous line.	540
	'}' should be on the same line.	545
	'{' should be on the previous line.	547
	Line is longer than 80 characters.	548
	Parameter rej should be final.	557
	'{' should be on the previous line.	558
	Line is longer than 80 characters.	561
	'}' should be on the same line.	564
	'{' should be on the previous line.	566
	'{' should be on the previous line.	571
	'{' should be on the previous line.	573
	'{' should be on the previous line.	580
	'}' should be on the same line.	587
	'{' should be on the previous line.	589
	Parameter redir should be final.	602
	'{' should be on the previous line.	603
	'{' should be on the previous line.	608
	'{' should be on the previous line.	610
	Line is longer than 80 characters.	611
	'}' should be on the same line.	615
	'{' should be on the previous line.	617
	Parameter req should be final.	628
	'{' should be on the previous line.	629
	'{' should be on the previous line.	637
	'1000000' is a magic number.	638
	Parameter rply should be final.	652
	'{' should be on the previous line.	653
	'{' should be on the previous line.	658
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	674
	'{' should be on the previous line.	675
	Line is longer than 80 characters.	680
	'{' should be on the previous line.	685
	Parameter candidates should be final.	700
	'candidates' hides a field.	700
	Expected @param tag for 'candidates'.	700
	'{' should be on the previous line.	701
	Method 'getTimestamp' is not designed for extension - needs to be abstract, final or empty.	711
	'{' should be on the previous line.	712
	Method 'isDone' is not designed for extension - needs to be abstract, final or empty.	721
	'{' should be on the previous line.	722
	Method 'mark' is not designed for extension - needs to be abstract, final or empty.	732
	Parameter node should be final.	732
	Parameter cost should be final.	732
	'{' should be on the previous line.	733
	'{' should be on the previous line.	735
	Method 'getMinimum' is not designed for extension - needs to be abstract, final or empty.	745
	'{' should be on the previous line.	746
	'{' should be on the previous line.	750
	'{' should be on the previous line.	755
	'{' should be on the previous line.	761

edu/nwu/freepastry/bll/ScribeMulticastPublisher.java

Violation	Message	Line
	'{' should be on the previous line.	37
	Line is longer than 80 characters.	39
	Parameter scribe should be final.	57
	'scribe' hides a field.	57
	Parameter agentid should be final.	57
	'agentid' hides a field.	57
	Parameter session should be final.	58
	'{' should be on the previous line.	59
	Method 'getAgentId' is not designed for extension - needs to be abstract, final or empty.	73
	'{' should be on the previous line.	74
	Method 'getTopic' is not designed for extension - needs to be abstract, final or empty.	83
	'{' should be on the previous line.	84
	Method 'setAgent' is not designed for extension - needs to be abstract, final or empty.	93
	Parameter agent should be final.	93
	'{' should be on the previous line.	94
	Method 'leaveSession' is not designed for extension - needs to be abstract, final or empty.	101
	'{' should be on the previous line.	102
	Empty statement.	103
	Method 'publish' is not designed for extension - needs to be abstract, final or empty.	111
	Parameter data should be final.	111
	'{' should be on the previous line.	112
	'{' should be on the previous line.	114
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	124
	'{' should be on the previous line.	125

edu/nwu/freepastry/bll/SplitStreamMulticast.java

Violation	Message	Line
	'{' should be on the previous line.	45
	Parameter splitStream should be final.	78
	'splitStream' hides a field.	78
	Parameter agentid should be final.	78
	'agentid' hides a field.	78
	Parameter session should be final.	79
	Parameter subscriber should be final.	79
	'subscriber' hides a field.	79
	'{' should be on the previous line.	80
	Method 'getPublisherSession' is not designed for extension - needs to be abstract, final or empty.	95
	'{' should be on the previous line.	96
	Method 'getAgentId' is not designed for extension - needs to be abstract, final or empty.	107
	'{' should be on the previous line.	108
	Method 'setAgent' is not designed for extension - needs to be abstract, final or empty.	117
	Parameter agent should be final.	117
	'{' should be on the previous line.	118
	'{' should be on the previous line.	128
	'{' should be on the previous line.	130
	Line is longer than 80 characters.	136
	Method 'subscribe' is not designed for extension - needs to be abstract, final or empty.	142
	'{' should be on the previous line.	143
	'{' should be on the previous line.	145
	'{' should be on the previous line.	150
	'{' should be on the previous line.	156
	Method 'tryUnsubscribe' is not designed for extension - needs to be abstract, final or empty.	167
	'{' should be on the previous line.	168
	'{' should be on the previous line.	170
	'{' should be on the previous line.	183
	'{' should be on the previous line.	185
	'{' should be on the previous line.	190
	'{' should be on the previous line.	194
	Method 'leaveSession' is not designed for extension - needs to be abstract, final or empty.	205
	'{' should be on the previous line.	206
	Method 'publish' is not designed for extension - needs to be abstract, final or empty.	216
	Parameter data should be final.	216
	'{' should be on the previous line.	217
	'{' should be on the previous line.	219
	'{' should be on the previous line.	224
	'{' should be on the previous line.	228
	'}' should be on the same line.	234
	'{' should be on the previous line.	236
	'}' should be on the same line.	240
	'{' should be on the previous line.	242
	Line is longer than 80 characters.	248
	Method 'joinFailed' is not designed for extension - needs to be abstract, final or empty.	253
	Parameter s should be final.	253
	'{' should be on the previous line.	254
	'{' should be on the previous line.	258
	First sentence should end with a period.	264
	Line is longer than 80 characters.	265
	Method 'deliver' is not designed for extension - needs to be abstract, final or empty.	270
	Parameter s should be final.	270
	Parameter data should be final.	270
	'{' should be on the previous line.	271
	'{' should be on the previous line.	273
	'{' should be on the previous line.	278
	'{' should be on the previous line.	280
	'}' should be on the same line.	285
	'{' should be on the previous line.	287
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	299
	'{' should be on the previous line.	300
	'{' should be on the previous line.	310
	Missing a Javadoc comment.	311
	'{' should be on the previous line.	317
	'{' should be on the previous line.	319
	Parameter data should be final.	331
	'{' should be on the previous line.	332
	'{' should be on the previous line.	342

edu/nwu/scribe/dll/package.html

Violation	Message	Line
	Missing package documentation file.	0

edu/nwu/scribe/dll/PublishRequest.java

Violation	Message	Line
	'{' should be on the previous line.	38
	'{' should be on the previous line.	81
	Unused @param tag for 'date'.	94
	Parameter session should be final.	96
	'session' hides a field.	96
	Parameter source should be final.	96
	'source' hides a field.	96
	Parameter seqNr should be final.	96
	'seqNr' hides a field.	96
	Parameter timestamp should be final.	97
	'timestamp' hides a field.	97
	Parameter hops should be final.	97
	'hops' hides a field.	97
	Parameter data should be final.	97
	'data' hides a field.	97
	Expected @param tag for 'data'.	97
	'{' should be on the previous line.	98
	Method 'getSessionId' is not designed for extension - needs to be abstract, final or empty.	113
	'{' should be on the previous line.	114
	Method 'getSource' is not designed for extension - needs to be abstract, final or empty.	123
	'{' should be on the previous line.	124
	Method 'getSeqNr' is not designed for extension - needs to be abstract, final or empty.	133
	'{' should be on the previous line.	134
	Method 'getSendTimestamp' is not designed for extension - needs to be abstract, final or empty.	143
	'{' should be on the previous line.	144
	Method 'getHops' is not designed for extension - needs to be abstract, final or empty.	153
	'{' should be on the previous line.	154
	Method 'getData' is not designed for extension - needs to be abstract, final or empty.	163
	'{' should be on the previous line.	164
	Unused @throws tag for 'InvalidPacketException'.	172
	Method 'decode' is not designed for extension - needs to be abstract, final or empty.	174
	Parameter in should be final.	174
	Expected @throws tag for 'edu.nwu.net.api.InvalidPacketException'.	175
	'{' should be on the previous line.	176
	Method 'encode' is not designed for extension - needs to be abstract, final or empty.	192
	Parameter out should be final.	192
	'{' should be on the previous line.	193

edu/nwu/scribe/bll/IScribeClient.java

Violation	Message	Line
	'{' should be on the previous line.	30

edu/nwu/scribe/bll/MulticastProtocol.java

Violation	Message	Line
	Line has trailing spaces.	52
	Parameter config should be final.	89
	'config' hides a field.	89
	Parameter sock should be final.	89
	'sock' hides a field.	89
	Parameter tsmpFactory should be final.	90
	'tsmpFactory' hides a field.	90
	Parameter preemptionProtocol should be final.	90
	'preemptionProtocol' hides a field.	90
	Parameter agent should be final.	91
	Parameter sessionStateMgr should be final.	91
	'sessionStateMgr' hides a field.	91
	Parameter subscriber should be final.	92
	'subscriber' hides a field.	92
	'{' should be on the previous line.	93
	Method 'publish' is not designed for extension - needs to be abstract, final or empty.	110
	Parameter session should be final.	110
	Parameter seqNr should be final.	110
	Parameter data should be final.	110
	'{' should be on the previous line.	111
	'{' should be on the previous line.	116
	'{' should be on the previous line.	121
	'}' should be on the same line.	123
	'{' should be on the previous line.	125
	'{' should be on the previous line.	127
	'100' is a magic number.	144
	'{' should be on the previous line.	145
	Method 'getSessionState' is not designed for extension - needs to be abstract, final or empty.	165
	Parameter session should be final.	165
	'{' should be on the previous line.	166
	Line is longer than 80 characters.	167
	Method 'getConfig' is not designed for extension - needs to be abstract, final or empty.	175
	'{' should be on the previous line.	176
	Method 'getTimestampFactory' is not designed for extension - needs to be abstract, final or empty.	185
	'{' should be on the previous line.	186
	Method 'epoch' is not designed for extension - needs to be abstract, final or empty.	193
	'{' should be on the previous line.	194
	'{' should be on the previous line.	196
	'{' should be on the previous line.	200
	First sentence should end with a period.	206
	Method 'receive' is not designed for extension - needs to be abstract, final or empty.	211
	Parameter msg should be final.	211
	'{' should be on the previous line.	212
	'{' should be on the previous line.	214
	'}' should be on the same line.	216
	'{' should be on the previous line.	218
	'}' should be on the same line.	220
	'{' should be on the previous line.	222
	Parameter pkt should be final.	232
	'{' should be on the previous line.	233
	'{' should be on the previous line.	235
	'{' should be on the previous line.	240
	'{' should be on the previous line.	247
	'{' should be on the previous line.	253
	'}' should be on the same line.	262
	'{' should be on the previous line.	264
	Parameter pkt should be final.	276
	'{' should be on the previous line.	277
	'{' should be on the previous line.	279
	'{' should be on the previous line.	285
	'{' should be on the previous line.	292
	'{' should be on the previous line.	298
	'}' should be on the same line.	307
	'{' should be on the previous line.	309
	'{' should be on the previous line.	312
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	332
	'{' should be on the previous line.	333

edu/nwu/scribe/dll/JoinRedirect.java

Violation	Message	Line
	'{' should be on the previous line.	36
	'{' should be on the previous line.	59
	Parameter session should be final.	70
	'session' hides a field.	70
	Parameter node should be final.	70
	'node' hides a field.	70
	'{' should be on the previous line.	71
	Method 'getSessionId' is not designed for extension - needs to be abstract, final or empty.	82
	'{' should be on the previous line.	83
	Method 'getNode' is not designed for extension - needs to be abstract, final or empty.	92
	'{' should be on the previous line.	93
	Unused @throws tag for 'InvalidPacketException'.	101
	Method 'decode' is not designed for extension - needs to be abstract, final or empty.	103
	Parameter in should be final.	103
	Expected @throws tag for 'edu.nwu.net.api.InvalidPacketException'.	104
	'{' should be on the previous line.	105
	Method 'encode' is not designed for extension - needs to be abstract, final or empty.	117
	Parameter out should be final.	117
	'{' should be on the previous line.	118

edu/nwu/scribe/bll/IJoinProtocol.java

Violation	Message	Line
	'{' should be on the previous line.	28
	Redundant 'public' modifier.	35
	Redundant 'public' modifier.	42

edu/nwu/scribe/bll/ScribePastryService.java

Violation	Message	Line
	'{' should be on the previous line.	60
	Line is longer than 80 characters.	68
	Method 'getSessionPolicy' is not designed for extension - needs to be abstract, final or empty.	90
	'{' should be on the previous line.	91
	Method 'setSessionPolicy' is not designed for extension - needs to be abstract, final or empty.	100
	Parameter policy should be final.	100
	'policy' hides a field.	100
	'{' should be on the previous line.	101
	First sentence should end with a period.	105
	Method 'getName' is not designed for extension - needs to be abstract, final or empty.	110
	'{' should be on the previous line.	111
	'{' should be on the previous line.	117
	Method 'getPredessorSocketAddress' is not designed for extension - needs to be abstract, final or empty.	131
	'{' should be on the previous line.	132
	'{' should be on the previous line.	137
	'{' should be on the previous line.	139
	Method 'getAgentId' is not designed for extension - needs to be abstract, final or empty.	152
	'{' should be on the previous line.	153
	Method 'joinSession' is not designed for extension - needs to be abstract, final or empty.	164
	Parameter session should be final.	164
	'{' should be on the previous line.	165
	Method 'joinSession' is not designed for extension - needs to be abstract, final or empty.	177
	Parameter session should be final.	177
	Parameter seqNrGenerator should be final.	178
	'{' should be on the previous line.	179
	'{' should be on the previous line.	184
	'{' should be on the previous line.	186
	Inner assignments should be avoided.	191
	Method 'getOutdegreeLimit' is not designed for extension - needs to be abstract, final or empty.	206
	Parameter config should be final.	206
	'config' hides a field.	206
	Parameter sock should be final.	206
	'{' should be on the previous line.	207
	'{' should be on the previous line.	211
	Line is longer than 80 characters.	213
	'0.75' is a magic number.	213
	Method 'setup' is not designed for extension - needs to be abstract, final or empty.	229
	Parameter socketFactory should be final.	229
	Parameter tsmpFactory should be final.	230
	Parameter epochServiceFactory should be final.	231
	Parameter config should be final.	231
	'config' hides a field.	231
	Parameter bootstrap should be final.	232
	'{' should be on the previous line.	233
	'{' should be on the previous line.	246
	'{' should be on the previous line.	254
	'}' should be on the same line.	257
	'{' should be on the previous line.	259
	'{' should be on the previous line.	273
	'}' should be on the same line.	275
	'{' should be on the previous line.	277
	Line is longer than 80 characters.	278
	'{' should be on the previous line.	286
	'{' should be on the previous line.	290
	Line is longer than 80 characters.	291
	'{' should be on the previous line.	297
	'{' should be on the previous line.	303
	'{' should be on the previous line.	307
	'{' should be on the previous line.	311
	Method 'tearDown' is not designed for extension - needs to be abstract, final or empty.	321
	'{' should be on the previous line.	322
	Method 'getStateChangedNotifier' is not designed for extension - needs to be abstract, final or empty.	337
	'{' should be on the previous line.	338
	Method 'notifyOnline' is not designed for extension - needs to be abstract, final or empty.	345
	'{' should be on the previous line.	346
	'{' should be on the previous line.	348
	Method 'notifyOffline' is not designed for extension - needs to be abstract, final or empty.	359
	'{' should be on the previous line.	360
	Method 'addSubscriber' is not designed for extension - needs to be abstract, final or empty.	373
	Parameter session should be final.	373
	Parameter subscriber should be final.	373
	'subscriber' hides a field.	373
	'{' should be on the previous line.	374
	'{' should be on the previous line.	376
	Line is longer than 80 characters.	377
	Method 'removeSubscriber' is not designed for extension - needs to be abstract, final or empty.	396
	Parameter session should be final.	396
	Parameter subscriber should be final.	396
	'subscriber' hides a field.	396
	'{' should be on the previous line.	399
	Line is longer than 80 characters.	400
	'{' should be on the previous line.	402
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	417
	'{' should be on the previous line.	418
	'{' should be on the previous line.	428
	Parameter arg should be final.	434
	'{' should be on the previous line.	435

edu/nwu/scribe/dll/EScribePacketType.java

Violation	Message	Line
	First sentence should end with a period.	21
	'{' should be on the previous line.	28
	Line is longer than 80 characters.	30
	'11' is a magic number.	30
	Line is longer than 80 characters.	33
	Line is longer than 80 characters.	37
	Line is longer than 80 characters.	41
	Line is longer than 80 characters.	45
	Line is longer than 80 characters.	49
	Line is longer than 80 characters.	57
	Line is longer than 80 characters.	61
	Parameter name should be final.	70
	Parameter id should be final.	70
	'{' should be on the previous line.	71

edu/nwu/scribe/dll/JoinAccepted.java

Violation	Message	Line
	'{' should be on the previous line.	37
	'{' should be on the previous line.	60
	Parameter session should be final.	71
	'session' hides a field.	71
	Parameter pathToRoot should be final.	71
	'pathToRoot' hides a field.	71
	'{' should be on the previous line.	72
	Method 'getSessionId' is not designed for extension - needs to be abstract, final or empty.	83
	'{' should be on the previous line.	84
	Method 'getPathToRoot' is not designed for extension - needs to be abstract, final or empty.	93
	'{' should be on the previous line.	94
	Unused @throws tag for 'InvalidPacketException'.	102
	Method 'decode' is not designed for extension - needs to be abstract, final or empty.	104
	Parameter in should be final.	104
	Expected @throws tag for 'edu.nwu.net.api.InvalidPacketException'.	105
	'{' should be on the previous line.	106
	Method 'encode' is not designed for extension - needs to be abstract, final or empty.	118
	Parameter out should be final.	118
	'{' should be on the previous line.	119

edu/nwu/freepastry/bll/ScribeFreePastryConfiguration.java

Violation	Message	Line
	'{' should be on the previous line.	28

edu/nwu/freepastry/bll/ScribeMulticastClient.java

Violation	Message	Line
	'{' should be on the previous line.	38
	Parameter scribe should be final.	58
	'scribe' hides a field.	58
	Parameter session should be final.	58
	Parameter subscriber should be final.	59
	'subscriber' hides a field.	59
	'{' should be on the previous line.	60
	Method 'getTopic' is not designed for extension - needs to be abstract, final or empty.	74
	'{' should be on the previous line.	75
	Method 'setAgent' is not designed for extension - needs to be abstract, final or empty.	84
	Parameter agent should be final.	84
	'{' should be on the previous line.	85
	Line is longer than 80 characters.	99
	Method 'anycast' is not designed for extension - needs to be abstract, final or empty.	101
	Parameter topic should be final.	101
	'topic' hides a field.	101
	Parameter content should be final.	101
	'{' should be on the previous line.	102
	'{' should be on the previous line.	104
	Line is longer than 80 characters.	119
	Method 'childAdded' is not designed for extension - needs to be abstract, final or empty.	121
	Parameter topic should be final.	121
	'topic' hides a field.	121
	Parameter child should be final.	121
	'{' should be on the previous line.	122
	'{' should be on the previous line.	124
	Line is longer than 80 characters.	136
	Method 'childRemoved' is not designed for extension - needs to be abstract, final or empty.	138
	Parameter topic should be final.	138
	'topic' hides a field.	138
	Parameter child should be final.	138
	'{' should be on the previous line.	139
	'{' should be on the previous line.	141
	Line is longer than 80 characters.	154
	Method 'deliver' is not designed for extension - needs to be abstract, final or empty.	156
	Parameter topic should be final.	156
	'topic' hides a field.	156
	Parameter content should be final.	156
	'{' should be on the previous line.	157
	'{' should be on the previous line.	159
	'{' should be on the previous line.	163
	'{' should be on the previous line.	168
	Method 'subscribeFailed' is not designed for extension - needs to be abstract, final or empty.	183
	Parameter topic should be final.	183
	'topic' hides a field.	183
	'{' should be on the previous line.	184
	'{' should be on the previous line.	186
	'{' should be on the previous line.	191
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	201
	'{' should be on the previous line.	202

edu/nwu/splitstream/bll/SplitStreamConfiguration.java

Violation	Message	Line
	'{' should be on the previous line.	28

edu/nwu/scribe/bll/MaintenanceProtocol.java

Violation	Message	Line
	'{' should be on the previous line.	40
	Line has trailing spaces.	45
	Parameter config should be final.	78
	'config' hides a field.	78
	Parameter sock should be final.	78
	'sock' hides a field.	78
	Parameter tsmpFactory should be final.	79
	'tsmpFactory' hides a field.	79
	Parameter preemptionProtocol should be final.	79
	'preemptionProtocol' hides a field.	79
	Parameter agent should be final.	80
	Parameter sessionStateMgr should be final.	80
	'sessionStateMgr' hides a field.	80
	'{' should be on the previous line.	81
	Line is longer than 80 characters.	87
	Parameter sock should be final.	99
	Parameter nodes should be final.	99
	'{' should be on the previous line.	100
	'{' should be on the previous line.	104
	Method 'getConfig' is not designed for extension - needs to be abstract, final or empty.	117
	'{' should be on the previous line.	118
	Method 'getTimestampFactory' is not designed for extension - needs to be abstract, final or empty.	127
	'{' should be on the previous line.	128
	Method 'epoch' is not designed for extension - needs to be abstract, final or empty.	135
	'{' should be on the previous line.	136
	'{' should be on the previous line.	138
	'{' should be on the previous line.	142
	'{' should be on the previous line.	154
	'{' should be on the previous line.	160
	'{' should be on the previous line.	165
	'{' should be on the previous line.	167
	'{' should be on the previous line.	173
	'{' should be on the previous line.	175
	'}' should be on the same line.	182
	'{' should be on the previous line.	184
	First sentence should end with a period.	191
	Method 'receive' is not designed for extension - needs to be abstract, final or empty.	196
	Parameter msg should be final.	196
	'{' should be on the previous line.	197
	'{' should be on the previous line.	199
	'}' should be on the same line.	201
	'{' should be on the previous line.	203
	'}' should be on the same line.	205
	'{' should be on the previous line.	207
	Parameter req should be final.	217
	'{' should be on the previous line.	218
	Line is longer than 80 characters.	219
	'{' should be on the previous line.	223
	'{' should be on the previous line.	226
	'}' should be on the same line.	232
	'{' should be on the previous line.	234
	'{' should be on the previous line.	236
	Parameter req should be final.	256
	'{' should be on the previous line.	257
	Line is longer than 80 characters.	258
	'{' should be on the previous line.	263
	'{' should be on the previous line.	265
	Line is longer than 80 characters.	266
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	279
	'{' should be on the previous line.	280

edu/nwu/scribe/bll/SessionStateManager.java

Violation	Message	Line
	'{' should be on the previous line.	35
	Parameter tsmpFactory should be final.	50
	'tsmpFactory' hides a field.	50
	'{' should be on the previous line.	51
	Method 'getTimestampFactory' is not designed for extension - needs to be abstract, final or empty.	60
	'{' should be on the previous line.	61
	Method 'getAllSessionState' is not designed for extension - needs to be abstract, final or empty.	70
	'{' should be on the previous line.	71
	Method 'getSessionState' is not designed for extension - needs to be abstract, final or empty.	83
	Parameter session should be final.	83
	Parameter preemptionProtocol should be final.	84
	'{' should be on the previous line.	85
	'{' should be on the previous line.	89
	Inner assignments should be avoided.	91
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	103
	'{' should be on the previous line.	104

edu/nwu/scribe/bll/ScribeConfiguration.java

Violation	Message	Line
	'{' should be on the previous line.	28

edu/nwu/scribe/bll/ISessionStateManager.java

Violation	Message	Line
	'{' should be on the previous line.	28
	Redundant 'public' modifier.	34
	Redundant 'public' modifier.	44

edu/nwu/splitstream/bll/SplitStreamSessionPolicy.java

Violation	Message	Line
	'{' should be on the previous line.	38
	Line is longer than 80 characters.	40
	Parameter self should be final.	55
	'self' hides a field.	55
	Parameter config should be final.	55
	'config' hides a field.	55
	Parameter outdegreeLimit should be final.	56
	'{' should be on the previous line.	57
	Method 'redirectSubscribe' is not designed for extension - needs to be abstract, final or empty.	72
	Parameter state should be final.	72
	Parameter subscriber should be final.	72
	Parameter outdegree should be final.	73
	'{' should be on the previous line.	74
	'{' should be on the previous line.	78
	'{' should be on the previous line.	80
	Method 'allowSubscribe' is not designed for extension - needs to be abstract, final or empty.	103
	Parameter state should be final.	103
	Parameter preemption should be final.	104
	Parameter subscriber should be final.	104
	Parameter outdegree should be final.	104
	'{' should be on the previous line.	105
	Comment matches to-do format 'TODO:'.	106
	'{' should be on the previous line.	110
	'{' should be on the previous line.	112
	Method 'doPreemption' is not designed for extension - needs to be abstract, final or empty.	134
	Parameter state should be final.	134
	Parameter preemption should be final.	135
	Parameter subscriber should be final.	135
	Parameter outdegree should be final.	135
	'{' should be on the previous line.	136
	'{' should be on the previous line.	143
	Line is longer than 80 characters.	144
	Line is longer than 80 characters.	146
	'{' should be on the previous line.	153
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	167
	'{' should be on the previous line.	168

edu/nwu/freepastry/bll/SplitStreamFreePastryConfiguration.java

Violation	Message	Line
	'{' should be on the previous line.	28

edu/nwu/splitstream/bll/SplitStreamPastryService.java

Violation	Message	Line
	'{' should be on the previous line.	44
	Line is longer than 80 characters.	47
	Method 'joinSession' is not designed for extension - needs to be abstract, final or empty.	59
	Parameter session should be final.	59
	'{' should be on the previous line.	60
	Line is longer than 80 characters.	62
	'{' should be on the previous line.	66
	Method 'setup' is not designed for extension - needs to be abstract, final or empty.	82
	Parameter socketFactory should be final.	82
	Parameter tsmpFactory should be final.	83
	Parameter epochServiceFactory should be final.	84
	Parameter config should be final.	84
	'config' hides a field.	84
	Parameter bootstrap should be final.	85
	'{' should be on the previous line.	86
	Method 'tearDown' is not designed for extension - needs to be abstract, final or empty.	100
	'{' should be on the previous line.	101
	Method 'addSubscriber' is not designed for extension - needs to be abstract, final or empty.	113
	Parameter session should be final.	113
	Parameter subscriber should be final.	113
	'{' should be on the previous line.	114
	'{' should be on the previous line.	119
	Method 'removeSubscriber' is not designed for extension - needs to be abstract, final or empty.	134
	Parameter session should be final.	134
	Parameter subscriber should be final.	134
	'{' should be on the previous line.	140
	Method 'buildSessionIdentifiers' is not designed for extension - needs to be abstract, final or empty.	154
	Parameter session should be final.	154
	'{' should be on the previous line.	155
	Method 'toString' is not designed for extension - needs to be abstract, final or empty.	164
	'{' should be on the previous line.	165

edu/nwu/scribe/dll/CostQueryRequest.java

Violation	Message	Line
	'{' should be on the previous line.	36
	'{' should be on the previous line.	58
	Parameter session should be final.	69
	'session' hides a field.	69
	Parameter node should be final.	69
	'node' hides a field.	69
	'{' should be on the previous line.	70
	Method 'getSessionId' is not designed for extension - needs to be abstract, final or empty.	81
	'{' should be on the previous line.	82
	Method 'getNode' is not designed for extension - needs to be abstract, final or empty.	91
	'{' should be on the previous line.	92
	Unused @throws tag for 'InvalidPacketException'.	100
	Method 'decode' is not designed for extension - needs to be abstract, final or empty.	102
	Parameter in should be final.	102
	Expected @throws tag for 'edu.nwu.net.api.InvalidPacketException'.	103
	'{' should be on the previous line.	104
	Method 'encode' is not designed for extension - needs to be abstract, final or empty.	116
	Parameter out should be final.	116
	'{' should be on the previous line.	117

edu/nwu/splitstream/bll/package.html

Violation	Message	Line
	Missing package documentation file.	0

Nemo

Project Documentation

Checkstyle Results

Summary

Rules

Files

Details

edu/nwu/scribe/bll/DefaultSessionPolicy.java

edu/nwu/scribe/dll/JoinRequest.java

edu/nwu/scribe/bll/ISessionPolicy.java

edu/nwu/scribe/bll/package.html

edu/nwu/scribe/dll/DataPacket.java

edu/nwu/scribe/dll/JoinRejected.java

edu/nwu/scribe/bll/IPreemptionProtocol.java

edu/nwu/scribe/bll/FatTreeJoinProtocol.java

edu/nwu/freepastry/bll/ScribeFreePastryService.java

edu/nwu/scribe/dll/ScribePacketMapping.java

edu/nwu/scribe/bll/ISessionState.java

edu/nwu/splitstream/bll/Configuration.java

edu/nwu/splitstream/bll/SplitStreamMulticastPublisherSession.java

edu/nwu/freepastry/bll/package.html

edu/nwu/scribe/bll/SessionState.java

edu/nwu/freepastry/bll/SplitStreamFreePastryService.java

edu/nwu/scribe/dll/CostQueryReply.java

edu/nwu/scribe/bll/Configuration.java

edu/nwu/scribe/bll/JoinProtocol.java

edu/nwu/freepastry/bll/ScribeMulticastPublisher.java

edu/nwu/freepastry/bll/SplitStreamMulticast.java

edu/nwu/scribe/dll/package.html

edu/nwu/scribe/dll/PublishRequest.java

edu/nwu/scribe/bll/IScribeClient.java

edu/nwu/scribe/bll/MulticastProtocol.java

edu/nwu/scribe/dll/JoinRedirect.java

edu/nwu/scribe/bll/IJoinProtocol.java

edu/nwu/scribe/bll/ScribePastryService.java

edu/nwu/scribe/dll/EScribePacketType.java

edu/nwu/scribe/dll/JoinAccepted.java

edu/nwu/freepastry/bll/ScribeFreePastryConfiguration.java

edu/nwu/freepastry/bll/ScribeMulticastClient.java

edu/nwu/splitstream/bll/SplitStreamConfiguration.java

edu/nwu/scribe/bll/MaintenanceProtocol.java

edu/nwu/scribe/bll/SessionStateManager.java

edu/nwu/scribe/bll/ScribeConfiguration.java

edu/nwu/scribe/bll/ISessionStateManager.java

edu/nwu/splitstream/bll/SplitStreamSessionPolicy.java

edu/nwu/freepastry/bll/SplitStreamFreePastryConfiguration.java

edu/nwu/splitstream/bll/SplitStreamPastryService.java

edu/nwu/scribe/dll/CostQueryRequest.java

edu/nwu/splitstream/bll/package.html