Checkstyle Results
The following document contains the results of
Checkstyle
.
Summary
| Files |
Infos |
Warnings |
Errors |
| 10 |
0 |
0 |
980 |
Files
org/apache/catalina/manager/HTMLManagerXServlet.java
| Error |
Line |
|
Line does not match expected header line of ''. |
1
|
|
Line has trailing spaces. |
3
|
|
Line has trailing spaces. |
7
|
|
Line has trailing spaces. |
9
|
|
Extra HTML tag found: </code> |
57
|
|
Line is longer than 80 characters. |
63
|
|
Parameter request should be final. |
80
|
|
Parameter response should be final. |
81
|
|
Must have at least one statement. |
98
|
|
Line is longer than 80 characters. |
100
|
|
Must have at least one statement. |
101
|
|
Must have at least one statement. |
102
|
|
Parameter request should be final. |
131
|
|
Parameter response should be final. |
132
|
|
',' is not followed by whitespace. |
139
|
|
Line has trailing spaces. |
157
|
|
Line has trailing spaces. |
165
|
|
Line has trailing spaces. |
170
|
|
'&&' should be on a new line. |
172
|
|
'}' should be on the same line. |
178
|
|
',' is not followed by whitespace. |
199
|
|
Line is longer than 80 characters. |
213
|
|
'}' should be on the same line. |
214
|
|
Line has trailing spaces. |
216
|
|
Line is longer than 80 characters. |
217
|
|
',' is not followed by whitespace. |
221
|
|
Line has trailing spaces. |
230
|
|
Line has trailing spaces. |
241
|
|
Line has trailing spaces. |
242
|
|
'catch' is not followed by whitespace. |
245
|
|
Line is longer than 80 characters. |
269
|
|
Parameter vhost should be final. |
269
|
|
Parameter config should be final. |
269
|
|
Parameter path should be final. |
269
|
|
Parameter war should be final. |
269
|
|
Method length is 277 lines (max allowed is 150). |
288
|
|
Parameter request should be final. |
288
|
|
Parameter response should be final. |
289
|
|
Parameter vhost should be final. |
290
|
|
Parameter message should be final. |
291
|
|
Expected @throws tag for 'IOException'. |
291
|
|
'if' construct must use '{}'s. |
293
|
|
'+' should be on a new line. |
294
|
|
Line is longer than 80 characters. |
302
|
|
Line has trailing spaces. |
304
|
|
Line has trailing spaces. |
306
|
|
'9' is a magic number. |
322
|
|
Line is longer than 80 characters. |
324
|
|
'3' is a magic number. |
326
|
|
'+' should be on a new line. |
327
|
|
'4' is a magic number. |
329
|
|
'5' is a magic number. |
330
|
|
'+' should be on a new line. |
331
|
|
'6' is a magic number. |
333
|
|
'7' is a magic number. |
334
|
|
'8' is a magic number. |
336
|
|
Line has trailing spaces. |
338
|
|
'4' is a magic number. |
340
|
|
'3' is a magic number. |
344
|
|
Line has trailing spaces. |
346
|
|
Array brackets at illegal position. |
348
|
|
Line has trailing spaces. |
349
|
|
Line is longer than 80 characters. |
358
|
|
'if' is not followed by whitespace. |
360
|
|
'5' is a magic number. |
366
|
|
'3' is a magic number. |
371
|
|
'4' is a magic number. |
373
|
|
Line is longer than 80 characters. |
376
|
|
Line has trailing spaces. |
377
|
|
Line has trailing spaces. |
379
|
|
Line has trailing spaces. |
381
|
|
'}' should be on the same line. |
382
|
|
'3' is a magic number. |
386
|
|
Line is longer than 80 characters. |
388
|
|
Avoid inline conditionals. |
388
|
|
Line has trailing spaces. |
390
|
|
'9' is a magic number. |
392
|
|
'+' should be on a new line. |
394
|
|
'3' is a magic number. |
397
|
|
'+' should be on a new line. |
398
|
|
'4' is a magic number. |
400
|
|
'5' is a magic number. |
401
|
|
'+' should be on a new line. |
402
|
|
'6' is a magic number. |
404
|
|
'7' is a magic number. |
405
|
|
'8' is a magic number. |
407
|
|
Line has trailing spaces. |
409
|
|
'6' is a magic number. |
411
|
|
Line is longer than 80 characters. |
412
|
|
'3' is a magic number. |
415
|
|
'4' is a magic number. |
416
|
|
'5' is a magic number. |
417
|
|
Line has trailing spaces. |
419
|
|
Array brackets at illegal position. |
422
|
|
Array brackets at illegal position. |
423
|
|
'for' construct must use '{}'s. |
424
|
|
Line has trailing spaces. |
426
|
|
Line has trailing spaces. |
428
|
|
Line has trailing spaces. |
433
|
|
Line is longer than 80 characters. |
437
|
|
Line has trailing spaces. |
438
|
|
'if' is not followed by whitespace. |
446
|
|
Line has trailing spaces. |
459
|
|
')' is preceded with whitespace. |
460
|
|
'6' is a magic number. |
461
|
|
'3' is a magic number. |
468
|
|
'+' should be on a new line. |
469
|
|
Line is longer than 80 characters. |
470
|
|
'4' is a magic number. |
472
|
|
'4' is a magic number. |
475
|
|
'5' is a magic number. |
478
|
|
Line has trailing spaces. |
482
|
|
'9' is a magic number. |
483
|
|
'+' should be on a new line. |
485
|
|
'+' should be on a new line. |
489
|
|
'3' is a magic number. |
491
|
|
'4' is a magic number. |
492
|
|
'+' should be on a new line. |
493
|
|
Line is longer than 80 characters. |
494
|
|
'5' is a magic number. |
495
|
|
'6' is a magic number. |
496
|
|
'+' should be on a new line. |
497
|
|
Line is longer than 80 characters. |
498
|
|
'7' is a magic number. |
499
|
|
Line has trailing spaces. |
500
|
|
'8' is a magic number. |
501
|
|
Line has trailing spaces. |
513
|
|
Line has trailing spaces. |
516
|
|
'8' is a magic number. |
518
|
|
Line is longer than 80 characters. |
521
|
|
Line has trailing spaces. |
522
|
|
'3' is a magic number. |
522
|
|
'4' is a magic number. |
523
|
|
'5' is a magic number. |
524
|
|
'6' is a magic number. |
525
|
|
'7' is a magic number. |
526
|
|
Line has trailing spaces. |
528
|
|
'5' is a magic number. |
529
|
|
Line is longer than 80 characters. |
531
|
|
'3' is a magic number. |
533
|
|
'4' is a magic number. |
534
|
|
'7' is a magic number. |
540
|
|
'3' is a magic number. |
544
|
|
'4' is a magic number. |
545
|
|
'5' is a magic number. |
546
|
|
'6' is a magic number. |
547
|
|
'6' is a magic number. |
552
|
|
'3' is a magic number. |
556
|
|
'4' is a magic number. |
557
|
|
'5' is a magic number. |
558
|
|
Parameter vhost should be final. |
578
|
|
Parameter path should be final. |
578
|
|
Parameter vhost should be final. |
597
|
|
Parameter path should be final. |
597
|
|
Parameter vhost should be final. |
616
|
|
Parameter path should be final. |
616
|
|
Parameter vhost should be final. |
635
|
|
Parameter path should be final. |
635
|
|
Parameter vhost should be final. |
654
|
|
Parameter path should be final. |
654
|
|
Missing a Javadoc comment. |
670
|
|
'+' should be on a new line. |
671
|
|
'+' should be on a new line. |
672
|
|
'+' should be on a new line. |
673
|
|
'+' should be on a new line. |
674
|
|
'+' should be on a new line. |
675
|
|
'+' should be on a new line. |
676
|
|
'+' should be on a new line. |
677
|
|
'+' should be on a new line. |
678
|
|
'+' should be on a new line. |
679
|
|
'+' should be on a new line. |
680
|
|
'+' should be on a new line. |
681
|
|
'+' should be on a new line. |
682
|
|
Missing a Javadoc comment. |
685
|
|
'+' should be on a new line. |
686
|
|
'+' should be on a new line. |
687
|
|
'+' should be on a new line. |
688
|
|
'+' should be on a new line. |
689
|
|
'+' should be on a new line. |
690
|
|
'+' should be on a new line. |
691
|
|
'+' should be on a new line. |
692
|
|
'+' should be on a new line. |
693
|
|
Missing a Javadoc comment. |
696
|
|
'+' should be on a new line. |
697
|
|
'+' should be on a new line. |
698
|
|
'+' should be on a new line. |
699
|
|
Line is longer than 80 characters. |
700
|
|
'+' should be on a new line. |
700
|
|
Missing a Javadoc comment. |
703
|
|
'+' should be on a new line. |
704
|
|
Line has trailing spaces. |
706
|
|
Missing a Javadoc comment. |
707
|
|
'+' should be on a new line. |
708
|
|
'+' should be on a new line. |
709
|
|
'+' should be on a new line. |
710
|
|
'+' should be on a new line. |
711
|
|
'+' should be on a new line. |
712
|
|
'+' should be on a new line. |
713
|
|
'+' should be on a new line. |
714
|
|
'+' should be on a new line. |
715
|
|
'+' should be on a new line. |
716
|
|
'+' should be on a new line. |
717
|
|
Missing a Javadoc comment. |
720
|
|
'+' should be on a new line. |
721
|
|
Line is longer than 80 characters. |
722
|
|
'+' should be on a new line. |
722
|
|
'+' should be on a new line. |
723
|
|
'+' should be on a new line. |
724
|
|
Line is longer than 80 characters. |
725
|
|
Line has trailing spaces. |
726
|
|
Missing a Javadoc comment. |
727
|
|
'+' should be on a new line. |
728
|
|
'+' should be on a new line. |
729
|
|
'+' should be on a new line. |
730
|
|
'+' should be on a new line. |
731
|
|
'+' should be on a new line. |
732
|
|
'+' should be on a new line. |
733
|
|
'+' should be on a new line. |
734
|
|
'+' should be on a new line. |
735
|
|
Missing a Javadoc comment. |
738
|
|
'+' should be on a new line. |
739
|
|
'+' should be on a new line. |
740
|
|
'+' should be on a new line. |
741
|
|
Line is longer than 80 characters. |
742
|
|
'+' should be on a new line. |
742
|
|
Line is longer than 80 characters. |
743
|
|
'+' should be on a new line. |
743
|
|
Line is longer than 80 characters. |
744
|
|
'+' should be on a new line. |
744
|
|
'+' should be on a new line. |
745
|
|
'+' should be on a new line. |
746
|
|
Missing a Javadoc comment. |
749
|
|
'+' should be on a new line. |
750
|
|
'+' should be on a new line. |
751
|
|
Line is longer than 80 characters. |
752
|
|
'+' should be on a new line. |
752
|
|
'+' should be on a new line. |
753
|
|
'+' should be on a new line. |
754
|
|
Line is longer than 80 characters. |
755
|
|
'+' should be on a new line. |
755
|
|
'+' should be on a new line. |
756
|
|
'+' should be on a new line. |
757
|
|
Missing a Javadoc comment. |
760
|
|
'+' should be on a new line. |
761
|
|
'+' should be on a new line. |
762
|
|
'+' should be on a new line. |
763
|
|
'+' should be on a new line. |
764
|
|
'+' should be on a new line. |
765
|
|
'+' should be on a new line. |
766
|
|
'+' should be on a new line. |
767
|
|
'+' should be on a new line. |
768
|
|
'+' should be on a new line. |
769
|
|
'+' should be on a new line. |
770
|
|
'+' should be on a new line. |
771
|
|
'+' should be on a new line. |
772
|
|
'+' should be on a new line. |
773
|
|
'+' should be on a new line. |
774
|
|
'+' should be on a new line. |
775
|
|
'+' should be on a new line. |
776
|
|
'+' should be on a new line. |
777
|
|
'+' should be on a new line. |
778
|
|
'+' should be on a new line. |
779
|
|
'+' should be on a new line. |
780
|
|
'+' should be on a new line. |
781
|
|
'+' should be on a new line. |
782
|
|
'+' should be on a new line. |
783
|
|
'+' should be on a new line. |
784
|
|
'+' should be on a new line. |
785
|
|
'+' should be on a new line. |
786
|
|
'+' should be on a new line. |
787
|
|
'+' should be on a new line. |
788
|
|
'+' should be on a new line. |
789
|
|
'+' should be on a new line. |
790
|
|
'+' should be on a new line. |
791
|
|
'+' should be on a new line. |
792
|
|
'+' should be on a new line. |
793
|
|
'+' should be on a new line. |
794
|
|
'+' should be on a new line. |
795
|
|
'+' should be on a new line. |
796
|
|
'+' should be on a new line. |
797
|
|
'+' should be on a new line. |
798
|
|
'+' should be on a new line. |
799
|
|
'+' should be on a new line. |
800
|
|
'+' should be on a new line. |
801
|
|
'+' should be on a new line. |
802
|
|
'+' should be on a new line. |
803
|
|
'+' should be on a new line. |
804
|
|
'+' should be on a new line. |
805
|
|
'+' should be on a new line. |
806
|
|
'+' should be on a new line. |
807
|
|
'+' should be on a new line. |
808
|
|
'+' should be on a new line. |
809
|
|
Missing a Javadoc comment. |
812
|
|
'+' should be on a new line. |
813
|
|
'+' should be on a new line. |
814
|
|
'+' should be on a new line. |
815
|
|
'+' should be on a new line. |
816
|
|
'+' should be on a new line. |
817
|
|
'+' should be on a new line. |
818
|
|
'+' should be on a new line. |
819
|
|
'+' should be on a new line. |
820
|
|
'+' should be on a new line. |
821
|
|
'+' should be on a new line. |
822
|
|
'+' should be on a new line. |
823
|
|
'+' should be on a new line. |
824
|
|
'+' should be on a new line. |
825
|
|
'+' should be on a new line. |
826
|
|
'+' should be on a new line. |
827
|
|
'+' should be on a new line. |
828
|
|
'+' should be on a new line. |
829
|
|
'+' should be on a new line. |
830
|
|
'+' should be on a new line. |
831
|
|
'+' should be on a new line. |
832
|
|
'+' should be on a new line. |
833
|
|
'+' should be on a new line. |
834
|
|
'+' should be on a new line. |
835
|
|
'+' should be on a new line. |
836
|
|
'+' should be on a new line. |
837
|
|
'+' should be on a new line. |
838
|
|
'+' should be on a new line. |
839
|
|
'+' should be on a new line. |
840
|
|
'+' should be on a new line. |
841
|
org/apache/catalina/manager/ManagerXServlet.java
| Error |
Line |
|
Line does not match expected header line of ''. |
1
|
|
Line has trailing spaces. |
3
|
|
Line has trailing spaces. |
7
|
|
Line has trailing spaces. |
9
|
|
Line is longer than 80 characters. |
156
|
|
Line has trailing spaces. |
157
|
|
Variable 'configBase' must be private and have accessor methods. |
171
|
|
Variable 'context' must be private and have accessor methods. |
177
|
|
Variable 'debug' must be private and have accessor methods. |
183
|
|
Variable 'versioned' must be private and have accessor methods. |
189
|
|
Variable 'contextDescriptors' must be private and have accessor methods. |
195
|
|
First sentence should end with a period. |
198
|
|
Variable 'engine' must be private and have accessor methods. |
201
|
|
Variable 'appBase' must be private and have accessor methods. |
207
|
|
Line has trailing spaces. |
208
|
|
Line has trailing spaces. |
209
|
|
Variable 'mBeanServer' must be private and have accessor methods. |
213
|
|
Variable 'global' must be private and have accessor methods. |
220
|
|
Variable 'sm' must be private and have accessor methods. |
226
|
|
Variable 'wrapper' must be private and have accessor methods. |
233
|
|
Expected an @return tag. |
242
|
|
Method 'getWrapper' is not designed for extension - needs to be abstract, final or empty. |
242
|
|
Method 'setWrapper' is not designed for extension - needs to be abstract, final or empty. |
254
|
|
Parameter wrapper should be final. |
254
|
|
'wrapper' hides a field. |
254
|
|
Line has trailing spaces. |
267
|
|
Method 'destroy' is not designed for extension - needs to be abstract, final or empty. |
277
|
|
Empty statement. |
279
|
|
Method 'doGet' is not designed for extension - needs to be abstract, final or empty. |
293
|
|
Parameter request should be final. |
293
|
|
Parameter response should be final. |
294
|
|
'if' construct must use '{}'s. |
298
|
|
'if' construct must use '{}'s. |
304
|
|
Line has trailing spaces. |
313
|
|
Method 'doPut' is not designed for extension - needs to be abstract, final or empty. |
380
|
|
Parameter request should be final. |
380
|
|
Parameter response should be final. |
381
|
|
'if' construct must use '{}'s. |
385
|
|
'if' construct must use '{}'s. |
391
|
|
Line has trailing spaces. |
397
|
|
'+' is not preceded with whitespace. |
403
|
|
'+' is not followed by whitespace. |
403
|
|
Method 'init' is not designed for extension - needs to be abstract, final or empty. |
426
|
|
Expected @throws tag for 'ServletException'. |
426
|
|
'if' construct must use '{}'s. |
429
|
|
'if' construct must use '{}'s. |
435
|
|
'if' construct must use '{}'s. |
437
|
|
Empty statement. |
447
|
|
'engine' hides a field. |
463
|
|
'if' construct must use '{}'s. |
465
|
|
'if' construct must use '{}'s. |
467
|
|
'+' should be on a new line. |
481
|
|
Method 'vhosts' is not designed for extension - needs to be abstract, final or empty. |
500
|
|
Parameter writer should be final. |
500
|
|
'if' construct must use '{}'s. |
502
|
|
'+' should be on a new line. |
503
|
|
Line has trailing spaces. |
508
|
|
Array brackets at illegal position. |
509
|
|
Line has trailing spaces. |
510
|
|
Line has trailing spaces. |
512
|
|
Line has trailing spaces. |
514
|
|
Line has trailing spaces. |
516
|
|
Line has trailing spaces. |
519
|
|
Line is longer than 80 characters. |
523
|
|
Method 'save' is not designed for extension - needs to be abstract, final or empty. |
523
|
|
Parameter writer should be final. |
523
|
|
Expected @param tag for 'writer'. |
523
|
|
Parameter vhost should be final. |
523
|
|
Parameter path should be final. |
523
|
|
'if' is not followed by whitespace. |
532
|
|
Line is longer than 80 characters. |
533
|
|
Line has trailing spaces. |
535
|
|
Line is longer than 80 characters. |
539
|
|
Line has trailing spaces. |
540
|
|
'context' hides a field. |
558
|
|
Line has trailing spaces. |
565
|
|
Method 'deploy' is not designed for extension - needs to be abstract, final or empty. |
588
|
|
Parameter writer should be final. |
589
|
|
Parameter vhost should be final. |
589
|
|
Parameter path should be final. |
589
|
|
Parameter tag should be final. |
590
|
|
Parameter update should be final. |
590
|
|
Expected @param tag for 'update'. |
590
|
|
Parameter request should be final. |
590
|
|
Line is longer than 80 characters. |
593
|
|
'if' is not followed by whitespace. |
596
|
|
Line is longer than 80 characters. |
597
|
|
Line has trailing spaces. |
599
|
|
Line is longer than 80 characters. |
603
|
|
Line has trailing spaces. |
604
|
|
'if' construct must use '{}'s. |
613
|
|
'if' is not followed by whitespace. |
613
|
|
'(' is followed by whitespace. |
613
|
|
')' is preceded with whitespace. |
613
|
|
'context' hides a field. |
618
|
|
Line is longer than 80 characters. |
655
|
|
Line is longer than 80 characters. |
658
|
|
Line has trailing spaces. |
672
|
|
Line is longer than 80 characters. |
675
|
|
Line is longer than 80 characters. |
678
|
|
Line has trailing spaces. |
680
|
|
Line is longer than 80 characters. |
693
|
|
Method 'deploy' is not designed for extension - needs to be abstract, final or empty. |
693
|
|
Parameter writer should be final. |
693
|
|
Parameter vhost should be final. |
693
|
|
Parameter path should be final. |
693
|
|
Parameter tag should be final. |
693
|
|
'if' is not followed by whitespace. |
695
|
|
Line is longer than 80 characters. |
696
|
|
Line has trailing spaces. |
698
|
|
Line is longer than 80 characters. |
702
|
|
Line has trailing spaces. |
703
|
|
'if' construct must use '{}'s. |
712
|
|
'if' is not followed by whitespace. |
712
|
|
'(' is followed by whitespace. |
712
|
|
')' is preceded with whitespace. |
712
|
|
'context' hides a field. |
727
|
|
Line is longer than 80 characters. |
737
|
|
Line has trailing spaces. |
750
|
|
Line is longer than 80 characters. |
753
|
|
Line is longer than 80 characters. |
756
|
|
Line has trailing spaces. |
758
|
|
Method 'deploy' is not designed for extension - needs to be abstract, final or empty. |
773
|
|
Parameter writer should be final. |
773
|
|
Parameter vhost should be final. |
773
|
|
Parameter config should be final. |
773
|
|
Parameter path should be final. |
774
|
|
Parameter war should be final. |
774
|
|
Parameter update should be final. |
774
|
|
'if' is not followed by whitespace. |
776
|
|
Line is longer than 80 characters. |
777
|
|
Line has trailing spaces. |
779
|
|
Line is longer than 80 characters. |
783
|
|
Line has trailing spaces. |
784
|
|
Line has trailing spaces. |
793
|
|
'+' should be on a new line. |
797
|
|
'+' should be on a new line. |
800
|
|
'+' should be on a new line. |
805
|
|
Line is longer than 80 characters. |
808
|
|
Line has trailing spaces. |
812
|
|
'if' is not followed by whitespace. |
819
|
|
Line has trailing spaces. |
822
|
|
'context' hides a field. |
824
|
|
Line has trailing spaces. |
837
|
|
Line has trailing spaces. |
844
|
|
Line has trailing spaces. |
850
|
|
Line is longer than 80 characters. |
851
|
|
Line has trailing spaces. |
855
|
|
Line is longer than 80 characters. |
856
|
|
Line has trailing spaces. |
858
|
|
Line is longer than 80 characters. |
859
|
|
Line is longer than 80 characters. |
870
|
|
Line is longer than 80 characters. |
873
|
|
Line has trailing spaces. |
880
|
|
Method 'list' is not designed for extension - needs to be abstract, final or empty. |
890
|
|
Parameter writer should be final. |
890
|
|
Parameter vhost should be final. |
890
|
|
'if' construct must use '{}'s. |
892
|
|
'+' should be on a new line. |
893
|
|
'if' is not followed by whitespace. |
896
|
|
Line is longer than 80 characters. |
897
|
|
Line has trailing spaces. |
899
|
|
Line is longer than 80 characters. |
903
|
|
Line has trailing spaces. |
904
|
|
'context' hides a field. |
911
|
|
'if' construct must use '{}'s. |
913
|
|
'if' is not followed by whitespace. |
913
|
|
'(' is followed by whitespace. |
913
|
|
')' is preceded with whitespace. |
913
|
|
')' is preceded with whitespace. |
915
|
|
Line is longer than 80 characters. |
920
|
|
Method 'reload' is not designed for extension - needs to be abstract, final or empty. |
941
|
|
Parameter writer should be final. |
941
|
|
Parameter vhost should be final. |
941
|
|
Parameter path should be final. |
941
|
|
'if' construct must use '{}'s. |
943
|
|
Line is longer than 80 characters. |
944
|
|
'if' is not followed by whitespace. |
946
|
|
Line is longer than 80 characters. |
947
|
|
Line has trailing spaces. |
949
|
|
Line is longer than 80 characters. |
953
|
|
Line has trailing spaces. |
954
|
|
'if' construct must use '{}'s. |
963
|
|
'if' is not followed by whitespace. |
963
|
|
'(' is followed by whitespace. |
963
|
|
')' is preceded with whitespace. |
963
|
|
'context' hides a field. |
967
|
|
Method 'resources' is not designed for extension - needs to be abstract, final or empty. |
997
|
|
Parameter writer should be final. |
997
|
|
Expected @param tag for 'writer'. |
997
|
|
Parameter type should be final. |
997
|
|
Method 'printResources' is not designed for extension - needs to be abstract, final or empty. |
1041
|
|
Parameter writer should be final. |
1041
|
|
Expected @param tag for 'writer'. |
1041
|
|
Parameter prefix should be final. |
1041
|
|
Expected @param tag for 'prefix'. |
1041
|
|
Parameter namingContext should be final. |
1042
|
|
Expected @param tag for 'namingContext'. |
1042
|
|
Parameter type should be final. |
1043
|
|
Expected @param tag for 'type'. |
1043
|
|
Parameter clazz should be final. |
1043
|
|
Expected @param tag for 'clazz'. |
1043
|
|
'&&' should be on a new line. |
1054
|
|
Method 'roles' is not designed for extension - needs to be abstract, final or empty. |
1083
|
|
Parameter writer should be final. |
1083
|
|
Method 'serverinfo' is not designed for extension - needs to be abstract, final or empty. |
1127
|
|
Parameter writer should be final. |
1127
|
|
'if' construct must use '{}'s. |
1128
|
|
',' is not followed by whitespace. |
1147
|
|
Line is longer than 80 characters. |
1154
|
|
Line is longer than 80 characters. |
1160
|
|
Method 'sessions' is not designed for extension - needs to be abstract, final or empty. |
1162
|
|
Parameter writer should be final. |
1162
|
|
Parameter vhost should be final. |
1162
|
|
Parameter path should be final. |
1162
|
|
'if' construct must use '{}'s. |
1164
|
|
Line is longer than 80 characters. |
1165
|
|
'if' is not followed by whitespace. |
1167
|
|
Line is longer than 80 characters. |
1168
|
|
Line has trailing spaces. |
1170
|
|
Line is longer than 80 characters. |
1174
|
|
Line has trailing spaces. |
1175
|
|
'if' construct must use '{}'s. |
1184
|
|
'if' is not followed by whitespace. |
1184
|
|
'(' is followed by whitespace. |
1184
|
|
')' is preceded with whitespace. |
1184
|
|
'context' hides a field. |
1187
|
|
Line is longer than 80 characters. |
1193
|
|
Line is longer than 80 characters. |
1195
|
|
'/' is not preceded with whitespace. |
1195
|
|
'/' is not followed by whitespace. |
1195
|
|
'60' is a magic number. |
1195
|
|
'60' is a magic number. |
1197
|
|
'/' is not preceded with whitespace. |
1200
|
|
'/' is not followed by whitespace. |
1200
|
|
'10' is a magic number. |
1200
|
|
'*' is not preceded with whitespace. |
1200
|
|
'*' is not followed by whitespace. |
1200
|
|
'60' is a magic number. |
1200
|
|
'if' construct must use '{}'s. |
1201
|
|
'if' construct must use '{}'s. |
1203
|
|
'-' is not preceded with whitespace. |
1204
|
|
'-' is not followed by whitespace. |
1204
|
|
'else' construct must use '{}'s. |
1205
|
|
'if' construct must use '{}'s. |
1208
|
|
'-' is not preceded with whitespace. |
1211
|
|
'-' is not followed by whitespace. |
1211
|
|
'if' construct must use '{}'s. |
1212
|
|
'*' is not preceded with whitespace. |
1214
|
|
'*' is not followed by whitespace. |
1214
|
|
'10' is a magic number. |
1214
|
|
'+' is not preceded with whitespace. |
1214
|
|
'+' is not followed by whitespace. |
1214
|
|
'*' is not preceded with whitespace. |
1214
|
|
'*' is not followed by whitespace. |
1214
|
|
'10' is a magic number. |
1214
|
|
'if' construct must use '{}'s. |
1217
|
|
'-' is not preceded with whitespace. |
1217
|
|
'-' is not followed by whitespace. |
1217
|
|
'*' is not preceded with whitespace. |
1219
|
|
'*' is not followed by whitespace. |
1219
|
|
'10' is a magic number. |
1219
|
|
'-' is not preceded with whitespace. |
1220
|
|
'-' is not followed by whitespace. |
1220
|
|
'if' construct must use '{}'s. |
1221
|
|
',' is not followed by whitespace. |
1223
|
|
Method 'start' is not designed for extension - needs to be abstract, final or empty. |
1240
|
|
Parameter writer should be final. |
1240
|
|
Parameter vhost should be final. |
1240
|
|
Parameter path should be final. |
1240
|
|
'if' construct must use '{}'s. |
1242
|
|
Line is longer than 80 characters. |
1243
|
|
'if' is not followed by whitespace. |
1245
|
|
Line is longer than 80 characters. |
1246
|
|
Line has trailing spaces. |
1248
|
|
Line is longer than 80 characters. |
1252
|
|
Line has trailing spaces. |
1253
|
|
'if' construct must use '{}'s. |
1262
|
|
'if' is not followed by whitespace. |
1262
|
|
'(' is followed by whitespace. |
1262
|
|
')' is preceded with whitespace. |
1262
|
|
'context' hides a field. |
1266
|
|
Line has trailing spaces. |
1268
|
|
'if' construct must use '{}'s. |
1273
|
|
'else' construct must use '{}'s. |
1276
|
|
Method 'stop' is not designed for extension - needs to be abstract, final or empty. |
1298
|
|
Parameter writer should be final. |
1298
|
|
Parameter vhost should be final. |
1298
|
|
Parameter path should be final. |
1298
|
|
'if' construct must use '{}'s. |
1300
|
|
Line is longer than 80 characters. |
1301
|
|
'if' is not followed by whitespace. |
1303
|
|
Line is longer than 80 characters. |
1304
|
|
Line has trailing spaces. |
1306
|
|
Line is longer than 80 characters. |
1310
|
|
Line has trailing spaces. |
1311
|
|
'if' construct must use '{}'s. |
1320
|
|
'if' is not followed by whitespace. |
1320
|
|
'(' is followed by whitespace. |
1320
|
|
')' is preceded with whitespace. |
1320
|
|
'context' hides a field. |
1324
|
|
Line has trailing spaces. |
1326
|
|
Method 'undeploy' is not designed for extension - needs to be abstract, final or empty. |
1353
|
|
Parameter writer should be final. |
1353
|
|
Parameter vhost should be final. |
1353
|
|
Parameter path should be final. |
1353
|
|
'if' construct must use '{}'s. |
1355
|
|
Line is longer than 80 characters. |
1356
|
|
'if' is not followed by whitespace. |
1358
|
|
Line is longer than 80 characters. |
1359
|
|
Line has trailing spaces. |
1361
|
|
Line is longer than 80 characters. |
1365
|
|
Line has trailing spaces. |
1366
|
|
'if' construct must use '{}'s. |
1375
|
|
'if' is not followed by whitespace. |
1375
|
|
'(' is followed by whitespace. |
1375
|
|
')' is preceded with whitespace. |
1375
|
|
'context' hides a field. |
1381
|
|
'appBase' hides a field. |
1389
|
|
Must have at least one statement. |
1405
|
|
Line is longer than 80 characters. |
1409
|
|
Line is longer than 80 characters. |
1411
|
|
Expected an @return tag. |
1442
|
|
Method 'getConfigFile' is not designed for extension - needs to be abstract, final or empty. |
1442
|
|
Parameter path should be final. |
1442
|
|
Expected @param tag for 'path'. |
1442
|
|
Expected an @return tag. |
1456
|
|
Method 'getDocBase' is not designed for extension - needs to be abstract, final or empty. |
1456
|
|
Parameter path should be final. |
1456
|
|
Expected @param tag for 'path'. |
1456
|
|
Line has trailing spaces. |
1466
|
|
Expected an @return tag. |
1471
|
|
Method 'getAppBase' is not designed for extension - needs to be abstract, final or empty. |
1471
|
|
Parameter host should be final. |
1471
|
|
Expected @param tag for 'host'. |
1471
|
|
'if' construct must use '{}'s. |
1478
|
|
Line has trailing spaces. |
1494
|
|
Method 'check' is not designed for extension - needs to be abstract, final or empty. |
1494
|
|
Parameter host should be final. |
1494
|
|
Expected @param tag for 'host'. |
1494
|
|
Parameter name should be final. |
1494
|
|
Expected @param tag for 'name'. |
1494
|
|
Expected @throws tag for 'Exception'. |
1495
|
|
'{' is followed by whitespace. |
1496
|
|
'{' is followed by whitespace. |
1497
|
|
Line has trailing spaces. |
1500
|
|
Expected an @return tag. |
1505
|
|
Line has trailing spaces. |
1505
|
|
Method 'isServiced' is not designed for extension - needs to be abstract, final or empty. |
1505
|
|
Parameter host should be final. |
1505
|
|
Expected @param tag for 'host'. |
1505
|
|
Parameter name should be final. |
1505
|
|
Expected @param tag for 'name'. |
1505
|
|
Expected @throws tag for 'Exception'. |
1506
|
|
'{' is followed by whitespace. |
1507
|
|
'{' is followed by whitespace. |
1508
|
|
Line has trailing spaces. |
1509
|
|
Line is longer than 80 characters. |
1510
|
|
Line has trailing spaces. |
1513
|
|
Line has trailing spaces. |
1518
|
|
Method 'addServiced' is not designed for extension - needs to be abstract, final or empty. |
1518
|
|
Parameter host should be final. |
1518
|
|
Expected @param tag for 'host'. |
1518
|
|
Parameter name should be final. |
1518
|
|
Expected @param tag for 'name'. |
1518
|
|
Expected @throws tag for 'Exception'. |
1519
|
|
'{' is followed by whitespace. |
1520
|
|
'{' is followed by whitespace. |
1521
|
|
Line is longer than 80 characters. |
1522
|
|
Line has trailing spaces. |
1524
|
|
Line has trailing spaces. |
1529
|
|
Method 'removeServiced' is not designed for extension - needs to be abstract, final or empty. |
1529
|
|
Parameter host should be final. |
1529
|
|
Expected @param tag for 'host'. |
1529
|
|
Parameter name should be final. |
1529
|
|
Expected @param tag for 'name'. |
1529
|
|
Expected @throws tag for 'Exception'. |
1530
|
|
'{' is followed by whitespace. |
1531
|
|
'{' is followed by whitespace. |
1532
|
|
Line is longer than 80 characters. |
1533
|
|
Line has trailing spaces. |
1535
|
|
Method 'undeployDir' is not designed for extension - needs to be abstract, final or empty. |
1543
|
|
Parameter dir should be final. |
1543
|
|
Array brackets at illegal position. |
1545
|
|
Method 'uploadWar' is not designed for extension - needs to be abstract, final or empty. |
1571
|
|
Parameter request should be final. |
1571
|
|
Parameter war should be final. |
1571
|
|
'1024' is a magic number. |
1580
|
|
Array brackets at illegal position. |
1581
|
|
'1024' is a magic number. |
1581
|
|
Empty statement. |
1602
|
|
Empty statement. |
1610
|
|
Expected an @return tag. |
1625
|
|
Parameter src should be final. |
1625
|
|
Parameter dest should be final. |
1625
|
|
'if' is not followed by whitespace. |
1628
|
|
'(' is followed by whitespace. |
1628
|
|
'&&' should be on a new line. |
1628
|
|
')' is preceded with whitespace. |
1629
|
|
'4096' is a magic number. |
1630
|
|
Line has trailing spaces. |
1638
|
|
Expected an @return tag. |
1645
|
|
Parameter src should be final. |
1645
|
|
Parameter dest should be final. |
1645
|
|
Parameter buf should be final. |
1645
|
|
Expected @param tag for 'buf'. |
1645
|
|
Line has trailing spaces. |
1646
|
|
Line has trailing spaces. |
1648
|
|
Array brackets at illegal position. |
1649
|
|
'if' construct must use '{}'s. |
1674
|
|
Must have at least one statement. |
1685
|
|
Must have at least one statement. |
1691
|
|
Line has trailing spaces. |
1698
|
|
First sentence should end with a period. |
1702
|
|
Line has trailing spaces. |
1705
|
|
Line has trailing spaces. |
1707
|
|
Line has trailing spaces. |
1708
|
|
Parameter context should be final. |
1710
|
|
'context' hides a field. |
1710
|
|
Line has trailing spaces. |
1711
|
|
Line has trailing spaces. |
1713
|
|
'appBase' hides a field. |
1714
|
|
Line has trailing spaces. |
1720
|
|
Line has trailing spaces. |
1722
|
|
Line has trailing spaces. |
1724
|
|
Line has trailing spaces. |
1725
|
|
First sentence should end with a period. |
1727
|
|
Line has trailing spaces. |
1730
|
|
Line has trailing spaces. |
1732
|
|
Line has trailing spaces. |
1733
|
|
Parameter host should be final. |
1735
|
|
Line has trailing spaces. |
1736
|
|
Line has trailing spaces. |
1739
|
|
Must have at least one statement. |
1741
|
|
Line has trailing spaces. |
1744
|
|
Line has trailing spaces. |
1746
|
|
Line has trailing spaces. |
1748
|
org/apache/catalina/manager/StatusManagerXServlet.java
| Error |
Line |
|
Line does not match expected header line of ''. |
1
|
|
Line has trailing spaces. |
3
|
|
Line has trailing spaces. |
7
|
|
Line has trailing spaces. |
9
|
|
Using the '.*' form of import should be avoided - javax.management.*. |
21
|
|
Line is longer than 80 characters. |
43
|
|
Variable 'mBeanServer' must be private and have accessor methods. |
62
|
|
Variable 'protocolHandlers' must be private and have accessor methods. |
68
|
|
Variable 'threadPools' must be private and have accessor methods. |
74
|
|
Variable 'requestProcessors' must be private and have accessor methods. |
80
|
|
Variable 'globalRequestProcessors' must be private and have accessor methods. |
86
|
|
Variable 'sm' must be private and have accessor methods. |
92
|
|
Method 'init' is not designed for extension - needs to be abstract, final or empty. |
102
|
|
Expected @throws tag for 'ServletException'. |
102
|
|
Empty statement. |
113
|
|
Method 'destroy' is not designed for extension - needs to be abstract, final or empty. |
173
|
|
Empty statement. |
175
|
|
Method 'doGet' is not designed for extension - needs to be abstract, final or empty. |
189
|
|
Parameter request should be final. |
189
|
|
Parameter response should be final. |
190
|
|
Line has trailing spaces. |
196
|
|
Line has trailing spaces. |
205
|
|
',' is not followed by whitespace. |
210
|
|
',' is not followed by whitespace. |
221
|
|
',' is not followed by whitespace. |
221
|
|
'9' is a magic number. |
224
|
|
'3' is a magic number. |
228
|
|
'+' should be on a new line. |
229
|
|
'4' is a magic number. |
231
|
|
'5' is a magic number. |
232
|
|
'+' should be on a new line. |
233
|
|
'6' is a magic number. |
235
|
|
'7' is a magic number. |
237
|
|
'8' is a magic number. |
239
|
|
'7' is a magic number. |
241
|
|
'8' is a magic number. |
243
|
|
',' is not followed by whitespace. |
246
|
|
',' is not followed by whitespace. |
246
|
|
'7' is a magic number. |
249
|
|
'3' is a magic number. |
253
|
|
'4' is a magic number. |
254
|
|
'5' is a magic number. |
255
|
|
'6' is a magic number. |
256
|
|
',' is not followed by whitespace. |
258
|
|
',' is not followed by whitespace. |
258
|
|
'6' is a magic number. |
261
|
|
'3' is a magic number. |
265
|
|
'4' is a magic number. |
266
|
|
'5' is a magic number. |
267
|
|
',' is not followed by whitespace. |
274
|
|
Line contains a tab character. |
276
|
|
',' is not followed by whitespace. |
277
|
|
Line has trailing spaces. |
290
|
|
Method 'handleNotification' is not designed for extension - needs to be abstract, final or empty. |
310
|
|
Missing a Javadoc comment. |
310
|
|
Parameter notification should be final. |
310
|
|
Parameter handback should be final. |
311
|
|
Line has trailing spaces. |
314
|
|
Must have at least one statement. |
345
|
|
Line has trailing spaces. |
346
|
org/apache/catalina/manager/package.html
| Error |
Line |
|
Missing package documentation file. |
0 |
org/talika/tms/Constants.java
| Error |
Line |
|
Missing a header - not enough lines in file. |
1
|
|
Line has trailing spaces. |
3
|
|
Line has trailing spaces. |
7
|
|
Line has trailing spaces. |
9
|
|
Missing a Javadoc comment. |
21
|
|
Missing a Javadoc comment. |
23
|
|
Name 'Package' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. |
23
|
|
Line has trailing spaces. |
24
|
|
Missing a Javadoc comment. |
25
|
|
Missing a Javadoc comment. |
27
|
|
'+' should be on a new line. |
28
|
|
'+' should be on a new line. |
29
|
|
'+' should be on a new line. |
30
|
|
'+' should be on a new line. |
31
|
|
'+' should be on a new line. |
32
|
|
'+' should be on a new line. |
33
|
|
'+' should be on a new line. |
34
|
|
'+' should be on a new line. |
35
|
|
'+' should be on a new line. |
36
|
|
'+' should be on a new line. |
37
|
|
'+' should be on a new line. |
38
|
|
'+' should be on a new line. |
39
|
|
'+' should be on a new line. |
40
|
|
'+' should be on a new line. |
41
|
|
'+' should be on a new line. |
42
|
|
'+' should be on a new line. |
43
|
|
'+' should be on a new line. |
44
|
|
'+' should be on a new line. |
45
|
|
'+' should be on a new line. |
46
|
|
'+' should be on a new line. |
47
|
|
'+' should be on a new line. |
48
|
|
'+' should be on a new line. |
49
|
|
'+' should be on a new line. |
50
|
|
'+' should be on a new line. |
51
|
|
'+' should be on a new line. |
52
|
|
'+' should be on a new line. |
53
|
|
'+' should be on a new line. |
54
|
|
'+' should be on a new line. |
55
|
|
'+' should be on a new line. |
56
|
|
'+' should be on a new line. |
57
|
|
'+' should be on a new line. |
58
|
|
'+' should be on a new line. |
59
|
|
'+' should be on a new line. |
60
|
|
'+' should be on a new line. |
61
|
|
'+' should be on a new line. |
62
|
|
'+' should be on a new line. |
63
|
|
'+' should be on a new line. |
64
|
|
'+' should be on a new line. |
65
|
|
'+' should be on a new line. |
66
|
|
'+' should be on a new line. |
67
|
|
'+' should be on a new line. |
68
|
|
'+' should be on a new line. |
69
|
|
'+' should be on a new line. |
70
|
|
'+' should be on a new line. |
71
|
|
'+' should be on a new line. |
72
|
|
'+' should be on a new line. |
73
|
|
'+' should be on a new line. |
74
|
|
'+' should be on a new line. |
75
|
|
'+' should be on a new line. |
76
|
|
'+' should be on a new line. |
77
|
|
'+' should be on a new line. |
78
|
|
'+' should be on a new line. |
79
|
|
'+' should be on a new line. |
80
|
|
'+' should be on a new line. |
81
|
|
'+' should be on a new line. |
82
|
|
'+' should be on a new line. |
83
|
|
'+' should be on a new line. |
84
|
|
'+' should be on a new line. |
85
|
|
'+' should be on a new line. |
86
|
|
'+' should be on a new line. |
87
|
|
'+' should be on a new line. |
88
|
|
'+' should be on a new line. |
89
|
|
'+' should be on a new line. |
90
|
|
'+' should be on a new line. |
91
|
|
'+' should be on a new line. |
92
|
|
'+' should be on a new line. |
93
|
|
'+' should be on a new line. |
94
|
|
'+' should be on a new line. |
95
|
|
Missing a Javadoc comment. |
98
|
|
'+' should be on a new line. |
99
|
|
'+' should be on a new line. |
100
|
|
'+' should be on a new line. |
101
|
|
'+' should be on a new line. |
102
|
|
'+' should be on a new line. |
103
|
|
'+' should be on a new line. |
104
|
|
'+' should be on a new line. |
105
|
|
'+' should be on a new line. |
106
|
|
'+' should be on a new line. |
107
|
|
Line is longer than 80 characters. |
108
|
|
'+' should be on a new line. |
108
|
|
'+' should be on a new line. |
109
|
|
'+' should be on a new line. |
110
|
|
'+' should be on a new line. |
111
|
|
'+' should be on a new line. |
112
|
|
'+' should be on a new line. |
113
|
|
'+' should be on a new line. |
114
|
|
'+' should be on a new line. |
115
|
|
'+' should be on a new line. |
116
|
|
'+' should be on a new line. |
117
|
|
'+' should be on a new line. |
118
|
|
'+' should be on a new line. |
119
|
|
'+' should be on a new line. |
120
|
|
'+' should be on a new line. |
121
|
|
'+' should be on a new line. |
122
|
|
'+' should be on a new line. |
123
|
|
'+' should be on a new line. |
124
|
|
'+' should be on a new line. |
125
|
|
'+' should be on a new line. |
126
|
|
'+' should be on a new line. |
127
|
|
Missing a Javadoc comment. |
130
|
|
'+' should be on a new line. |
131
|
|
'+' should be on a new line. |
132
|
|
'+' should be on a new line. |
133
|
|
'+' should be on a new line. |
134
|
|
'+' should be on a new line. |
135
|
|
'+' should be on a new line. |
136
|
|
'+' should be on a new line. |
137
|
|
'+' should be on a new line. |
138
|
|
Missing a Javadoc comment. |
141
|
|
'+' should be on a new line. |
142
|
|
'+' should be on a new line. |
143
|
|
'+' should be on a new line. |
144
|
|
'+' should be on a new line. |
145
|
|
'+' should be on a new line. |
146
|
|
'+' should be on a new line. |
147
|
|
'+' should be on a new line. |
148
|
|
'+' should be on a new line. |
149
|
|
'+' should be on a new line. |
150
|
|
'+' should be on a new line. |
151
|
|
'+' should be on a new line. |
152
|
|
Missing a Javadoc comment. |
155
|
|
'+' should be on a new line. |
156
|
|
'+' should be on a new line. |
157
|
|
'+' should be on a new line. |
158
|
|
'+' should be on a new line. |
159
|
|
'+' should be on a new line. |
160
|
|
'+' should be on a new line. |
161
|
|
'+' should be on a new line. |
162
|
|
'+' should be on a new line. |
163
|
|
'+' should be on a new line. |
164
|
|
'+' should be on a new line. |
165
|
|
Missing a Javadoc comment. |
168
|
|
'+' should be on a new line. |
169
|
|
'+' should be on a new line. |
170
|
|
'+' should be on a new line. |
171
|
|
'+' should be on a new line. |
172
|
|
'+' should be on a new line. |
173
|
|
'+' should be on a new line. |
174
|
|
Missing a Javadoc comment. |
176
|
|
'=' is not preceded with whitespace. |
176
|
|
'=' is not followed by whitespace. |
176
|
|
Missing a Javadoc comment. |
178
|
|
'+' is not preceded with whitespace. |
179
|
|
'+' is not followed by whitespace. |
179
|
|
'+' is not preceded with whitespace. |
179
|
|
'+' is not followed by whitespace. |
179
|
|
Line has trailing spaces. |
180
|
|
Line contains a tab character. |
180
|
|
Missing a Javadoc comment. |
181
|
org/talika/tms/package.html
| Error |
Line |
|
Missing package documentation file. |
0 |