|
|
The following document contains the results of
Checkstyle
.
| Files | Infos | Warnings | Errors |
|---|
| 55 | 4 | 342 | 24 |
| Error | Line |
|---|
| First sentence should end with a period. |
23
|
|
| 'public' modifier out of order with the JLS suggestions. |
30
|
|
| First sentence should end with a period. |
32
|
|
| Expected an @return tag. |
33
|
|
| 'public' modifier out of order with the JLS suggestions. |
33
|
|
| Unused @throws tag for 'Exception'. |
41
|
|
| Expected @throws tag for 'JellyTagException'. |
43
|
|
| Il y a un espace de trop apr?Ęs '('. |
48
|
|
| Il y a un espace de trop avant ')'. |
48
|
|
| Il y a un espace de trop apr?Ęs '('. |
49
|
|
| '+' devrait ??tre sur une nouvelle ligne. |
49
|
|
| Line is longer than 100 characters. |
50
|
|
| Il y a un espace de trop avant ')'. |
50
|
| Error | Line |
|---|
|
| First sentence should end with a period. |
20
|
|
| First sentence should end with a period. |
28
|
|
| Expected an @return tag. |
29
|
| Error | Line |
|---|
|
| First sentence should end with a period. |
19
|
|
| First sentence should end with a period. |
27
|
|
| Expected an @return tag. |
28
|
| Error | Line |
|---|
|
| Wrong order for 'java.io.IOException' import. |
29
|
|
| First sentence should end with a period. |
33
|
|
| First sentence should end with a period. |
41
|
| Name '_contextPath' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
42
|
|
| First sentence should end with a period. |
44
|
|
| Name '_resourceBase' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
45
|
|
| First sentence should end with a period. |
47
|
|
| Name '_realmName' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
48
|
|
| First sentence should end with a period. |
50
|
|
| Name '_context' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
51
|
|
| First sentence should end with a period. |
53
|
|
| Unused @throws tag for 'Exception'. |
64
|
|
| Expected @throws tag for 'JellyTagException'. |
66
|
|
| Il y a un espace de trop apr?Ęs '('. |
70
|
|
| Il y a un espace de trop avant ')'. |
70
|
|
| Line is longer than 100 characters. |
71
|
|
| Il y a un espace de trop apr?Ęs '('. |
71
|
|
| Il y a un espace de trop avant ')'. |
71
|
|
| '}' should be on the same line. |
84
|
|
| '}' should be on the same line. |
87
|
|
| First sentence should end with a period. |
99
|
|
| Unused @param tag for 'handler'. |
102
|
|
| Expected @param tag for 'httHandler'. |
104
|
|
| First sentence should end with a period. |
108
|
|
| First sentence should end with a period. |
119
|
|
| '{' should be on the previous line. |
125
|
|
| Unused @param tag for 'path'. |
144
|
|
| Expected @param tag for 'contextPath'. |
146
|
|
| Unused @param tag for 'path'. |
180
|
|
| Expected @param tag for 'realmName'. |
182
|
| Error | Line |
|---|
|
| First sentence should end with a period. |
23
|
|
| Name '_jellyResourceHttpHandler' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
33
|
|
| First sentence should end with a period. |
35
|
|
| Unused @throws tag for 'Exception'. |
44
|
|
| Expected @throws tag for 'JellyTagException'. |
46
|
|
| Il y a un espace de trop apr?Ęs '('. |
49
|
|
| Il y a un espace de trop avant ')'. |
49
|
|
| Line is longer than 100 characters. |
50
|
|
| Il y a un espace de trop apr?Ęs '('. |
50
|
|
| Il y a un espace de trop avant ')'. |
50
|
|
| Missing a Javadoc comment. |
66
|
| Error | Line |
|---|
|
| Wrong order for 'java.io.BufferedReader' import. |
31
|
|
| Redundant import from the java.lang package - java.lang.StringBuffer. |
35
|
|
| First sentence should end with a period. |
39
|
|
| Field 'log' is used in only one method. |
48
|
|
| Field 'OVERRIDE_SET_HANDLED_VAR' is used in only one method. |
51
|
|
| First sentence should end with a period. |
53
|
|
| Name '_tagMap' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
54
|
|
| First sentence should end with a period. |
56
|
|
| Name '_xmlOutput' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
57
|
|
| First sentence should end with a period. |
59
|
|
| Il y a un espace de trop apr?Ęs '('. |
60
|
|
| Expected @param tag for 'xmlOutput'. |
60
|
|
| Il y a un espace de trop avant ')'. |
60
|
|
| Missing a Javadoc comment. |
71
|
|
| Il manque un espace avant '{'. |
71
|
|
| Missing a Javadoc comment. |
86
|
|
| Redundant throws: 'HttpException' is subclass of 'IOException'. |
90
|
|
| '{' should be on the previous line. |
91
|
|
| Il y a un espace de trop apr?Ęs '('. |
96
|
|
| Il y a un espace de trop apr?Ęs '('. |
97
|
|
| Il y a un espace de trop apr?Ęs '('. |
98
|
|
| Il y a un espace de trop apr?Ęs '('. |
99
|
|
| Il y a un espace de trop apr?Ęs '('. |
100
|
|
| Il y a un espace de trop avant ')'. |
112
|
|
| Line is longer than 100 characters. |
114
|
|
| '+' devrait ??tre sur une nouvelle ligne. |
117
|
|
| '+' devrait ??tre sur une nouvelle ligne. |
118
|
|
| '+' devrait ??tre sur une nouvelle ligne. |
119
|
|
| Missing a Javadoc comment. |
126
|
|
| Unused local variable 'br'. |
131
|
|
| '1024' is a magic number. |
133
|
|
| 'while' construct must use '{}'s. |
136
|
|
| '1024' is a magic number. |
136
|
|
| method call child at indentation level 10 not at correct indentation, 12 |
137
|
|
| 'if' construct must use '{}'s. |
139
|
|
| if child at indentation level 10 not at correct indentation, 12 |
140
|
|
| 'else' construct must use '{}'s. |
141
|
|
| else child at indentation level 10 not at correct indentation, 12 |
142
|
| Error | Line |
|---|
|
| First sentence should end with a period. |
39
|
|
| First sentence should end with a period. |
47
|
|
| First sentence should end with a period. |
50
|
|
| First sentence should end with a period. |
53
|
|
| First sentence should end with a period. |
56
|
|
| First sentence should end with a period. |
59
|
|
| First sentence should end with a period. |
66
|
|
| Name '_logSink' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
67
|
|
| try at indentation level 8 not at correct indentation, 4 |
72
|
|
| try child at indentation level 12 not at correct indentation, 8 |
73
|
|
| try child at indentation level 12 not at correct indentation, 8 |
75
|
|
| try rcurly at indentation level 8 not at correct indentation, 4 |
76
|
|
| Il y a un espace de trop avant ')'. |
76
|
|
| catch child at indentation level 12 not at correct indentation, 8 |
77
|
|
| catch rcurly at indentation level 8 not at correct indentation, 4 |
78
|
|
| First sentence should end with a period. |
82
|
|
| Name '_var' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
83
|
|
| First sentence should end with a period. |
85
|
|
| Name '_server' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
86
|
|
| First sentence should end with a period. |
88
|
|
| Name '_logFileName' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
89
|
|
| First sentence should end with a period. |
91
|
|
| Il manque un espace avant '='. |
95
|
|
| Il manque un espace apr?Ęs '='. |
95
|
|
| Unused @throws tag for 'Exception'. |
108
|
| Cyclomatic Complexity is 13 (max allowed is 10). |
110
|
|
| Expected @throws tag for 'JellyTagException'. |
110
|
|
| Il y a un espace de trop avant ')'. |
116
|
|
| Il manque un espace avant '='. |
126
|
|
| Il manque un espace apr?Ęs '='. |
126
|
|
| '}' should be on the same line. |
145
|
|
| '}' should be on the same line. |
148
|
|
| '+' devrait ??tre sur une nouvelle ligne. |
159
|
|
| '}' should be on the same line. |
169
|
|
| First sentence should end with a period. |
180
|
|
| First sentence should end with a period. |
189
|
|
| First sentence should end with a period. |
199
|
|
| '{' should be on the previous line. |
206
|
| Error | Line |
|---|
|
| First sentence should end with a period. |
23
|
|
| First sentence should end with a period. |
31
|
|
| Javadoc has empty description section. |
54
|
| Error | Line |
|---|
|
| First sentence should end with a period. |
24
|
|
| First sentence should end with a period. |
32
|
|
| Unused @throws tag for 'Exception'. |
41
|
|
| Expected @throws tag for 'JellyTagException'. |
43
|
|
| Il y a un espace de trop apr?Ęs '('. |
46
|
|
| Il y a un espace de trop avant ')'. |
46
|
|
| Line is longer than 100 characters. |
47
|
|
| Il y a un espace de trop apr?Ęs '('. |
47
|
|
| Il y a un espace de trop avant ')'. |
47
|
| Error | Line |
|---|
|
| Found duplicate of 19 lines in /Volumes/Data/paul/projects/jakarta-commons/jelly/jelly-tags/jetty/src/java/org/apache/commons/jelly/tags/jetty/GetRequestTag.java, starting from line 1 |
1
|
| Error | Line |
|---|
|
| First sentence should end with a period. |
19
|
|
| First sentence should end with a period. |
27
|
|
| Expected an @return tag. |
28
|
| Error | Line |
|---|
|
| Found duplicate of 19 lines in /Volumes/Data/paul/projects/jakarta-commons/jelly/jelly-tags/jetty/src/java/org/apache/commons/jelly/tags/jetty/GetRequestTag.java, starting from line 1 |
1
|
|
| Found duplicate of 19 lines in /Volumes/Data/paul/projects/jakarta-commons/jelly/jelly-tags/jetty/src/java/org/apache/commons/jelly/tags/jetty/PostRequestTag.java, starting from line 1 |
1
|
| Error | Line |
|---|
|
| First sentence should end with a period. |
25
|
|
| First sentence should end with a period. |
28
|
|
| Expected an @return tag. |
29
|
| Error | Line |
|---|
|
| Wrong order for 'java.io.IOException' import. |
25
|
|
| First sentence should end with a period. |
28
|
|
| First sentence should end with a period. |
36
|
|
| Name '_name' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
37
|
|
| First sentence should end with a period. |
39
|
|
| Name '_config' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
40
|
|
| First sentence should end with a period. |
42
|
|
| Unused @throws tag for 'Exception'. |
52
|
|
| Expected @throws tag for 'JellyTagException'. |
54
|
|
| Il y a un espace de trop apr?Ęs '('. |
57
|
|
| Il y a un espace de trop avant ')'. |
57
|
|
| Il y a un espace de trop apr?Ęs '('. |
58
|
|
| Il y a un espace de trop avant ')'. |
58
|
|
| Il y a un espace de trop apr?Ęs '('. |
61
|
|
| Il y a un espace de trop avant ')'. |
61
|
|
| Il y a un espace de trop apr?Ęs '('. |
68
|
|
| Il y a un espace de trop avant ')'. |
68
|
|
| Il y a un espace de trop avant ')'. |
68
|
| Error | Line |
|---|
|
| Wrong order for 'java.util.StringTokenizer' import. |
25
|
|
| First sentence should end with a period. |
27
|
|
| First sentence should end with a period. |
35
|
|
| Name '_allowedMethods' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
36
|
|
| First sentence should end with a period. |
38
|
|
| Unused @throws tag for 'Exception'. |
47
|
|
| Expected @throws tag for 'JellyTagException'. |
49
|
|
| Il y a un espace de trop apr?Ęs '('. |
52
|
|
| Il y a un espace de trop avant ')'. |
52
|
|
| Line is longer than 100 characters. |
53
|
|
| Il y a un espace de trop apr?Ęs '('. |
53
|
|
| Il y a un espace de trop avant ')'. |
53
|
|
| Il y a un espace de trop apr?Ęs '('. |
59
|
|
| Il y a un espace de trop avant ')'. |
59
|
| Error | Line |
|---|
|
| First sentence should end with a period. |
28
|
|
| Unused @throws tag for 'Exception'. |
40
|
|
| Expected @throws tag for 'JellyTagException'. |
42
|
|
| '1500' is a magic number. |
50
|
|
| try child at indentation level 13 not at correct indentation, 12 |
52
|
|
| try child at indentation level 13 not at correct indentation, 12 |
53
|
|
| try child at indentation level 13 not at correct indentation, 12 |
54
|
|
| try child at indentation level 13 not at correct indentation, 12 |
55
|
|
| '}' should be on the same line. |
56
|
| Error | Line |
|---|
|
| First sentence should end with a period. |
25
|
|
| First sentence should end with a period. |
32
|
|
| Name '_value' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
33
|
|
| Unused @throws tag for 'Exception'. |
40
|
|
| Expected @throws tag for 'JellyTagException'. |
42
|
|
| '100' is a magic number. |
44
|
| Error | Line |
|---|
|
| First sentence should end with a period. |
25
|
|
| First sentence should end with a period. |
32
|
|
| Name '_name' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
33
|
|
| First sentence should end with a period. |
35
|
|
| Name '_value' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
36
|
|
| Unused @throws tag for 'Exception'. |
43
|
|
| Expected @throws tag for 'JellyTagException'. |
45
|
|
| Unused @param tag for 'path'. |
83
|
|
| Expected @param tag for 'name'. |
85
|
| Error | Line |
|---|
|
| Wrong order for 'java.io.IOException' import. |
36
|
|
| First sentence should end with a period. |
40
|
|
| First sentence should end with a period. |
48
|
|
| Field '_formAuthenticator' is used in only one method. |
49
|
|
| Name '_formAuthenticator' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
49
|
|
| First sentence should end with a period. |
51
|
|
| Name '_authenticationMethod' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
52
|
|
| First sentence should end with a period. |
54
|
|
| Unclosed HTML tag found: <constraints> tag to allow parsing of a well-formed snippet, |
63
|
|
| Unused @throws tag for 'Exception'. |
93
|
|
| Cyclomatic Complexity is 11 (max allowed is 10). |
95
|
|
| Expected @throws tag for 'JellyTagException'. |
95
|
|
| Il y a un espace de trop apr?Ęs '('. |
98
|
|
| Il y a un espace de trop avant ')'. |
98
|
|
| Line is longer than 100 characters. |
99
|
|
| Il y a un espace de trop apr?Ęs '('. |
99
|
|
| Il y a un espace de trop avant ')'. |
99
|
|
| '}' should be on the same line. |
119
|
|
| '}' should be on the same line. |
122
|
|
| Il manque un espace avant '='. |
127
|
|
| Il manque un espace apr?Ęs '='. |
127
|
|
| '{' should be on the previous line. |
130
|
|
| while child at indentation level 16 not at correct indentation, 12 |
131
|
|
| 'if' construct must use '{}'s. |
132
|
|
| if at indentation level 16 not at correct indentation, 12 |
132
|
|
| while child at indentation level 16 not at correct indentation, 12 |
135
|
|
| Il manque un espace avant '='. |
135
|
|
| Il manque un espace apr?Ęs '='. |
135
|
|
| Il manque un espace apr?Ęs 'cast'. |
135
|
|
| while child at indentation level 16 not at correct indentation, 12 |
136
|
|
| Il manque un espace avant '='. |
136
|
|
| Il manque un espace apr?Ęs '='. |
136
|
|
| if at indentation level 16 not at correct indentation, 12 |
138
|
|
| if child at indentation level 20 not at correct indentation, 16 |
139
|
|
| if rcurly at indentation level 16 not at correct indentation, 12 |
140
|
|
| if child at indentation level 20 not at correct indentation, 16 |
141
|
|
| if rcurly at indentation level 16 not at correct indentation, 12 |
142
|
|
| Line is longer than 100 characters. |
143
|
|
| else rcurly at indentation level 16 not at correct indentation, 12 |
144
|
|
| Cyclomatic Complexity is 13 (max allowed is 10). |
159
|
|
| Missing a Javadoc comment. |
159
|
|
| '{' should be on the previous line. |
161
|
|
| Il manque un espace avant '='. |
164
|
|
| Il manque un espace apr?Ęs '='. |
164
|
|
| Il manque un espace avant '!='. |
165
|
|
| Il manque un espace apr?Ęs '!='. |
165
|
|
| '{' should be on the previous line. |
166
|
|
| Il manque un espace avant '='. |
169
|
|
| Il manque un espace apr?Ęs 'while'. |
170
|
|
| '{' should be on the previous line. |
171
|
|
| Il manque un espace avant '='. |
172
|
|
| Il manque un espace apr?Ęs '='. |
172
|
|
| Il manque un espace apr?Ęs 'cast'. |
172
|
|
| Il manque un espace apr?Ęs ','. |
172
|
|
| Il manque un espace avant '='. |
177
|
|
| Il manque un espace apr?Ęs '='. |
177
|
|
| Il manque un espace avant '!='. |
178
|
|
| Il manque un espace apr?Ęs '!='. |
178
|
|
| '{' should be on the previous line. |
179
|
|
| Il manque un espace avant '='. |
180
|
|
| Il manque un espace apr?Ęs '='. |
180
|
|
| Il manque un espace apr?Ęs ','. |
181
|
|
| 'if' construct must use '{}'s. |
182
|
|
| Il manque un espace avant '=='. |
182
|
|
| Il manque un espace apr?Ęs '=='. |
182
|
|
| Il manque un espace avant '=='. |
182
|
|
| Il manque un espace apr?Ęs '=='. |
182
|
|
| '||' devrait ??tre sur une nouvelle ligne. |
182
|
|
| 'if' construct must use '{}'s. |
185
|
|
| 'if' construct must use '{}'s. |
187
|
|
| '{' should be on the previous line. |
190
|
|
| Il manque un espace avant '+'. |
191
|
|
| Il manque un espace apr?Ęs '+'. |
191
|
|
| Il manque un espace avant '='. |
196
|
|
| Il manque un espace apr?Ęs 'while'. |
197
|
|
| '{' should be on the previous line. |
198
|
|
| Il manque un espace avant '='. |
199
|
|
| Il manque un espace apr?Ęs '='. |
199
|
|
| Il manque un espace apr?Ęs 'cast'. |
199
|
|
| Il manque un espace avant '='. |
200
|
|
| Il manque un espace apr?Ęs '='. |
200
|
|
| Il manque un espace apr?Ęs ','. |
200
|
|
| Il manque un espace apr?Ęs ','. |
200
|
|
| Il manque un espace apr?Ęs 'cast'. |
201
|
|
| Il manque un espace avant '='. |
204
|
|
| 'while' construct must use '{}'s. |
205
|
|
| Il manque un espace apr?Ęs 'while'. |
205
|
|
| Il manque un espace apr?Ęs 'cast'. |
206
|
|
| Il manque un espace apr?Ęs ','. |
207
|
|
| Il manque un espace avant '='. |
209
|
|
| Il manque un espace apr?Ęs 'while'. |
210
|
|
| '{' should be on the previous line. |
211
|
|
| Il manque un espace avant '='. |
212
|
|
| Il manque un espace apr?Ęs 'cast'. |
213
|
|
| Il manque un espace apr?Ęs ','. |
213
|
|
| Il manque un espace apr?Ęs ','. |
214
|
|
| Cyclomatic Complexity is 13 (max allowed is 10). |
230
|
|
| Missing a Javadoc comment. |
230
|
|
| '{' should be on the previous line. |
232
|
|
| Il manque un espace avant '='. |
233
|
|
| Il manque un espace apr?Ęs '='. |
233
|
|
| Il manque un espace avant '!='. |
234
|
|
| Il manque un espace apr?Ęs '!='. |
234
|
|
| '{' should be on the previous line. |
235
|
|
| Il manque un espace avant '='. |
236
|
|
| Il manque un espace apr?Ęs '='. |
236
|
|
| Il manque un espace avant '='. |
237
|
|
| Il manque un espace apr?Ęs '='. |
237
|
|
| Il manque un espace apr?Ęs ','. |
237
|
|
| 'if' construct must use '{}'s. |
239
|
|
| Il manque un espace avant '='. |
240
|
|
| Il manque un espace apr?Ęs '='. |
240
|
|
| Il manque un espace avant '='. |
240
|
|
| Inner assignments should be avoided. |
240
|
|
| Il manque un espace apr?Ęs '='. |
240
|
|
| 'if' construct must use '{}'s. |
241
|
|
| Il manque un espace avant '='. |
242
|
|
| Il manque un espace apr?Ęs '='. |
242
|
|
| 'if' construct must use '{}'s. |
243
|
|
| Il manque un espace avant '='. |
244
|
|
| Il manque un espace apr?Ęs '='. |
244
|
|
| 'if' construct must use '{}'s. |
245
|
|
| Il manque un espace avant '='. |
246
|
|
| Il manque un espace apr?Ęs '='. |
246
|
|
| 'else' construct must use '{}'s. |
247
|
|
| Il manque un espace avant '+'. |
248
|
|
| Il manque un espace apr?Ęs '+'. |
248
|
|
| Il manque un espace avant '='. |
253
|
|
| Il manque un espace apr?Ęs '='. |
253
|
|
| 'if' construct must use '{}'s. |
254
|
|
| Il manque un espace avant '!='. |
254
|
|
| Il manque un espace apr?Ęs '!='. |
254
|
|
| Il manque un espace apr?Ęs ','. |
255
|
|
| Il manque un espace apr?Ęs 'if'. |
258
|
|
| '{' should be on the previous line. |
259
|
|
| 'if' construct must use '{}'s. |
260
|
|
| Il manque un espace avant '=='. |
260
|
|
| Il manque un espace apr?Ęs '=='. |
260
|
|
| '{' should be on the previous line. |
263
|
|
| 'if' construct must use '{}'s. |
265
|
|
| Il manque un espace apr?Ęs ','. |
266
|
|
| 'if' construct must use '{}'s. |
268
|
|
| Il manque un espace apr?Ęs ','. |
269
|
| Error | Line |
|---|
|
| First sentence should end with a period. |
28
|
|
| First sentence should end with a period. |
36
|
|
| Name '_port' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
37
|
|
| First sentence should end with a period. |
39
|
|
| Name '_host' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
40
|
|
| First sentence should end with a period. |
42
|
|
| Unused @throws tag for 'Exception'. |
51
|
|
| Expected @throws tag for 'JellyTagException'. |
53
|
|
| Il y a un espace de trop apr?Ęs '('. |
56
|
|
| Il y a un espace de trop avant ')'. |
56
|
|
| Line is longer than 100 characters. |
57
|
|
| Il y a un espace de trop apr?Ęs '('. |
57
|
|
| Il y a un espace de trop avant ')'. |
57
|
|
| '}' should be on the same line. |
63
|
|
