Checkstyle Results
The following document contains the results of
Checkstyle
.
Summary
| Files |
Infos |
Warnings |
Errors |
| 8 |
0 |
0 |
386 |
Files
org/apache/catalina/servlets/HTMLManagerXServlet.java
| Error |
Line |
|
Line does not match expected header line of '/* ===================================================================='. |
1
|
|
Extra HTML tag found: </code> |
93
|
|
Line is longer than 80 characters. |
97
|
|
Parameter request should be final. |
114
|
|
Parameter response should be final. |
115
|
|
'+' is not preceded with whitespace. |
131
|
|
'+' is not followed by whitespace. |
131
|
|
Parameter writer should be final. |
164
|
|
'if' construct must use '{}'s. |
166
|
|
'+' should be on a new line. |
167
|
|
'+' is not preceded with whitespace. |
172
|
|
'+' is not followed by whitespace. |
172
|
|
'+' is not preceded with whitespace. |
179
|
|
'+' is not followed by whitespace. |
179
|
|
Array brackets at illegal position. |
191
|
|
'for' is not followed by whitespace. |
192
|
|
Line has trailing spaces. |
193
|
|
Array brackets at illegal position. |
195
|
|
Line has trailing spaces. |
196
|
|
'for' construct must use '{}'s. |
205
|
|
'for' is not followed by whitespace. |
205
|
|
'for' is not followed by whitespace. |
211
|
|
'if' construct must use '{}'s. |
212
|
|
'if' is not followed by whitespace. |
212
|
|
'else' construct must use '{}'s. |
214
|
|
Line is longer than 80 characters. |
222
|
|
'+' is not preceded with whitespace. |
224
|
|
'+' is not followed by whitespace. |
224
|
|
Line has trailing spaces. |
228
|
|
Parameter writer should be final. |
236
|
|
Parameter vhost should be final. |
236
|
|
'if' construct must use '{}'s. |
238
|
|
'+' should be on a new line. |
239
|
|
'if' is not followed by whitespace. |
242
|
|
Line is longer than 80 characters. |
243
|
|
Line is longer than 80 characters. |
249
|
|
Line has trailing spaces. |
251
|
|
Line has trailing spaces. |
252
|
|
Line has trailing spaces. |
254
|
|
'+' is not preceded with whitespace. |
257
|
|
'+' is not followed by whitespace. |
257
|
|
'+' is not preceded with whitespace. |
265
|
|
'+' is not followed by whitespace. |
265
|
|
Array brackets at illegal position. |
283
|
|
'if' construct must use '{}'s. |
288
|
|
'if' is not followed by whitespace. |
288
|
|
'(' is followed by whitespace. |
288
|
|
')' is preceded with whitespace. |
288
|
|
')' is preceded with whitespace. |
290
|
|
'+' is not preceded with whitespace. |
294
|
|
'+' is not followed by whitespace. |
294
|
|
Line is longer than 80 characters. |
299
|
|
'+' is not preceded with whitespace. |
299
|
|
'+' is not followed by whitespace. |
299
|
|
'+' is not preceded with whitespace. |
299
|
|
'+' is not followed by whitespace. |
299
|
|
'+' is not preceded with whitespace. |
299
|
|
'+' is not followed by whitespace. |
299
|
|
Line is longer than 80 characters. |
302
|
|
'+' is not preceded with whitespace. |
302
|
|
'+' is not followed by whitespace. |
302
|
|
'+' is not preceded with whitespace. |
302
|
|
'+' is not followed by whitespace. |
302
|
|
'+' is not preceded with whitespace. |
302
|
|
'+' is not followed by whitespace. |
302
|
|
Line is longer than 80 characters. |
309
|
|
'+' is not preceded with whitespace. |
309
|
|
'+' is not followed by whitespace. |
309
|
|
'+' is not preceded with whitespace. |
309
|
|
'+' is not followed by whitespace. |
309
|
|
'+' is not preceded with whitespace. |
309
|
|
'+' is not followed by whitespace. |
309
|
|
'+' is not preceded with whitespace. |
309
|
|
'+' is not followed by whitespace. |
309
|
|
Line is longer than 80 characters. |
316
|
|
'+' is not preceded with whitespace. |
316
|
|
'+' is not followed by whitespace. |
316
|
|
'+' is not preceded with whitespace. |
316
|
|
'+' is not followed by whitespace. |
316
|
|
'+' is not preceded with whitespace. |
316
|
|
'+' is not followed by whitespace. |
316
|
|
'+' is not preceded with whitespace. |
316
|
|
'+' is not followed by whitespace. |
316
|
|
Line is longer than 80 characters. |
325
|
|
'+' is not preceded with whitespace. |
325
|
|
'+' is not followed by whitespace. |
325
|
|
'+' is not preceded with whitespace. |
325
|
|
'+' is not followed by whitespace. |
325
|
|
'+' is not preceded with whitespace. |
325
|
|
'+' is not followed by whitespace. |
325
|
|
'+' is not preceded with whitespace. |
325
|
|
'+' is not followed by whitespace. |
325
|
|
Line is longer than 80 characters. |
336
|
|
Line is longer than 80 characters. |
339
|
|
Line is longer than 80 characters. |
343
|
|
Line is longer than 80 characters. |
344
|
|
'+' is not preceded with whitespace. |
346
|
|
'+' is not followed by whitespace. |
346
|
|
Parameter writer should be final. |
357
|
|
Parameter vhost should be final. |
357
|
|
Parameter path should be final. |
357
|
|
'if' construct must use '{}'s. |
359
|
|
Line is longer than 80 characters. |
360
|
|
'if' is not followed by whitespace. |
362
|
|
Line is longer than 80 characters. |
363
|
|
Line is longer than 80 characters. |
369
|
|
Line has trailing spaces. |
371
|
|
Line has trailing spaces. |
372
|
|
'if' construct must use '{}'s. |
384
|
|
'if' is not followed by whitespace. |
384
|
|
'(' is followed by whitespace. |
384
|
|
')' is preceded with whitespace. |
384
|
|
Line is longer than 80 characters. |
391
|
|
Line is longer than 80 characters. |
400
|
|
Parameter writer should be final. |
413
|
|
Parameter vhost should be final. |
413
|
|
Parameter path should be final. |
413
|
|
'if' construct must use '{}'s. |
415
|
|
Line is longer than 80 characters. |
416
|
|
'if' is not followed by whitespace. |
418
|
|
Line is longer than 80 characters. |
419
|
|
Line is longer than 80 characters. |
425
|
|
Line has trailing spaces. |
427
|
|
Line has trailing spaces. |
428
|
|
'if' construct must use '{}'s. |
440
|
|
'if' is not followed by whitespace. |
440
|
|
'(' is followed by whitespace. |
440
|
|
')' is preceded with whitespace. |
440
|
|
Line is longer than 80 characters. |
447
|
|
Line is longer than 80 characters. |
456
|
|
Parameter writer should be final. |
470
|
|
Parameter vhost should be final. |
470
|
|
Parameter path should be final. |
470
|
|
'if' construct must use '{}'s. |
472
|
|
Line is longer than 80 characters. |
473
|
|
'if' is not followed by whitespace. |
475
|
|
Line is longer than 80 characters. |
476
|
|
Line is longer than 80 characters. |
482
|
|
Line has trailing spaces. |
484
|
|
Line has trailing spaces. |
485
|
|
'if' construct must use '{}'s. |
497
|
|
'if' is not followed by whitespace. |
497
|
|
'(' is followed by whitespace. |
497
|
|
')' is preceded with whitespace. |
497
|
|
Line is longer than 80 characters. |
504
|
|
Parameter writer should be final. |
527
|
|
Parameter vhost should be final. |
527
|
|
Parameter path should be final. |
527
|
|
'if' construct must use '{}'s. |
529
|
|
Line is longer than 80 characters. |
530
|
|
'if' is not followed by whitespace. |
532
|
|
Line is longer than 80 characters. |
533
|
|
Line is longer than 80 characters. |
539
|
|
Line has trailing spaces. |
541
|
|
Line has trailing spaces. |
542
|
|
'if' construct must use '{}'s. |
554
|
|
'if' is not followed by whitespace. |
554
|
|
'(' is followed by whitespace. |
554
|
|
')' is preceded with whitespace. |
554
|
|
Line is longer than 80 characters. |
561
|
org/apache/catalina/servlets/ManagerXServlet.java
| Error |
Line |
|
Line does not match expected header line of '/* ===================================================================='. |
1
|
|
Line is longer than 80 characters. |
98
|
|
Variable 'context' must be private and have accessor methods. |
110
|
|
Variable 'debug' must be private and have accessor methods. |
116
|
|
First sentence should end with a period. |
119
|
|
Variable 'engine' must be private and have accessor methods. |
122
|
|
Line has trailing spaces. |
127
|
|
Variable 'sm' must be private and have accessor methods. |
127
|
|
Variable 'wrapper' must be private and have accessor methods. |
134
|
|
Expected an @return tag. |
143
|
|
Method 'getWrapper' is not designed for extension - needs to be abstract, final or empty. |
143
|
|
Method 'setWrapper' is not designed for extension - needs to be abstract, final or empty. |
155
|
|
Parameter wrapper should be final. |
155
|
|
'wrapper' hides a field. |
155
|
|
Method 'destroy' is not designed for extension - needs to be abstract, final or empty. |
175
|
|
Empty statement. |
177
|
|
Method 'doGet' is not designed for extension - needs to be abstract, final or empty. |
191
|
|
Parameter request should be final. |
191
|
|
Parameter response should be final. |
192
|
|
'if' construct must use '{}'s. |
196
|
|
'if' construct must use '{}'s. |
202
|
|
Method 'init' is not designed for extension - needs to be abstract, final or empty. |
246
|
|
Expected @throws tag for 'ServletException'. |
246
|
|
'if' construct must use '{}'s. |
249
|
|
'if' construct must use '{}'s. |
255
|
|
'if' construct must use '{}'s. |
257
|
|
Empty statement. |
267
|
|
Line is longer than 80 characters. |
273
|
|
Method 'vhosts' is not designed for extension - needs to be abstract, final or empty. |
288
|
|
Parameter writer should be final. |
288
|
|
'if' construct must use '{}'s. |
290
|
|
'+' should be on a new line. |
291
|
|
Line has trailing spaces. |
296
|
|
Array brackets at illegal position. |
297
|
|
Line has trailing spaces. |
298
|
|
Line has trailing spaces. |
300
|
|
Line has trailing spaces. |
302
|
|
Line has trailing spaces. |
304
|
|
Line is longer than 80 characters. |
314
|
|
Method 'install' is not designed for extension - needs to be abstract, final or empty. |
314
|
|
Parameter writer should be final. |
314
|
|
Parameter vhost should be final. |
314
|
|
Parameter path should be final. |
314
|
|
Parameter war should be final. |
314
|
|
'if' construct must use '{}'s. |
316
|
|
Line is longer than 80 characters. |
317
|
|
'+' should be on a new line. |
317
|
|
'if' is not followed by whitespace. |
320
|
|
Line is longer than 80 characters. |
321
|
|
Line has trailing spaces. |
323
|
|
Line is longer than 80 characters. |
326
|
|
Line has trailing spaces. |
327
|
|
Line has trailing spaces. |
329
|
|
'if' construct must use '{}'s. |
335
|
|
'if' is not followed by whitespace. |
335
|
|
'(' is followed by whitespace. |
335
|
|
')' is preceded with whitespace. |
335
|
|
'||' should be on a new line. |
337
|
|
'context' hides a field. |
344
|
|
Line is longer than 80 characters. |
351
|
|
Line is longer than 80 characters. |
353
|
|
Method 'list' is not designed for extension - needs to be abstract, final or empty. |
367
|
|
Parameter writer should be final. |
367
|
|
Parameter vhost should be final. |
367
|
|
'if' construct must use '{}'s. |
369
|
|
'+' should be on a new line. |
370
|
|
'if' is not followed by whitespace. |
373
|
|
Line is longer than 80 characters. |
374
|
|
Line has trailing spaces. |
376
|
|
Line is longer than 80 characters. |
379
|
|
Line has trailing spaces. |
380
|
|
Line has trailing spaces. |
382
|
|
Array brackets at illegal position. |
385
|
|
'context' hides a field. |
387
|
|
'if' construct must use '{}'s. |
389
|
|
'if' is not followed by whitespace. |
389
|
|
'(' is followed by whitespace. |
389
|
|
')' is preceded with whitespace. |
389
|
|
')' is preceded with whitespace. |
391
|
|
Line is longer than 80 characters. |
396
|
|
Method 'reload' is not designed for extension - needs to be abstract, final or empty. |
415
|
|
Parameter writer should be final. |
415
|
|
Parameter vhost should be final. |
415
|
|
Parameter path should be final. |
415
|
|
'if' construct must use '{}'s. |
417
|
|
Line is longer than 80 characters. |
418
|
|
'if' is not followed by whitespace. |
420
|
|
Line is longer than 80 characters. |
421
|
|
Line has trailing spaces. |
423
|
|
Line is longer than 80 characters. |
426
|
|
Line has trailing spaces. |
427
|
|
Line has trailing spaces. |
429
|
|
'if' construct must use '{}'s. |
435
|
|
'if' is not followed by whitespace. |
435
|
|
'(' is followed by whitespace. |
435
|
|
')' is preceded with whitespace. |
435
|
|
'context' hides a field. |
439
|
|
Line is longer than 80 characters. |
441
|
|
Line is longer than 80 characters. |
445
|
|
Line is longer than 80 characters. |
447
|
|
Method 'remove' is not designed for extension - needs to be abstract, final or empty. |
462
|
|
Parameter writer should be final. |
462
|
|
Parameter vhost should be final. |
462
|
|
Parameter path should be final. |
462
|
|
'if' construct must use '{}'s. |
464
|
|
Line is longer than 80 characters. |
465
|
|
'if' is not followed by whitespace. |
467
|
|
Line is longer than 80 characters. |
468
|
|
Line has trailing spaces. |
470
|
|
Line is longer than 80 characters. |
473
|
|
Line has trailing spaces. |
474
|
|
Line has trailing spaces. |
476
|
|
'if' construct must use '{}'s. |
482
|
|
'if' is not followed by whitespace. |
482
|
|
'(' is followed by whitespace. |
482
|
|
')' is preceded with whitespace. |
482
|
|
'context' hides a field. |
486
|
|
Line is longer than 80 characters. |
488
|
|
Line is longer than 80 characters. |
494
|
|
Line is longer than 80 characters. |
504
|
|
Line is longer than 80 characters. |
509
|
|
Line is longer than 80 characters. |
510
|
|
Method 'sessions' is not designed for extension - needs to be abstract, final or empty. |
512
|
|
Parameter writer should be final. |
512
|
|
Parameter vhost should be final. |
512
|
|
Parameter path should be final. |
512
|
|
'if' construct must use '{}'s. |
514
|
|
Line is longer than 80 characters. |
515
|
|
'if' is not followed by whitespace. |
517
|
|
Line is longer than 80 characters. |
518
|
|
Line has trailing spaces. |
520
|
|
Line is longer than 80 characters. |
523
|
|
Line has trailing spaces. |
524
|
|
Line has trailing spaces. |
526
|
|
'if' construct must use '{}'s. |
532
|
|
'if' is not followed by whitespace. |
532
|
|
'(' is followed by whitespace. |
532
|
|
')' is preceded with whitespace. |
532
|
|
'context' hides a field. |
535
|
|
Line is longer than 80 characters. |
537
|
|
Line is longer than 80 characters. |
540
|
|
Line is longer than 80 characters. |
542
|
|
'/' is not preceded with whitespace. |
542
|
|
'/' is not followed by whitespace. |
542
|
|
'60' is a magic number. |
542
|
|
'60' is a magic number. |
544
|
|
'/' is not preceded with whitespace. |
547
|
|
'/' is not followed by whitespace. |
547
|
|
'10' is a magic number. |
547
|
|
'*' is not preceded with whitespace. |
547
|
|
'*' is not followed by whitespace. |
547
|
|
'60' is a magic number. |
547
|
|
'if' construct must use '{}'s. |
548
|
|
'if' construct must use '{}'s. |
550
|
|
'-' is not preceded with whitespace. |
551
|
|
'-' is not followed by whitespace. |
551
|
|
'else' construct must use '{}'s. |
552
|
|
'if' construct must use '{}'s. |
555
|
|
'-' is not preceded with whitespace. |
558
|
|
'-' is not followed by whitespace. |
558
|
|
'if' construct must use '{}'s. |
559
|
|
'*' is not preceded with whitespace. |
561
|
|
'*' is not followed by whitespace. |
561
|
|
'10' is a magic number. |
561
|
|
'+' is not preceded with whitespace. |
561
|
|
'+' is not followed by whitespace. |
561
|
|
'*' is not preceded with whitespace. |
561
|
|
'*' is not followed by whitespace. |
561
|
|
'10' is a magic number. |
561
|
|
'if' construct must use '{}'s. |
564
|
|
'-' is not preceded with whitespace. |
564
|
|
'-' is not followed by whitespace. |
564
|
|
'*' is not preceded with whitespace. |
566
|
|
'*' is not followed by whitespace. |
566
|
|
'10' is a magic number. |
566
|
|
'-' is not preceded with whitespace. |
567
|
|
'-' is not followed by whitespace. |
567
|
|
'if' construct must use '{}'s. |
568
|
|
',' is not followed by whitespace. |
570
|
|
Line is longer than 80 characters. |
572
|
|
Method 'start' is not designed for extension - needs to be abstract, final or empty. |
587
|
|
Parameter writer should be final. |
587
|
|
Parameter vhost should be final. |
587
|
|
Parameter path should be final. |
587
|
|
'if' construct must use '{}'s. |
589
|
|
Line is longer than 80 characters. |
590
|
|
'if' is not followed by whitespace. |
592
|
|
Line is longer than 80 characters. |
593
|
|
Line has trailing spaces. |
595
|
|
Line is longer than 80 characters. |
598
|
|
Line has trailing spaces. |
599
|
|
Line has trailing spaces. |
601
|
|
'if' construct must use '{}'s. |
607
|
|
'if' is not followed by whitespace. |
607
|
|
'(' is followed by whitespace. |
607
|
|
')' is preceded with whitespace. |
607
|
|
'context' hides a field. |
611
|
|
Line is longer than 80 characters. |
613
|
|
'if' construct must use '{}'s. |
617
|
|
'else' construct must use '{}'s. |
620
|
|
Method 'stop' is not designed for extension - needs to be abstract, final or empty. |
642
|
|
Parameter writer should be final. |
642
|
|
Parameter vhost should be final. |
642
|
|
Parameter path should be final. |
642
|
|
'if' construct must use '{}'s. |
644
|
|
Line is longer than 80 characters. |
645
|
|
'if' is not followed by whitespace. |
647
|
|
Line is longer than 80 characters. |
648
|
|
Line has trailing spaces. |
650
|
|
Line is longer than 80 characters. |
653
|
|
Line has trailing spaces. |
654
|
|
Line has trailing spaces. |
656
|
|
'if' construct must use '{}'s. |
662
|
|
'if' is not followed by whitespace. |
662
|
|
'(' is followed by whitespace. |
662
|
|
')' is preceded with whitespace. |
662
|
|
'context' hides a field. |
666
|
|
Line is longer than 80 characters. |
668
|
org/apache/catalina/servlets/package.html
| Error |
Line |
|
Missing package documentation file. |
0 |
org/talika/tms/Constants.java
| Error |
Line |
|
Line does not match expected header line of '/* ===================================================================='. |
1
|
|
Missing a Javadoc comment. |
68
|
|
Missing a Javadoc comment. |
70
|
|
Name 'Package' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. |
70
|
|
Line has trailing spaces. |
71
|
|
Missing a Javadoc comment. |
72
|
org/talika/tms/package.html
| Error |
Line |
|
Missing package documentation file. |
0 |