Checkstyle Results
The following document contains the results of
Checkstyle
.
Summary
Files | Infos | Warnings | Errors |
23 | 0 | 0 | 303 |
Files
org/apache/excalibur/instrument/AbstractInstrument.java
Error | Line |
| Line has trailing spaces. |
1
|
| Missing a header - not enough lines in file. |
1
|
| Line has trailing spaces. |
5
|
| Line has trailing spaces. |
6
|
| Line has trailing spaces. |
8
|
| Line has trailing spaces. |
13
|
| '{' should be on the previous line. |
30
|
| Name 'm_name' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
32
|
| Name 'm_proxy' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
35
|
| '(' is followed by whitespace. |
46
|
| Parameter name should be final. |
46
|
| ')' is preceded with whitespace. |
46
|
| '{' should be on the previous line. |
47
|
| Method 'getInstrumentName' is not designed for extension - needs to be abstract, final or empty. |
62
|
| '{' should be on the previous line. |
63
|
| Method 'setInstrumentProxy' is not designed for extension - needs to be abstract, final or empty. |
81
|
| '(' is followed by whitespace. |
81
|
| Parameter proxy should be final. |
81
|
| ')' is preceded with whitespace. |
81
|
| '{' should be on the previous line. |
82
|
| 'if' is not followed by whitespace. |
83
|
| '(' is followed by whitespace. |
83
|
| ')' is preceded with whitespace. |
83
|
| '{' should be on the previous line. |
84
|
| Line is longer than 80 characters. |
86
|
| ')' is preceded with whitespace. |
86
|
| Method 'isActive' is not designed for extension - needs to be abstract, final or empty. |
97
|
| '{' should be on the previous line. |
98
|
| '(' is followed by whitespace. |
99
|
| ')' is preceded with whitespace. |
99
|
| '(' is followed by whitespace. |
99
|
| ')' is preceded with whitespace. |
99
|
| Method 'getInstrumentProxy' is not designed for extension - needs to be abstract, final or empty. |
108
|
| '{' should be on the previous line. |
109
|
org/apache/excalibur/instrument/AbstractInstrumentable.java
Error | Line |
| Line does not match expected header line of ''. |
1
|
| Line has trailing spaces. |
1
|
| Line has trailing spaces. |
5
|
| Line has trailing spaces. |
6
|
| Line has trailing spaces. |
8
|
| Line has trailing spaces. |
13
|
| Line is longer than 80 characters. |
23
|
| '{' should be on the previous line. |
33
|
| Name 'm_instrumentableName' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
35
|
| Name 'm_instrumentList' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
38
|
| Name 'm_childList' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
41
|
| Line is longer than 80 characters. |
43
|
| Name 'm_registered' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
44
|
| '{' should be on the previous line. |
53
|
| Method 'addInstrument' is not designed for extension - needs to be abstract, final or empty. |
69
|
| '(' is followed by whitespace. |
69
|
| Parameter instrument should be final. |
69
|
| ')' is preceded with whitespace. |
69
|
| '{' should be on the previous line. |
70
|
| 'if' is not followed by whitespace. |
71
|
| '(' is followed by whitespace. |
71
|
| ')' is preceded with whitespace. |
71
|
| '{' should be on the previous line. |
72
|
| Line is longer than 80 characters. |
73
|
| '(' is followed by whitespace. |
73
|
| Line is longer than 80 characters. |
74
|
| ')' is preceded with whitespace. |
74
|
| '(' is followed by whitespace. |
76
|
| ')' is preceded with whitespace. |
76
|
| Method 'addChildInstrumentable' is not designed for extension - needs to be abstract, final or empty. |
89
|
| '(' is followed by whitespace. |
89
|
| Parameter child should be final. |
89
|
| ')' is preceded with whitespace. |
89
|
| '{' should be on the previous line. |
90
|
| 'if' is not followed by whitespace. |
91
|
| '(' is followed by whitespace. |
91
|
| ')' is preceded with whitespace. |
91
|
| '{' should be on the previous line. |
92
|
| Line is longer than 80 characters. |
93
|
| '(' is followed by whitespace. |
93
|
| Line is longer than 80 characters. |
94
|
| ')' is preceded with whitespace. |
94
|
| '(' is followed by whitespace. |
96
|
| ')' is preceded with whitespace. |
96
|
| '{' should be on the previous line. |
108
|
| '(' is followed by whitespace. |
125
|
| Parameter name should be final. |
125
|
| ')' is preceded with whitespace. |
125
|
| '{' should be on the previous line. |
126
|
| '{' should be on the previous line. |
140
|
| 'if' is not followed by whitespace. |
142
|
| '(' is followed by whitespace. |
142
|
| ')' is preceded with whitespace. |
142
|
| '{' should be on the previous line. |
143
|
| '}' should be on the same line. |
145
|
| '{' should be on the previous line. |
147
|
| Line is longer than 80 characters. |
148
|
| '(' is followed by whitespace. |
149
|
| ')' is preceded with whitespace. |
149
|
| '{' should be on the previous line. |
166
|
| 'if' is not followed by whitespace. |
168
|
| '(' is followed by whitespace. |
168
|
| ')' is preceded with whitespace. |
168
|
| '{' should be on the previous line. |
169
|
| '}' should be on the same line. |
171
|
| '{' should be on the previous line. |
173
|
| Line is longer than 80 characters. |
174
|
| '(' is followed by whitespace. |
175
|
| ')' is preceded with whitespace. |
175
|
org/apache/excalibur/instrument/AbstractLogEnabledInstrumentable.java
Error | Line |
| Line does not match expected header line of ''. |
1
|
| Line has trailing spaces. |
1
|
| Line has trailing spaces. |
5
|
| Line has trailing spaces. |
6
|
| Line has trailing spaces. |
8
|
| Line has trailing spaces. |
13
|
| '{' should be on the previous line. |
37
|
| Name 'm_instrumentableName' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
39
|
| Name 'm_instrumentList' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
42
|
| Name 'm_childList' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
45
|
| Line is longer than 80 characters. |
47
|
| Name 'm_registered' must match pattern '^[a-z][a-zA-Z0-9]*$'. |
48
|
| '{' should be on the previous line. |
57
|
| Method 'addInstrument' is not designed for extension - needs to be abstract, final or empty. |
73
|
| '(' is followed by whitespace. |
73
|
| Parameter instrument should be final. |
73
|
| ')' is preceded with whitespace. |
73
|
| '{' should be on the previous line. |
74
|
| 'if' is not followed by whitespace. |
75
|
| '(' is followed by whitespace. |
75
|
| ')' is preceded with whitespace. |
75
|
| '{' should be on the previous line. |
76
|
| Line is longer than 80 characters. |
77
|
| '(' is followed by whitespace. |
77
|
| Line is longer than 80 characters. |
78
|
| ')' is preceded with whitespace. |
78
|
| '(' is followed by whitespace. |
80
|
| ')' is preceded with whitespace. |
80
|
| Method 'addChildInstrumentable' is not designed for extension - needs to be abstract, final or empty. |
93
|
| '(' is followed by whitespace. |
93
|
| Parameter child should be final. |
93
|
| ')' is preceded with whitespace. |
93
|
| '{' should be on the previous line. |
94
|
| 'if' is not followed by whitespace. |
95
|
| '(' is followed by whitespace. |
95
|
| ')' is preceded with whitespace. |
95
|
| '{' should be on the previous line. |
96
|
| Line is longer than 80 characters. |
97
|
| '(' is followed by whitespace. |
97
|
| Line is longer than 80 characters. |
98
|
| ')' is preceded with whitespace. |
98
|
| '(' is followed by whitespace. |
100
|
| ')' is preceded with whitespace. |
100
|
| '{' should be on the previous line. |
112
|
| '(' is followed by whitespace. |
129
|
| Parameter name should be final. |
129
|
| ')' is preceded with whitespace. |
129
|
| '{' should be on the previous line. |
130
|
| '{' should be on the previous line. |
144
|
| 'if' is not followed by whitespace. |
146
|
| '(' is followed by whitespace. |
146
|
| ')' is preceded with whitespace. |
146
|
| '{' should be on the previous line. |
147
|
| '}' should be on the same line. |
149
|
| '{' should be on the previous line. |
151
|
| Line is longer than 80 characters. |
152
|
| '(' is followed by whitespace. |
153
|
| ')' is preceded with whitespace. |
153
|
| '{' should be on the previous line. |
170
|
| 'if' is not followed by whitespace. |
172
|
| '(' is followed by whitespace. |
172
|
| ')' is preceded with whitespace. |
172
|
| '{' should be on the previous line. |
173
|
| '}' should be on the same line. |
175
|
| '{' should be on the previous line. |
177
|
| Line is longer than 80 characters. |
178
|
| '(' is followed by whitespace. |
179
|
| ')' is preceded with whitespace. |
179
|
org/apache/excalibur/instrument/CounterInstrument.java
Error | Line |
| Line has trailing spaces. |
1
|
| Missing a header - not enough lines in file. |
1
|
| Line has trailing spaces. |
5
|
| Line has trailing spaces. |
6
|
| Line has trailing spaces. |
8
|
| Line has trailing spaces. |
13
|
| '{' should be on the previous line. |
32
|
| '(' is followed by whitespace. |
42
|
| Parameter name should be final. |
42
|
| ')' is preceded with whitespace. |
42
|
| '{' should be on the previous line. |
43
|
| '(' is followed by whitespace. |
44
|
| ')' is preceded with whitespace. |
44
|
| Method 'increment' is not designed for extension - needs to be abstract, final or empty. |
55
|
| '{' should be on the previous line. |
56
|
| 'if' is not followed by whitespace. |
58
|
| '(' is followed by whitespace. |
58
|
| ')' is preceded with whitespace. |
58
|
| '{' should be on the previous line. |
59
|
| '(' is followed by whitespace. |
60
|
| ')' is preceded with whitespace. |
60
|
| Unused @throws tag for 'IllegalArgumentException'. |
71
|
| Method 'increment' is not designed for extension - needs to be abstract, final or empty. |
73
|
| '(' is followed by whitespace. |
73
|
| Parameter count should be final. |
73
|
| ')' is preceded with whitespace. |
73
|
| '{' should be on the previous line. |
74
|
| 'if' is not followed by whitespace. |
76
|
| '(' is followed by whitespace. |
76
|
| ')' is preceded with whitespace. |
76
|
| '{' should be on the previous line. |
77
|
| Line is longer than 80 characters. |
78
|
| '(' is followed by whitespace. |
78
|
| ')' is preceded with whitespace. |
78
|
| 'if' is not followed by whitespace. |
82
|
| '(' is followed by whitespace. |
82
|
| ')' is preceded with whitespace. |
82
|
| '{' should be on the previous line. |
83
|
| '(' is followed by whitespace. |
84
|
| ')' is preceded with whitespace. |
84
|
org/apache/excalibur/instrument/Instrument.java
Error | Line |
| Line has trailing spaces. |
1
|
| Missing a header - not enough lines in file. |
1
|
| Line has trailing spaces. |
5
|
| Line has trailing spaces. |
6
|
| Line has trailing spaces. |
8
|
| Line has trailing spaces. |
13
|
| '{' should be on the previous line. |
29
|
org/apache/excalibur/instrument/InstrumentManageable.java
Error | Line |
| Line has trailing spaces. |
1
|
| Missing a header - not enough lines in file. |
1
|
| Line has trailing spaces. |
5
|
| Line has trailing spaces. |
6
|
| Line has trailing spaces. |
8
|
| Line has trailing spaces. |
13
|
| '{' should be on the previous line. |
32
|
| '(' is followed by whitespace. |
39
|
| ')' is preceded with whitespace. |
39
|
org/apache/excalibur/instrument/InstrumentManager.java
Error | Line |
| Line has trailing spaces. |
1
|
| Missing a header - not enough lines in file. |
1
|
| Line has trailing spaces. |
5
|
| Line has trailing spaces. |
6
|
| Line has trailing spaces. |
8
|
| Line has trailing spaces. |
13
|
| '{' should be on the previous line. |
27
|
| Missing a Javadoc comment. |
28
|
| Line is longer than 80 characters. |
36
|
| Line is longer than 80 characters. |
37
|
| Line is longer than 80 characters. |
39
|
| Line is longer than 80 characters. |
41
|
| '(' is followed by whitespace. |
41
|
| ')' is preceded with whitespace. |
41
|
org/apache/excalibur/instrument/InstrumentProxy.java
Error | Line |
| Line has trailing spaces. |
1
|
| Missing a header - not enough lines in file. |
1
|
| Line has trailing spaces. |
5
|
| Line has trailing spaces. |
6
|
| Line has trailing spaces. |
8
|
| Line has trailing spaces. |
13
|
| '{' should be on the previous line. |
35
|
| '(' is followed by whitespace. |
54
|
| ')' is preceded with whitespace. |
54
|
| '(' is followed by whitespace. |
66
|
| ')' is preceded with whitespace. |
66
|
org/apache/excalibur/instrument/Instrumentable.java
Error | Line |
| Line has trailing spaces. |
1
|
| Missing a header - not enough lines in file. |
1
|
| Line has trailing spaces. |
5
|
| Line has trailing spaces. |
6
|
| Line has trailing spaces. |
8
|
| Line has trailing spaces. |
13
|
| '{' should be on the previous line. |
34
|
| '(' is followed by whitespace. |
59
|
| ')' is preceded with whitespace. |
59
|
org/apache/excalibur/instrument/NoopInstrumentManager.java
Error | Line |
| Line has trailing spaces. |
1
|
| Missing a header - not enough lines in file. |
1
|
| Line has trailing spaces. |
5
|
| Line has trailing spaces. |
6
|
| Line has trailing spaces. |
8
|
| Line has trailing spaces. |
13
|
| Line is longer than 80 characters. |
26
|
| '{' should be on the previous line. |
30
|
| Line is longer than 80 characters. |
37
|
| Line is longer than 80 characters. |
38
|
| Line is longer than 80 characters. |
40
|
| Unused @throws tag for 'Exception'. |
40
|
| Line is longer than 80 characters. |
42
|
| '(' is followed by whitespace. |
42
|
| Parameter instrumentable should be final. |
42
|
| Parameter instrumentableName should be final. |
42
|
| ')' is preceded with whitespace. |
42
|
org/apache/excalibur/instrument/ValueInstrument.java
Error | Line |
| Line has trailing spaces. |
1
|
| Missing a header - not enough lines in file. |
1
|
| Line has trailing spaces. |
5
|
| Line has trailing spaces. |
6
|
| Line has trailing spaces. |
8
|
| Line has trailing spaces. |
13
|
| '{' should be on the previous line. |
31
|
| '(' is followed by whitespace. |
41
|
| Parameter name should be final. |
41
|
| ')' is preceded with whitespace. |
41
|
| '{' should be on the previous line. |
42
|
| '(' is followed by whitespace. |
43
|
| ')' is preceded with whitespace. |
43
|
| Method 'setValue' is not designed for extension - needs to be abstract, final or empty. |
65
|
| '(' is followed by whitespace. |
65
|
| Parameter value should be final. |
65
|
| ')' is preceded with whitespace. |
65
|
| '{' should be on the previous line. |
66
|
| 'if' is not followed by whitespace. |
68
|
| '(' is followed by whitespace. |
68
|
| ')' is preceded with whitespace. |
68
|
| '{' should be on the previous line. |
69
|
| '(' is followed by whitespace. |
70
|
| ')' is preceded with whitespace. |
70
|
org/apache/excalibur/instrument/package.html
Error | Line |
| Missing package documentation file. | 0 |