author | Jan Vrany <jan.vrany@fit.cvut.cz> |
Mon, 23 Jan 2012 10:07:54 +0100 | |
changeset 84 | 77e12dbe97d3 |
parent 82 | e676efa2a286 |
child 117 | d7f87303b984 |
permissions | -rw-r--r-- |
71 | 1 |
"{ Package: 'stx:goodies/builder/reports' }" |
2 |
||
3 |
"{ NameSpace: Builder }" |
|
4 |
||
5 |
Report subclass:#LintReport |
|
6 |
instanceVariableNames:'environment rules' |
|
7 |
classVariableNames:'' |
|
8 |
poolDictionaries:'' |
|
9 |
category:'Builder-Reports' |
|
10 |
! |
|
11 |
||
12 |
||
13 |
!LintReport class methodsFor:'class initialization'! |
|
14 |
||
15 |
initialize |
|
16 |
||
17 |
Smalltalk loadPackage: 'stx:goodies/refactoryBrowser/lint'. |
|
18 |
||
19 |
"Modified: / 01-04-2011 / 12:58:47 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
20 |
! ! |
|
21 |
||
22 |
!LintReport methodsFor:'accessing - defaults'! |
|
23 |
||
24 |
defaultFileSuffix |
|
25 |
||
26 |
^ 'Lint' |
|
27 |
||
28 |
"Modified: / 08-10-2011 / 10:49:30 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
29 |
! |
|
30 |
||
31 |
defaultFormat |
|
32 |
"raise an error: must be redefined in concrete subclass(es)" |
|
33 |
||
34 |
^ LintReportFormat::PMD new |
|
35 |
||
36 |
"Modified: / 25-11-2011 / 22:06:33 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
79
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
37 |
! |
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
38 |
|
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
39 |
defaultName |
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
40 |
|
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
41 |
environment isNil ifTrue:[^super defaultName]. |
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
42 |
^environment label |
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
43 |
|
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
44 |
"Modified: / 25-11-2011 / 22:06:33 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
45 |
"Created: / 13-01-2012 / 12:43:07 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
71 | 46 |
! ! |
47 |
||
48 |
!LintReport methodsFor:'generating'! |
|
49 |
||
50 |
generateClass: aClass |
|
51 |
| sourceStream sourceName | |
|
52 |
sourceStream := WriteStream on: String new. |
|
53 |
sourceName := self encodeFilename: (self sourceFilenameFor: aClass). |
|
54 |
||
55 |
format writeFile: sourceName with: [ |
|
56 |
self generateClass: aClass source: sourceStream. |
|
57 |
self generateClass: aClass class source: sourceStream. |
|
58 |
]. |
|
59 |
||
60 |
format writeSource: sourceStream contents to: sourceName |
|
61 |
||
62 |
"Created: / 06-10-2011 / 23:54:37 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
63 |
! |
|
64 |
||
65 |
generateClass: aClass selector: aSelector source: sourceStream |
|
66 |
| offset source matching | |
|
67 |
offset := self |
|
68 |
lineAndColumn: sourceStream contents |
|
69 |
at: sourceStream position. |
|
70 |
sourceStream |
|
71 |
nextPutAll: (source := self convert: (aClass sourceCodeAt: aSelector)); |
|
72 |
nextPut: Character lf; nextPut: Character lf. |
|
73 |
matching := rules select: [ :each | |
|
74 |
(self isSelectorEnvironment: each result) |
|
75 |
and: [ each result includesSelector: aSelector in: aClass ] ]. |
|
76 |
self generateViolations: matching class: aClass selector: aSelector source: source offset: offset |
|
77 |
||
78 |
"Created: / 07-10-2011 / 11:04:12 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
79 |
! |
|
80 |
||
81 |
generateClass: aClass source: sourceStream |
|
82 |
||
83 |
| offset source matching | |
|
84 |
offset := self |
|
85 |
lineAndColumn: sourceStream contents |
|
86 |
at: sourceStream position. |
|
87 |
sourceStream |
|
88 |
nextPutAll: (source := self convert: aClass definition); |
|
89 |
nextPut: Character lf; nextPut: Character lf. |
|
90 |
(environment definesClass: aClass) ifTrue: [ |
|
91 |
matching := rules select: [ :rule | |
|
92 |
(self isClassEnvironment: rule result) |
|
93 |
and: [ rule result includesClass: aClass ] ]. |
|
94 |
self generateViolations: matching class: aClass selector: nil source: source offset: offset ]. |
|
95 |
(environment selectorsForClass: aClass) asSortedCollection |
|
96 |
do: [ :selector | self generateClass: aClass selector: selector source: sourceStream] |
|
97 |
||
98 |
"Created: / 07-10-2011 / 10:29:42 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
99 |
! |
|
100 |
||
101 |
generateViolations: aCollection class: aClass selector: aSelector source: aString offset: aPoint |
|
102 |
||
103 |
aCollection do: [ :rule | |
|
104 |
| interval start stop | |
|
105 |
||
106 |
interval := (rule result selectionIntervalFor: aString) ifNil: [ 1 to: aString size ]. |
|
107 |
start := self lineAndColumn: aString at: interval first. |
|
108 |
stop := self lineAndColumn: aString at: interval last. |
|
109 |
||
110 |
format writeViolation: rule |
|
111 |
class: aClass selector: aSelector |
|
112 |
startLine: aPoint x + start x column: aPoint y + start y - 1 |
|
113 |
stopLine: aPoint x + stop x column: aPoint y + stop y - 1. |
|
114 |
] |
|
115 |
||
116 |
"Created: / 25-11-2011 / 22:23:09 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
117 |
! ! |
|
118 |
||
119 |
!LintReport methodsFor:'initialization'! |
|
120 |
||
121 |
setupForClasses: classes |
|
122 |
||
123 |
environment := BrowserEnvironment new forClasses: classes. |
|
124 |
environment label: name |
|
125 |
||
126 |
"Created: / 04-08-2011 / 14:40:31 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
127 |
! |
|
128 |
||
129 |
setupForPackages: packages |
|
130 |
||
79
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
131 |
packages isEmpty ifTrue:[^self]. |
71 | 132 |
environment := PackageEnvironment |
133 |
onEnvironment: BrowserEnvironment new |
|
134 |
packageNames: packages. |
|
79
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
135 |
name isNil ifTrue:[ |
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
136 |
packages size > 1 ifTrue:[ |
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
137 |
name := packages size printString , ' packages'. |
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
138 |
] ifFalse:[ |
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
139 |
name := packages anElement |
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
140 |
] |
889d3747a0a8
Fix in default report file name for LintReport
Jan Vrany <jan.vrany@fit.cvut.cz>
parents:
71
diff
changeset
|
141 |
]. |
71 | 142 |
|
143 |
environment label: name. |
|
144 |
||
145 |
" |
|
146 |
LintReport runPackage:'stx:goodies/monticello'. |
|
147 |
LintReport runPackage:'stx:libjava' |
|
148 |
" |
|
149 |
||
150 |
"Created: / 04-08-2011 / 14:40:06 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
151 |
! ! |
|
152 |
||
153 |
!LintReport methodsFor:'private'! |
|
154 |
||
155 |
lineAndColumn: aString at: anInteger |
|
156 |
| line last stream | |
|
157 |
line := 1. |
|
158 |
last := 0. |
|
159 |
stream := aString readStream. |
|
160 |
[ (stream nextLine isNil or: [ anInteger <= stream position ]) |
|
161 |
ifTrue: [ ^ line @ (anInteger - last) ]. |
|
162 |
last := stream position. |
|
163 |
line := line + 1 ] |
|
164 |
repeat |
|
165 |
! |
|
166 |
||
167 |
sourceFilenameFor: aClass |
|
168 |
||
169 |
| fn | |
|
170 |
||
171 |
fn := aClass package asString replaceAll:$: with:$_;replaceAll:$/ with:$_. |
|
172 |
fn := fn , '_' , (aClass asString copyReplaceAll:$: with: $_), '.' , aClass programmingLanguage sourceFileSuffix. |
|
173 |
||
174 |
^self encodeFilename: fn. |
|
175 |
||
176 |
" |
|
177 |
Builder::LintReportFormat::CheckStyle basicNew |
|
178 |
sourceFilenameFor: Class |
|
179 |
||
180 |
Builder::LintReportFormat::CheckStyle basicNew |
|
181 |
sourceFilenameFor: Builder::LintReportFormat |
|
182 |
" |
|
183 |
||
184 |
"Created: / 07-10-2011 / 09:06:00 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
185 |
! ! |
|
186 |
||
187 |
!LintReport methodsFor:'running'! |
|
188 |
||
189 |
runReport |
|
82 | 190 |
| wasTryLocalSources | |
71 | 191 |
|
82 | 192 |
[ |
193 |
wasTryLocalSources := Class tryLocalSourceFirst. |
|
84 | 194 |
Class tryLocalSourceFirst: true. |
82 | 195 |
SmalllintChecker |
196 |
runRule: (RBCompositeLintRule rules: rules) |
|
197 |
onEnvironment: environment. |
|
71 | 198 |
|
82 | 199 |
(environment classes asSortedCollection: [ :a :b | a name <= b name ]) |
200 |
do: [ :class | self generateClass: class]. |
|
201 |
] ensure:[ |
|
202 |
Class tryLocalSourceFirst: wasTryLocalSources |
|
203 |
] |
|
71 | 204 |
|
84 | 205 |
"Modified: / 23-01-2012 / 09:04:55 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
71 | 206 |
! |
207 |
||
208 |
setUp |
|
209 |
||
210 |
super setUp. |
|
211 |
rules := (RBCompositeLintRule rulesFor: RBBasicLintRule) |
|
212 |
reject: [ :each | each class name endsWith: 'SpellingRule' ] |
|
213 |
||
214 |
"Created: / 04-08-2011 / 14:35:27 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
215 |
! ! |
|
216 |
||
217 |
!LintReport methodsFor:'testing'! |
|
218 |
||
219 |
isClassEnvironment: anEnvironment |
|
220 |
^ #(CategoryEnvironment ClassEnvironment VariableEnvironment) includes: anEnvironment class name |
|
221 |
! |
|
222 |
||
223 |
isSelectorEnvironment: anEnvironment |
|
224 |
^ #(SelectorEnvironment ParseTreeEnvironment VariableEnvironment) includes: anEnvironment class name |
|
225 |
! ! |
|
226 |
||
227 |
!LintReport class methodsFor:'documentation'! |
|
228 |
||
229 |
version |
|
230 |
^ '$Header$' |
|
231 |
! |
|
232 |
||
233 |
version_CVS |
|
234 |
^ '$Header$' |
|
235 |
! |
|
236 |
||
237 |
version_SVN |
|
238 |
^ '§Id: Builder__LintReport.st 293 2011-11-25 21:42:31Z vranyj1 §' |
|
239 |
! ! |
|
240 |
||
241 |
LintReport initialize! |