Checkstyle Results

The following document contains the results of Checkstyle. 

Summary

Files	Infos	Warnings	Errors
18	0	0	175

Files

Files	I	W	E
org/apache/myfaces/commons/converter/AbstractTypedNumberConverter.java	0	0	28
org/apache/myfaces/commons/converter/BooleanConverter.java	0	0	1
org/apache/myfaces/commons/converter/ConvertDateTimeTagHandler.java	0	0	2
org/apache/myfaces/commons/converter/ConvertEnumTagHandler.java	0	0	3
org/apache/myfaces/commons/converter/ConverterBase.java	0	0	22
org/apache/myfaces/commons/converter/ConverterBaseTag.java	0	0	2
org/apache/myfaces/commons/converter/ConverterBaseTagHandler.java	0	0	1
org/apache/myfaces/commons/converter/ConverterTag.java	0	0	2
org/apache/myfaces/commons/converter/DateTimeConverter.java	0	0	19
org/apache/myfaces/commons/converter/EnumConverter.java	0	0	46
org/apache/myfaces/commons/converter/TypedNumberConverterTagHandler.java	0	0	2
org/apache/myfaces/commons/converter/_AttachedListStateWrapper.java	0	0	1
org/apache/myfaces/commons/converter/_AttachedStateWrapper.java	0	0	2
org/apache/myfaces/commons/converter/_ConverterRule.java	0	0	11
org/apache/myfaces/commons/converter/_DateTimeConverterRule.java	0	0	11
org/apache/myfaces/commons/converter/_DeltaStateHelper.java	0	0	2
org/apache/myfaces/commons/converter/_LocaleRule.java	0	0	10
org/apache/myfaces/commons/converter/_TimeZoneRule.java	0	0	10

Rules

Rules	Violations	Severity
ConstantName format: "^_?((log)|([a-z][a-zA-Z]*ThreadLocal)|([A-Z][A-Z0-9]*(_[A-Z0-9]+)*))$"	4	Error
LocalVariableName	2	Error
MethodName format: "^_?[a-z][a-zA-Z0-9]*$"	0	Error
PackageName	0	Error
LocalFinalVariableName	0	Error
ParameterName	0	Error
StaticVariableName	0	Error
TypeName format: "^_?[A-Z][a-zA-Z0-9]*$"	0	Error
Header headerFile: "/home/lu4242/workspace/trunk/commons/jsf_20/myfaces-commons-converters/target/checkstyle-header.txt"	18	Error
AvoidStarImport	0	Error
IllegalImport	0	Error
RedundantImport	0	Error
UnusedImports	3	Error
FileLength	0	Error
LineLength max: "120" ignorePattern: "@version|@see"	22	Error
MethodLength max: "200"	0	Error
ParameterNumber max: "10"	0	Error
TabCharacter	0	Error
NeedBraces	38	Error
LeftCurly option: "nl"	76	Error
RightCurly option: "alone"	8	Error
DoubleCheckedLocking	0	Error
EmptyStatement	1	Error
EqualsHashCode	0	Error
InnerAssignment	0	Error
DefaultComesLast	0	Error
MissingSwitchDefault	0	Error
FallThrough	2	Error
MultipleVariableDeclarations	1	Error
UpperEll	0	Error

Details

org/apache/myfaces/commons/converter/AbstractTypedNumberConverter.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	Unused import - javax.faces.FacesException.	30
	Unused import - javax.faces.component.StateHolder.	32
	'if' construct must use '{}'s.	145
	'if' construct must use '{}'s.	146
	'if' construct must use '{}'s.	206
	Line is longer than 120 characters.	207
	Line is longer than 120 characters.	209
	'if' construct must use '{}'s.	210
	Line is longer than 120 characters.	211
	Line is longer than 120 characters.	213
	'if' construct must use '{}'s.	214
	Line is longer than 120 characters.	215
	Line is longer than 120 characters.	217
	'if' construct must use '{}'s.	218
	Line is longer than 120 characters.	219
	Line is longer than 120 characters.	221
	'if' construct must use '{}'s.	231
	'if' construct must use '{}'s.	232
	'if' construct must use '{}'s.	249
	'if' construct must use '{}'s.	250
	'if' construct must use '{}'s.	251
	'if' construct must use '{}'s.	252
	Line is longer than 120 characters.	260
	Each variable declaration must be in its own statement.	497
	Fall through from previous branch of the switch statement.	516
	Empty statement.	521
	Fall through from previous branch of the switch statement.	522

org/apache/myfaces/commons/converter/BooleanConverter.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2

org/apache/myfaces/commons/converter/ConvertDateTimeTagHandler.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	Unused import - org.apache.myfaces.buildtools.maven2.plugin.builder.annotation.JSFConverter.	26

org/apache/myfaces/commons/converter/ConvertEnumTagHandler.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	Line is longer than 120 characters.	54
	Line is longer than 120 characters.	58

org/apache/myfaces/commons/converter/ConverterBase.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	'{' should be on a new line.	48
	'{' should be on a new line.	71
	'{' should be on a new line.	82
	'{' should be on a new line.	90
	'{' should be on a new line.	121
	'{' should be on a new line.	125
	'{' should be on a new line.	136
	'}' should be alone on a line.	142
	'{' should be on a new line.	142
	'if' construct must use '{}'s.	156
	'if' construct must use '{}'s.	175
	'{' should be on a new line.	176
	'}' should be alone on a line.	178
	'{' should be on a new line.	178
	'if' construct must use '{}'s.	215
	'if' construct must use '{}'s.	255
	'if' construct must use '{}'s.	257
	'{' should be on a new line.	324
	'{' should be on a new line.	334
	'{' should be on a new line.	335
	'{' should be on a new line.	338

org/apache/myfaces/commons/converter/ConverterBaseTag.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	'{' should be on a new line.	34

org/apache/myfaces/commons/converter/ConverterBaseTagHandler.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2

org/apache/myfaces/commons/converter/ConverterTag.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	'}' should be alone on a line.	113

org/apache/myfaces/commons/converter/DateTimeConverter.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	'if' construct must use '{}'s.	93
	'if' construct must use '{}'s.	94
	'if' construct must use '{}'s.	103
	Line is longer than 120 characters.	112
	'if' construct must use '{}'s.	114
	Line is longer than 120 characters.	115
	'if' construct must use '{}'s.	116
	Line is longer than 120 characters.	117
	'if' construct must use '{}'s.	118
	Line is longer than 120 characters.	119
	'else' construct must use '{}'s.	120
	'if' construct must use '{}'s.	130
	'if' construct must use '{}'s.	131
	Line is longer than 120 characters.	154
	'if' construct must use '{}'s.	264
	'if' construct must use '{}'s.	278
	Name '_localeValue' must match pattern '^[a-z][a-zA-Z0-9]*$'.	391
	Name '_timeZoneValue' must match pattern '^[a-z][a-zA-Z0-9]*$'.	478

org/apache/myfaces/commons/converter/EnumConverter.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	'{' should be on a new line.	45
	Line is longer than 120 characters.	50
	'{' should be on a new line.	58
	'{' should be on a new line.	61
	'if' construct must use '{}'s.	62
	Line is longer than 120 characters.	66
	'{' should be on a new line.	66
	'if' construct must use '{}'s.	67
	'if' construct must use '{}'s.	68
	'if' construct must use '{}'s.	69
	'{' should be on a new line.	74
	'}' should be alone on a line.	76
	'{' should be on a new line.	76
	'}' should be alone on a line.	79
	'{' should be on a new line.	79
	'{' should be on a new line.	84
	'{' should be on a new line.	88
	'{' should be on a new line.	89
	Line is longer than 120 characters.	97
	'{' should be on a new line.	97
	'if' construct must use '{}'s.	98
	'if' construct must use '{}'s.	99
	'if' construct must use '{}'s.	100
	'if' construct must use '{}'s.	102
	'{' should be on a new line.	107
	'}' should be alone on a line.	110
	'{' should be on a new line.	110
	'{' should be on a new line.	112
	'{' should be on a new line.	115
	'}' should be alone on a line.	117
	'{' should be on a new line.	117
	'{' should be on a new line.	121
	'{' should be on a new line.	126
	'{' should be on a new line.	132
	'}' should be alone on a line.	135
	'{' should be on a new line.	135
	Line is longer than 120 characters.	146
	'{' should be on a new line.	146
	'{' should be on a new line.	147
	'{' should be on a new line.	163
	'if' construct must use '{}'s.	166
	'{' should be on a new line.	171
	'{' should be on a new line.	178
	'{' should be on a new line.	186
	'{' should be on a new line.	190

org/apache/myfaces/commons/converter/TypedNumberConverterTagHandler.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	Line is longer than 120 characters.	58

org/apache/myfaces/commons/converter/_AttachedListStateWrapper.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2

org/apache/myfaces/commons/converter/_AttachedStateWrapper.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	Line is longer than 120 characters.	42

org/apache/myfaces/commons/converter/_ConverterRule.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	'{' should be on a new line.	27
	'{' should be on a new line.	29
	'{' should be on a new line.	38
	'{' should be on a new line.	44
	Name 'Instance' must match pattern '^_?((log)|([a-z][a-zA-Z]*ThreadLocal)|([A-Z][A-Z0-9]*(_[A-Z0-9]+)*))$'.	75
	'{' should be on a new line.	77
	'{' should be on a new line.	82
	'{' should be on a new line.	83
	'{' should be on a new line.	86
	'{' should be on a new line.	88

org/apache/myfaces/commons/converter/_DateTimeConverterRule.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	'{' should be on a new line.	27
	'{' should be on a new line.	29
	'{' should be on a new line.	38
	'{' should be on a new line.	44
	Name 'Instance' must match pattern '^_?((log)|([a-z][a-zA-Z]*ThreadLocal)|([A-Z][A-Z0-9]*(_[A-Z0-9]+)*))$'.	75
	'{' should be on a new line.	77
	'{' should be on a new line.	82
	'{' should be on a new line.	83
	'{' should be on a new line.	86
	'{' should be on a new line.	88

org/apache/myfaces/commons/converter/_DeltaStateHelper.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	'if' construct must use '{}'s.	596

org/apache/myfaces/commons/converter/_LocaleRule.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	'{' should be on a new line.	31
	'{' should be on a new line.	33
	'{' should be on a new line.	42
	'{' should be on a new line.	48
	Name 'Instance' must match pattern '^_?((log)|([a-z][a-zA-Z]*ThreadLocal)|([A-Z][A-Z0-9]*(_[A-Z0-9]+)*))$'.	93
	'{' should be on a new line.	95
	'{' should be on a new line.	100
	'{' should be on a new line.	101
	'{' should be on a new line.	106

org/apache/myfaces/commons/converter/_TimeZoneRule.java

Violation	Message	Line
	Line does not match expected header line of ' * Licensed to the Apache Software Foundation (ASF) under one'.	2
	'{' should be on a new line.	31
	'{' should be on a new line.	33
	'{' should be on a new line.	42
	'{' should be on a new line.	48
	Name 'Instance' must match pattern '^_?((log)|([a-z][a-zA-Z]*ThreadLocal)|([A-Z][A-Z0-9]*(_[A-Z0-9]+)*))$'.	93
	'{' should be on a new line.	95
	'{' should be on a new line.	100
	'{' should be on a new line.	101
	'{' should be on a new line.	106