Checkstyle Results

The following document contains the results of Checkstyle 9.3 with config/maven_checks.xml ruleset. 

Summary

Files	Info	Warnings	Errors
14	3	4	175

Files

File	I	W	E
org/codehaus/mojo/exec/EnvStreamConsumer.java	1	0	5
org/codehaus/mojo/exec/ExecJavaMojo.java	0	0	15
org/codehaus/mojo/exec/ExecMojo.java	2	1	50
org/codehaus/mojo/exec/ExecutableDependency.java	0	0	1
org/codehaus/mojo/exec/LineRedirectOutputStream.java	0	0	20
org/codehaus/mojo/exec/PathsToolchain.java	0	0	1
org/codehaus/mojo/exec/PathsToolchainFactory.java	0	0	9
org/codehaus/mojo/exec/URLClassLoaderBuilder.java	0	3	74

Rules

Category	Rule	Violations	Severity
blocks	LeftCurly option: "nl"	15	Error
	NeedBraces	15	Error
	RightCurly option: "alone"	1	Error
coding	HiddenField ignoreConstructorParameter: "true" ignoreSetter: "true"	4	Warning
	MagicNumber ignoreNumbers: "-4, -3, -2, -1, 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 31, 32, 37, 64, 100, 128, 256, 512, 1000, 1024"	1	Error
header	RegexpHeader fileExtensions: "java" headerFile: "target/checkstyle-header.txt"	2	Error
javadoc	JavadocVariable scope: "protected"	3	Info
modifier	RedundantModifier	4	Error
naming	LocalFinalVariableName	1	Error
sizes	LineLength max: "120" ignorePattern: "@version|@see|@todo|TODO"	5	Error
	MethodLength	2	Error
whitespace	OperatorWrap	2	Error
	ParenPad option: "space"	125	Error
	WhitespaceAfter	2	Error

Details

org/codehaus/mojo/exec/EnvStreamConsumer.java

Severity	Category	Rule	Message	Line
Error	header	RegexpHeader	Line does not match expected header line of '^/\*\s*$'.	3
Info	javadoc	JavadocVariable	Missing a Javadoc comment.	15
Error	blocks	RightCurly	'}' at column 13 should be alone on a line.	39
Error	blocks	LeftCurly	'{' at column 20 should be on a new line.	39
Error	whitespace	ParenPad	'(' is not followed by whitespace.	42
Error	whitespace	ParenPad	')' is not preceded with whitespace.	42

org/codehaus/mojo/exec/ExecJavaMojo.java

Severity	Category	Rule	Message	Line
Error	header	RegexpHeader	Line does not match expected header line of '^/\*\s*$'.	3
Error	sizes	LineLength	Line is longer than 120 characters (found 178).	284
Error	sizes	LineLength	Line is longer than 120 characters (found 122).	290
Error	sizes	LineLength	Line is longer than 120 characters (found 125).	308
Error	whitespace	ParenPad	'(' is not followed by whitespace.	335
Error	whitespace	ParenPad	')' is not preceded with whitespace.	335
Error	whitespace	OperatorWrap	':' should be on a new line.	366
Error	whitespace	ParenPad	'(' is not followed by whitespace.	373
Error	blocks	LeftCurly	'{' at column 33 should be on a new line.	373
Error	whitespace	ParenPad	')' is not preceded with whitespace.	382
Error	whitespace	ParenPad	'(' is not followed by whitespace.	386
Error	whitespace	ParenPad	')' is not preceded with whitespace.	386
Error	whitespace	ParenPad	'(' is not followed by whitespace.	388
Error	whitespace	ParenPad	')' is not preceded with whitespace.	388
Error	modifier	RedundantModifier	Redundant 'public' modifier.	400

org/codehaus/mojo/exec/ExecMojo.java

Severity	Category	Rule	Message	Line
Info	javadoc	JavadocVariable	Missing a Javadoc comment.	330
Info	javadoc	JavadocVariable	Missing a Javadoc comment.	332
Error	sizes	MethodLength	Method execute length is 167 lines (max allowed is 150).	339
Error	whitespace	ParenPad	'(' is not followed by whitespace.	344
Error	whitespace	ParenPad	')' is not preceded with whitespace.	344
Error	whitespace	ParenPad	')' is not preceded with whitespace.	350
Error	sizes	LineLength	Line is longer than 120 characters (found 132).	407
Error	whitespace	ParenPad	'(' is not followed by whitespace.	407
Error	whitespace	ParenPad	')' is not preceded with whitespace.	407
Error	whitespace	ParenPad	'(' is not followed by whitespace.	433
Error	whitespace	ParenPad	')' is not preceded with whitespace.	433
Error	whitespace	ParenPad	'(' is not followed by whitespace.	435
Error	whitespace	ParenPad	')' is not preceded with whitespace.	435
Error	whitespace	ParenPad	'(' is not followed by whitespace.	442
Error	whitespace	ParenPad	')' is not preceded with whitespace.	442
Error	whitespace	ParenPad	'(' is not followed by whitespace.	444
Error	whitespace	ParenPad	')' is not preceded with whitespace.	444
Error	whitespace	ParenPad	'(' is not followed by whitespace.	446
Error	whitespace	ParenPad	')' is not preceded with whitespace.	446
Error	whitespace	ParenPad	'(' is not followed by whitespace.	450
Error	whitespace	ParenPad	')' is not preceded with whitespace.	450
Error	whitespace	ParenPad	'(' is not followed by whitespace.	458
Error	whitespace	ParenPad	')' is not preceded with whitespace.	458
Error	whitespace	ParenPad	'(' is not followed by whitespace.	460
Error	whitespace	ParenPad	')' is not preceded with whitespace.	460
Error	whitespace	ParenPad	'(' is not followed by whitespace.	464
Error	whitespace	ParenPad	'(' is not followed by whitespace.	464
Error	whitespace	ParenPad	')' is not preceded with whitespace.	464
Error	whitespace	ParenPad	'(' is not followed by whitespace.	465
Error	whitespace	ParenPad	')' is not preceded with whitespace.	465
Error	whitespace	ParenPad	')' is not preceded with whitespace.	465
Error	blocks	LeftCurly	'{' at column 96 should be on a new line.	465
Error	whitespace	ParenPad	'(' is not followed by whitespace.	466
Error	whitespace	ParenPad	')' is not preceded with whitespace.	466
Error	sizes	LineLength	Line is longer than 120 characters (found 151).	689
Error	whitespace	ParenPad	'(' is not followed by whitespace.	689
Error	whitespace	ParenPad	')' is not preceded with whitespace.	689
Error	whitespace	ParenPad	'(' is not followed by whitespace.	689
Error	whitespace	ParenPad	')' is not preceded with whitespace.	689
Error	whitespace	ParenPad	'(' is not followed by whitespace.	693
Error	whitespace	ParenPad	')' is not preceded with whitespace.	693
Error	blocks	NeedBraces	'if' construct must use '{}'s.	887
Error	blocks	NeedBraces	'else' construct must use '{}'s.	889
Error	blocks	NeedBraces	'if' construct must use '{}'s.	893
Error	blocks	NeedBraces	'if' construct must use '{}'s.	898
Error	blocks	NeedBraces	'for' construct must use '{}'s.	913
Error	blocks	NeedBraces	'if' construct must use '{}'s.	914
Warning	coding	HiddenField	'outputFile' hides a field.	951
Error	naming	LocalFinalVariableName	Name 'EOL' must match pattern '^[a-z][a-zA-Z0-9]*$'.	1138
Error	whitespace	WhitespaceAfter	';' is not followed by whitespace.	1166
Error	whitespace	WhitespaceAfter	'if' is not followed by whitespace.	1188
Error	whitespace	ParenPad	'(' is not followed by whitespace.	1188
Error	whitespace	ParenPad	')' is not preceded with whitespace.	1188

org/codehaus/mojo/exec/ExecutableDependency.java

Severity	Category	Rule	Message	Line
Error	coding	MagicNumber	'29' is a magic number.	134

org/codehaus/mojo/exec/LineRedirectOutputStream.java

Severity	Category	Rule	Message	Line
Error	blocks	LeftCurly	'{' at column 53 should be on a new line.	33
Error	modifier	RedundantModifier	Redundant 'public' modifier.	38
Error	whitespace	ParenPad	'(' is not followed by whitespace.	38
Error	whitespace	ParenPad	')' is not preceded with whitespace.	38
Error	blocks	LeftCurly	'{' at column 67 should be on a new line.	38
Error	whitespace	ParenPad	'(' is not followed by whitespace.	43
Error	whitespace	ParenPad	')' is not preceded with whitespace.	43
Error	blocks	LeftCurly	'{' at column 36 should be on a new line.	43
Error	whitespace	ParenPad	'(' is not followed by whitespace.	44
Error	whitespace	ParenPad	')' is not preceded with whitespace.	44
Error	blocks	LeftCurly	'{' at column 31 should be on a new line.	44
Error	whitespace	ParenPad	'(' is not followed by whitespace.	48
Error	whitespace	ParenPad	')' is not preceded with whitespace.	48
Error	blocks	LeftCurly	'{' at column 25 should be on a new line.	52
Error	whitespace	ParenPad	'(' is not followed by whitespace.	53
Error	whitespace	ParenPad	')' is not preceded with whitespace.	53
Error	blocks	LeftCurly	'{' at column 39 should be on a new line.	53
Error	blocks	LeftCurly	'{' at column 34 should be on a new line.	58
Error	whitespace	ParenPad	'(' is not followed by whitespace.	59
Error	whitespace	ParenPad	')' is not preceded with whitespace.	59

org/codehaus/mojo/exec/PathsToolchain.java

Severity	Category	Rule	Message	Line
Error	modifier	RedundantModifier	Redundant 'public' modifier.	38

org/codehaus/mojo/exec/PathsToolchainFactory.java

Severity	Category	Rule	Message	Line
Error	blocks	NeedBraces	'if' construct must use '{}'s.	59
Error	blocks	NeedBraces	'if' construct must use '{}'s.	68
Error	blocks	NeedBraces	'if' construct must use '{}'s.	76
Error	blocks	NeedBraces	'if' construct must use '{}'s.	80
Error	blocks	NeedBraces	'if' construct must use '{}'s.	84
Error	blocks	NeedBraces	'if' construct must use '{}'s.	92
Error	blocks	NeedBraces	'if' construct must use '{}'s.	97
Error	blocks	NeedBraces	'for' construct must use '{}'s.	125
Error	blocks	NeedBraces	'if' construct must use '{}'s.	151

org/codehaus/mojo/exec/URLClassLoaderBuilder.java

Severity	Category	Rule	Message	Line
Error	whitespace	ParenPad	'(' is not followed by whitespace.	60
Warning	coding	HiddenField	'logger' hides a field.	60
Error	whitespace	ParenPad	'(' is not followed by whitespace.	66
Warning	coding	HiddenField	'exclusions' hides a field.	66
Error	whitespace	ParenPad	'(' is not followed by whitespace.	72
Warning	coding	HiddenField	'paths' hides a field.	72
Error	whitespace	ParenPad	'(' is not followed by whitespace.	88
Error	modifier	RedundantModifier	Redundant 'public' modifier.	122
Error	whitespace	ParenPad	'(' is not followed by whitespace.	122
Error	whitespace	ParenPad	'(' is not followed by whitespace.	124
Error	whitespace	ParenPad	')' is not preceded with whitespace.	124
Error	whitespace	ParenPad	'(' is not followed by whitespace.	129
Error	whitespace	ParenPad	')' is not preceded with whitespace.	129
Error	blocks	LeftCurly	'{' at column 107 should be on a new line.	129
Error	whitespace	ParenPad	'(' is not followed by whitespace.	130
Error	whitespace	ParenPad	')' is not preceded with whitespace.	130
Error	blocks	LeftCurly	'{' at column 31 should be on a new line.	130
Error	whitespace	ParenPad	'(' is not followed by whitespace.	166
Error	whitespace	ParenPad	')' is not preceded with whitespace.	166
Error	whitespace	ParenPad	'(' is not followed by whitespace.	182
Error	whitespace	ParenPad	')' is not preceded with whitespace.	182
Error	blocks	LeftCurly	'{' at column 37 should be on a new line.	182
Error	whitespace	ParenPad	'(' is not followed by whitespace.	198
Error	whitespace	ParenPad	')' is not preceded with whitespace.	198
Error	whitespace	ParenPad	'(' is not followed by whitespace.	205
Error	whitespace	ParenPad	')' is not preceded with whitespace.	205
Error	whitespace	ParenPad	'(' is not followed by whitespace.	206
Error	whitespace	ParenPad	')' is not preceded with whitespace.	206
Error	whitespace	ParenPad	'(' is not followed by whitespace.	207
Error	whitespace	ParenPad	')' is not preceded with whitespace.	207
Error	whitespace	ParenPad	'(' is not followed by whitespace.	211
Error	whitespace	ParenPad	')' is not preceded with whitespace.	211
Error	whitespace	ParenPad	'(' is not followed by whitespace.	213
Error	whitespace	ParenPad	')' is not preceded with whitespace.	213
Error	whitespace	ParenPad	'(' is not followed by whitespace.	216
Error	whitespace	ParenPad	')' is not preceded with whitespace.	216
Error	whitespace	ParenPad	')' is not preceded with whitespace.	216
Error	whitespace	ParenPad	'(' is not followed by whitespace.	221
Error	whitespace	ParenPad	')' is not preceded with whitespace.	221
Error	whitespace	ParenPad	'(' is not followed by whitespace.	222
Error	whitespace	ParenPad	')' is not preceded with whitespace.	222
Error	whitespace	ParenPad	'(' is not followed by whitespace.	226
Error	whitespace	ParenPad	')' is not preceded with whitespace.	226
Error	whitespace	ParenPad	'(' is not followed by whitespace.	231
Error	whitespace	ParenPad	')' is not preceded with whitespace.	231
Error	whitespace	OperatorWrap	'&&' should be on a new line.	232
Error	whitespace	ParenPad	'(' is not followed by whitespace.	233
Error	whitespace	ParenPad	'(' is not followed by whitespace.	240
Error	whitespace	ParenPad	')' is not preceded with whitespace.	240
Error	whitespace	ParenPad	')' is not preceded with whitespace.	242
Error	blocks	LeftCurly	'{' at column 44 should be on a new line.	248
Error	whitespace	ParenPad	'(' is not followed by whitespace.	261
Error	whitespace	ParenPad	')' is not preceded with whitespace.	261
Error	whitespace	ParenPad	'(' is not followed by whitespace.	265
Error	whitespace	ParenPad	')' is not preceded with whitespace.	265
Error	whitespace	ParenPad	'(' is not followed by whitespace.	295
Error	whitespace	ParenPad	'(' is not followed by whitespace.	300
Error	whitespace	ParenPad	')' is not preceded with whitespace.	300
Error	whitespace	ParenPad	'(' is not followed by whitespace.	308
Error	whitespace	ParenPad	')' is not preceded with whitespace.	308
Error	whitespace	ParenPad	'(' is not followed by whitespace.	328
Error	whitespace	ParenPad	')' is not preceded with whitespace.	328
Error	sizes	MethodLength	Method isDirectJvmClass length is 164 lines (max allowed is 150).	338
Error	whitespace	ParenPad	'(' is not followed by whitespace.	338
Error	whitespace	ParenPad	')' is not preceded with whitespace.	338
Error	blocks	LeftCurly	'{' at column 61 should be on a new line.	338
Error	whitespace	ParenPad	'(' is not followed by whitespace.	339
Error	whitespace	ParenPad	'(' is not followed by whitespace.	516
Error	whitespace	ParenPad	')' is not preceded with whitespace.	516
Error	whitespace	ParenPad	'(' is not followed by whitespace.	519
Error	whitespace	ParenPad	'(' is not followed by whitespace.	519
Error	whitespace	ParenPad	')' is not preceded with whitespace.	519
Error	whitespace	ParenPad	')' is not preceded with whitespace.	519
Error	whitespace	ParenPad	'(' is not followed by whitespace.	547
Error	whitespace	ParenPad	')' is not preceded with whitespace.	547
Error	whitespace	ParenPad	'(' is not followed by whitespace.	549
Error	whitespace	ParenPad	')' is not preceded with whitespace.	549