author | Claus Gittinger <cg@exept.de> |
Mon, 22 Jul 2013 13:39:55 +0200 | |
changeset 13201 | 3b6f308a7280 |
parent 13191 | 9745eed50f03 |
child 13206 | 9dc79fafe94c |
permissions | -rw-r--r-- |
9982 | 1 |
" |
10071 | 2 |
COPYRIGHT (c) 2010 by Jan Vrany, SWING Research Group. CTU in Prague |
11607 | 3 |
All Rights Reserved |
9982 | 4 |
|
10071 | 5 |
Permission is hereby granted, free of charge, to any person |
6 |
obtaining a copy of this software and associated documentation |
|
7 |
files (the 'Software'), to deal in the Software without |
|
8 |
restriction, including without limitation the rights to use, |
|
9 |
copy, modify, merge, publish, distribute, sublicense, and/or sell |
|
10 |
copies of the Software, and to permit persons to whom the |
|
11 |
Software is furnished to do so, subject to the following |
|
12 |
conditions: |
|
13 |
||
14 |
The above copyright notice and this permission notice shall be |
|
15 |
included in all copies or substantial portions of the Software. |
|
16 |
||
17 |
THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND, |
|
18 |
EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES |
|
19 |
OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND |
|
20 |
NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT |
|
21 |
HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, |
|
22 |
WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING |
|
23 |
FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR |
|
24 |
OTHER DEALINGS IN THE SOFTWARE. |
|
9982 | 25 |
" |
26 |
"{ Package: 'stx:libtool' }" |
|
27 |
||
28 |
"{ NameSpace: Tools }" |
|
29 |
||
30 |
CodeViewService subclass:#BreakpointService |
|
13106 | 31 |
instanceVariableNames:'breakpoints currentMethod currentMethodClass' |
11607 | 32 |
classVariableNames:'' |
33 |
poolDictionaries:'' |
|
34 |
category:'Interface-CodeView' |
|
9982 | 35 |
! |
36 |
||
37 |
!BreakpointService class methodsFor:'documentation'! |
|
38 |
||
39 |
copyright |
|
40 |
" |
|
10071 | 41 |
COPYRIGHT (c) 2010 by Jan Vrany, SWING Research Group. CTU in Prague |
11607 | 42 |
All Rights Reserved |
9982 | 43 |
|
10071 | 44 |
Permission is hereby granted, free of charge, to any person |
45 |
obtaining a copy of this software and associated documentation |
|
46 |
files (the 'Software'), to deal in the Software without |
|
47 |
restriction, including without limitation the rights to use, |
|
48 |
copy, modify, merge, publish, distribute, sublicense, and/or sell |
|
49 |
copies of the Software, and to permit persons to whom the |
|
50 |
Software is furnished to do so, subject to the following |
|
51 |
conditions: |
|
52 |
||
53 |
The above copyright notice and this permission notice shall be |
|
54 |
included in all copies or substantial portions of the Software. |
|
55 |
||
56 |
THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND, |
|
57 |
EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES |
|
58 |
OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND |
|
59 |
NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT |
|
60 |
HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, |
|
61 |
WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING |
|
62 |
FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR |
|
63 |
OTHER DEALINGS IN THE SOFTWARE. |
|
9982 | 64 |
" |
65 |
! ! |
|
66 |
||
13101 | 67 |
!BreakpointService class methodsFor:'accessing'! |
68 |
||
69 |
label |
|
70 |
"Answers a short label - for UI" |
|
71 |
||
72 |
^ 'Breakpoints' |
|
73 |
! ! |
|
74 |
||
10226 | 75 |
!BreakpointService methodsFor:'accessing'! |
76 |
||
77 |
breakpoints |
|
78 |
^ breakpoints |
|
79 |
! ! |
|
80 |
||
10208 | 81 |
!BreakpointService methodsFor:'change & update'! |
82 |
||
83 |
update: aspect with: param from: sender |
|
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
84 |
aspect == #visibility ifTrue:[^ self]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
85 |
aspect == #sizeOfView ifTrue:[^ self]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
86 |
aspect == #classHolder ifTrue:[^ self]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
87 |
aspect == #languageHolder ifTrue:[^ self]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
88 |
sender == codeView modifiedChannel ifTrue:[^ self]. |
10208 | 89 |
|
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
90 |
(aspect == #methodHolder or:[sender == codeView methodHolder]) ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
91 |
self updateCurrentMethod. |
10208 | 92 |
]. |
93 |
super update: aspect with: param from: sender |
|
94 |
||
95 |
"Created: / 06-07-2011 / 15:21:08 / cg" |
|
96 |
! |
|
97 |
||
98 |
updateBreakPointsFor:aMethod |
|
99 |
|methodsBreakPoints| |
|
100 |
||
101 |
aMethod notNil ifTrue:[ |
|
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
102 |
aMethod literalsDo:[:eachLiteral | |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
103 |
eachLiteral class == Breakpoint ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
104 |
methodsBreakPoints isNil ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
105 |
methodsBreakPoints := OrderedCollection new. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
106 |
]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
107 |
methodsBreakPoints add:eachLiteral copy. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
108 |
]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
109 |
]. |
13106 | 110 |
currentMethodClass := aMethod mclass. |
111 |
] ifFalse:[ |
|
112 |
currentMethodClass := nil |
|
10208 | 113 |
]. |
114 |
breakpoints := methodsBreakPoints. |
|
115 |
currentMethod := aMethod. |
|
116 |
||
117 |
"Created: / 06-07-2011 / 15:24:09 / cg" |
|
10226 | 118 |
"Modified: / 06-07-2011 / 17:32:54 / jv" |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
119 |
! |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
120 |
|
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
121 |
updateCurrentMethod |
13101 | 122 |
|method realMethod oldBreakPoints| |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
123 |
|
13106 | 124 |
"/ codeView methodHolder class == BlockValue ifTrue:[self breakPoint:#cg]. |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
125 |
|
13201
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
126 |
method := realMethod := codeView method. |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
127 |
(method notNil and:[method mclass isNil]) ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
128 |
"/ hack: ouch - was wrapped in the meantime; |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
129 |
"/ hurry up and update. Should be done elsewhere (in codeView) |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
130 |
realMethod := MethodWithBreakpoints allInstances detect:[:m | m originalMethod == method and:[m mclass notNil]] ifNone:nil. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
131 |
realMethod isNil ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
132 |
realMethod := WrappedMethod allInstances detect:[:m | m originalMethod == method and:[m mclass notNil]] ifNone:nil. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
133 |
]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
134 |
]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
135 |
realMethod ~~ currentMethod ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
136 |
"/ codeView methodHolder setValue:realMethod. |
13101 | 137 |
oldBreakPoints := breakpoints. |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
138 |
self updateBreakPointsFor:realMethod. |
13101 | 139 |
oldBreakPoints ~= breakpoints ifTrue:[ |
140 |
gutterView invalidate. |
|
141 |
] |
|
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
142 |
]. |
13201
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
143 |
|
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
144 |
"Modified: / 22-07-2013 / 13:33:28 / cg" |
10208 | 145 |
! ! |
146 |
||
9982 | 147 |
!BreakpointService methodsFor:'event handling'! |
148 |
||
11607 | 149 |
buttonPress:button x:x y:y in:view |
10714
3399fb50f42e
changed: #buttonPress:x:y:in:
Claus Gittinger <cg@exept.de>
parents:
10411
diff
changeset
|
150 |
|lineNr| |
3399fb50f42e
changed: #buttonPress:x:y:in:
Claus Gittinger <cg@exept.de>
parents:
10411
diff
changeset
|
151 |
|
9982 | 152 |
view == gutterView ifTrue:[ |
13152
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
153 |
button == 1 ifTrue:[ |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
154 |
lineNr := textView yVisibleToLineNr:y. |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
155 |
lineNr notNil ifTrue:[ self setOrToggleBreakpointAtLine:lineNr ]. |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
156 |
^ true. |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
157 |
]. |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
158 |
button == 3 ifTrue:[ |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
159 |
^ true. |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
160 |
] |
9982 | 161 |
]. |
10182 | 162 |
^ false |
9982 | 163 |
|
164 |
"Created: / 17-06-2011 / 13:05:22 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
165 |
"Modified: / 28-06-2011 / 08:31:39 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
10714
3399fb50f42e
changed: #buttonPress:x:y:in:
Claus Gittinger <cg@exept.de>
parents:
10411
diff
changeset
|
166 |
"Modified: / 19-09-2011 / 14:41:00 / cg" |
10226 | 167 |
! |
168 |
||
169 |
linesDeletedFrom: start to: end |
|
170 |
||
171 |
breakpoints isEmptyOrNil ifTrue:[^self]. |
|
172 |
self moveBreakpointsAfterLine: start - 1 by: (end - start + 1) negated |
|
173 |
||
174 |
"Created: / 06-07-2011 / 17:16:27 / jv" |
|
175 |
! |
|
176 |
||
177 |
linesInsertedFrom: start to: end |
|
178 |
||
179 |
breakpoints isEmptyOrNil ifTrue:[^self]. |
|
180 |
self moveBreakpointsAfterLine: start - 1 by: (end - start + 1) |
|
181 |
||
182 |
"Created: / 06-07-2011 / 17:16:36 / jv" |
|
9982 | 183 |
! ! |
184 |
||
13126
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
185 |
!BreakpointService methodsFor:'help'! |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
186 |
|
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
187 |
flyByHelpText |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
188 |
|topView| |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
189 |
|
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
190 |
(self canCreateOrToggleBreakpointAtLine:nil) ifFalse:[ |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
191 |
((topView := codeView topView) class == DebugView) ifTrue:[ |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
192 |
self hasBreakpoints ifFalse:[ |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
193 |
^ 'Sorry - cannot add breakpoint in the debugger (would need recompilation)\(can only add breakpoints if stopped at a method breakpoint)' withCRs |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
194 |
]. |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
195 |
^ 'Click to toggle existing breakpoint. Shift-Click to toggle tracepoint.\Sorry - cannot add new breakpoint if method is already entered\(i.e. if not stopped at a method breakpoint).' withCRs |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
196 |
]. |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
197 |
^ 'Cannot add breakpoint when modified. Please accept first.' |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
198 |
]. |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
199 |
|
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
200 |
^ 'Click to toggle breakpoint. Shift-Click to toggle tracepoint.' |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
201 |
|
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
202 |
"Created: / 27-01-2012 / 14:04:52 / cg" |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
203 |
! ! |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
204 |
|
9982 | 205 |
!BreakpointService methodsFor:'initialization'! |
206 |
||
207 |
initialize |
|
208 |
||
209 |
super initialize. |
|
210 |
breakpoints := OrderedCollection new. |
|
211 |
||
212 |
"Created: / 17-06-2011 / 13:49:12 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
213 |
! ! |
|
214 |
||
215 |
!BreakpointService methodsFor:'private'! |
|
216 |
||
11607 | 217 |
breakpointAtLine:line |
10182 | 218 |
|pos| |
9982 | 219 |
|
12969
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
220 |
breakpoints isNil ifTrue:[^ nil]. |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
221 |
|
9982 | 222 |
pos := textView characterPositionOfLine:line col:1. |
13191
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
223 |
^ breakpoints |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
224 |
detect:[:each | each position = pos ] |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
225 |
ifNone:[ |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
226 |
breakpoints |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
227 |
detect:[:each | each line == line and:[each position isNil ]] |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
228 |
ifNone:[ nil ] |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
229 |
] |
9982 | 230 |
|
231 |
"Modified: / 17-06-2011 / 13:59:17 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
10182 | 232 |
"Modified (format): / 05-07-2011 / 21:33:23 / cg" |
233 |
! |
|
234 |
||
10226 | 235 |
moveBreakpointsAfterLine:line by: delta |
236 |
|pos | |
|
237 |
||
238 |
breakpoints do:[:bpnt| |
|
11719 | 239 |
bpnt line >= line ifTrue:[ |
240 |
pos := textView characterPositionOfLine:bpnt line + delta col:1. |
|
241 |
bpnt position:pos line:(bpnt line + delta). |
|
242 |
] |
|
10226 | 243 |
]. |
244 |
||
245 |
"/gutterView redrawLinesFrom: line. |
|
246 |
||
247 |
"Created: / 17-06-2011 / 13:45:22 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
248 |
"Created: / 06-07-2011 / 17:26:30 / jv" |
|
11719 | 249 |
"Modified: / 02-08-2012 / 09:27:10 / cg" |
10226 | 250 |
! |
251 |
||
10182 | 252 |
recompile |
253 |
"recompile the current method for changed breakpoints" |
|
254 |
||
11719 | 255 |
|oldMethod newMethod compiler class selector| |
10182 | 256 |
|
13201
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
257 |
oldMethod := codeView method. |
11719 | 258 |
(oldMethod notNil and:[oldMethod hasPrimitiveCode not]) ifTrue:[ |
259 |
"/ be careful: if the text has been edited/modified, do not compile |
|
260 |
textView modified ifTrue:[ |
|
261 |
self breakPoint: #cg. |
|
262 |
self breakPoint: #jv. |
|
263 |
^self. |
|
264 |
] ifFalse:[ |
|
265 |
"/ prepare to get reachable bpts |
|
266 |
breakpoints do:[:bp | bp isReached:false]. |
|
267 |
||
268 |
class := oldMethod mclass. |
|
269 |
class isNil ifTrue:[ |
|
270 |
class := codeView classHolder value. |
|
12507
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
271 |
class isNil ifTrue:[ |
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
272 |
self breakPoint:#jv. |
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
273 |
Dialog warn:'oops - lost the methods''s class'. |
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
274 |
^ self. |
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
275 |
] |
11719 | 276 |
]. |
277 |
selector := oldMethod selector. |
|
10182 | 278 |
|
11719 | 279 |
Class withoutUpdatingChangesDo:[ |
280 |
compiler := ByteCodeCompilerWithBreakpointSupport new. |
|
281 |
compiler breakpoints:breakpoints. |
|
282 |
compiler methodClass:MethodWithBreakpoints. |
|
283 |
newMethod := compiler |
|
284 |
compile:oldMethod source |
|
285 |
forClass:class |
|
286 |
inCategory:oldMethod category |
|
287 |
notifying:nil |
|
288 |
install:false |
|
289 |
skipIfSame:false |
|
290 |
silent:true |
|
291 |
foldConstants:true |
|
11987 | 292 |
ifFail:[ Transcript showCR:'BreakpointService: failed to recompile for breakpoint' ]. |
10182 | 293 |
|
11719 | 294 |
selector isNil ifTrue:[ |
295 |
"/ May happen as the selector is not stored in the method but |
|
296 |
"/ searches through method's mclass methodDictionary. |
|
297 |
"/ Following should be save as breakpoint is not installed when |
|
298 |
"/ the code is modified... |
|
299 |
selector := compiler selector. |
|
300 |
]. |
|
301 |
||
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
302 |
oldMethod isWrapped ifTrue:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
303 |
"/ update the wrapped method - do not install |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
304 |
newMethod originalMethod: oldMethod originalMethod. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
305 |
oldMethod replaceOriginalMethodWith:newMethod. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
306 |
] ifFalse:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
307 |
"/ install |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
308 |
newMethod originalMethod: oldMethod. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
309 |
(class primAddSelector: selector withMethod:newMethod) ifFalse:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
310 |
oldMethod mclass:class. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
311 |
self breakPoint: #cg. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
312 |
self breakPoint: #jv. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
313 |
^ self |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
314 |
]. |
11719 | 315 |
]. |
12969
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
316 |
|
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
317 |
breakpoints := breakpoints |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
318 |
select:[:bp | |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
319 |
"/ bp isReached ifFalse:[ |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
320 |
"/ "/ Transcript show:'remove unreached:'; showCR:bp |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
321 |
"/ ]. |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
322 |
bp isReached |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
323 |
]. |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
324 |
|
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
325 |
"/ must update breakpoints BEFORE the following, because it leads to a change |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
326 |
"/ notification, which may clear the breakpoints collection!! |
11719 | 327 |
codeView methodHolder value:newMethod. |
328 |
oldMethod mclass isNil ifTrue:[ |
|
12507
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
329 |
"/ although this is not strictly true, not doing this |
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
330 |
"/ would confuse a lot of other tools (such as the browser) |
11719 | 331 |
oldMethod mclass:class. |
332 |
]. |
|
12507
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
333 |
class changed:#methodTrap with:selector. "/ tell browsers |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
334 |
Smalltalk changed:#methodTrap with:(MethodTrapChangeNotificationParameter changeClass:class changeSelector:selector). |
11719 | 335 |
]. |
336 |
] |
|
10182 | 337 |
] |
338 |
||
339 |
"Created: / 05-07-2011 / 21:33:13 / cg" |
|
11601
6500e91de9e8
changed: #recompile (fixes to allow multiple breakpoints in a method)
vrany
parents:
11569
diff
changeset
|
340 |
"Modified: / 18-07-2012 / 10:53:22 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
13201
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
341 |
"Modified: / 22-07-2013 / 13:33:09 / cg" |
9982 | 342 |
! |
343 |
||
11607 | 344 |
setOrToggleBreakpointAtLine:line |
12940
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
345 |
|pos bpnt prepareFullBreakSupport| |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
346 |
|
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
347 |
"/ if true, setting a single breakpoint in a method will create |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
348 |
"/ a whole set of invisible (and disabled) breakpoints in that method, |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
349 |
"/ one for each line. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
350 |
"/ These can later be enabled in the debugger |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
351 |
"/ (otherwise, the debugger's behavior is stupid, as it cannot recompile a method |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
352 |
"/ to set additional breakpoints). |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
353 |
"/ We accept the additional overhead, as we are in debug mode anyway. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
354 |
"/ prepareFullBreakSupport := false. |
12940
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
355 |
prepareFullBreakSupport := true. |
9982 | 356 |
|
13201
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
357 |
codeView method isNil ifTrue:[ |
13152
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
358 |
^ self |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
359 |
]. |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
360 |
|
12855
ab87c94ed5ac
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12507
diff
changeset
|
361 |
textView reallyModified ifTrue:[ |
ab87c94ed5ac
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12507
diff
changeset
|
362 |
"/ leads to ugly behavior (method no longer found), if we allow |
ab87c94ed5ac
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12507
diff
changeset
|
363 |
"/ this... |
13152
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
364 |
Dialog warn:'Please accept first (cannot set breakpoint while text is modified)'. |
12855
ab87c94ed5ac
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12507
diff
changeset
|
365 |
^ self |
ab87c94ed5ac
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12507
diff
changeset
|
366 |
]. |
ab87c94ed5ac
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12507
diff
changeset
|
367 |
|
9982 | 368 |
bpnt := self breakpointAtLine:line. |
10182 | 369 |
bpnt isNil ifTrue:[ |
12940
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
370 |
"/ no breakpoint there - add as required |
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
371 |
(self canCreateOrToggleBreakpointAtLine:line) ifTrue:[ |
12940
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
372 |
prepareFullBreakSupport ifTrue:[ |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
373 |
"/ add a (disabled) breakpoint for every source line. This |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
374 |
"/ allows for breakpoints to be enabled/disabled in the debugger... |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
375 |
1 to:textView numberOfLines do:[:eachLine | |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
376 |
|oldBPnt eachPos otherBpnt| |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
377 |
|
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
378 |
oldBPnt := self breakpointAtLine:eachLine. |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
379 |
oldBPnt isNil ifTrue:[ |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
380 |
eachPos := textView characterPositionOfLine:eachLine col:1. |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
381 |
breakpoints isNil ifTrue:[ breakpoints := OrderedCollection new]. |
12940
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
382 |
breakpoints add:((otherBpnt := Breakpoint new) position:eachPos line:eachLine). |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
383 |
eachLine == line ifTrue:[ |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
384 |
bpnt := otherBpnt. |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
385 |
] ifFalse:[ |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
386 |
otherBpnt beInvisible. |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
387 |
] |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
388 |
]. |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
389 |
]. |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
390 |
] ifFalse:[ |
13080
5ad43ae672b1
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12976
diff
changeset
|
391 |
pos := textView characterPositionOfLine:line col:1. |
12940
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
392 |
breakpoints add:((bpnt := Breakpoint new) position:pos line:line). |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
393 |
]. |
11719 | 394 |
Display shiftDown ifTrue:[ |
395 |
"/ trace |
|
396 |
bpnt beTracepoint |
|
397 |
]. |
|
12969
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
398 |
self assert: breakpoints notEmptyOrNil. |
11719 | 399 |
self recompile. |
12906
58e97bbbf5a4
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12902
diff
changeset
|
400 |
] ifFalse:[ |
58e97bbbf5a4
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12902
diff
changeset
|
401 |
codeView topView class == DebugView ifTrue:[ |
12976
d8aec7edf3d5
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12969
diff
changeset
|
402 |
Dialog warn:'Sorry, in an active method, I can only add new breakpoints in an already breakpointed method. |
d8aec7edf3d5
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12969
diff
changeset
|
403 |
(i.e. a method stopped at a method breakpoint or one which already has statement breakpoints) |
13157
5226414cb2c1
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13152
diff
changeset
|
404 |
The reason is that the method needs to be recompiled for the breakpoint, which would not affect the method being currently executed.'. |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
405 |
"/ Dialog warn:'Sorry, can only add a new breakpoint in a wrapped method which has not yet started.'. |
12906
58e97bbbf5a4
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12902
diff
changeset
|
406 |
] ifFalse:[ |
58e97bbbf5a4
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12902
diff
changeset
|
407 |
Dialog warn:'Sorry, cannot add a new breakpoint here.'. |
58e97bbbf5a4
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12902
diff
changeset
|
408 |
]. |
58e97bbbf5a4
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12902
diff
changeset
|
409 |
] |
10182 | 410 |
] ifFalse:[ |
12940
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
411 |
"/ breakpoint already there - just enable/disable |
11719 | 412 |
Display shiftDown ifTrue:[ |
413 |
bpnt toggleTracing |
|
414 |
] ifFalse:[ |
|
415 |
bpnt toggle. |
|
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
416 |
]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
417 |
currentMethod mclass isNil ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
418 |
"/ hack: ouch - was wrapped in the meantime; |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
419 |
"/ hurry up and update. Should be done elsewhere (in codeView) |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
420 |
self updateCurrentMethod. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
421 |
]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
422 |
Smalltalk changed:#methodTrap with:(MethodTrapChangeNotificationParameter changeClass:currentMethod mclass changeSelector:currentMethod selector). |
10182 | 423 |
]. |
11987 | 424 |
|
9982 | 425 |
gutterView redrawLine:line. |
426 |
||
427 |
"Created: / 17-06-2011 / 13:45:22 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
10411 | 428 |
"Modified: / 27-07-2011 / 13:27:55 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
13201
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
429 |
"Modified: / 22-07-2013 / 13:33:18 / cg" |
9982 | 430 |
! ! |
431 |
||
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
432 |
!BreakpointService methodsFor:'queries'! |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
433 |
|
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
434 |
canCreateOrToggleBreakpointAtLine:lineOrNilForAnywhere |
13126
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
435 |
|bpnt topView| |
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
436 |
|
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
437 |
textView reallyModified ifTrue:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
438 |
^ false |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
439 |
]. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
440 |
"/ can always toggle existing breakpoints... |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
441 |
lineOrNilForAnywhere notNil ifTrue:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
442 |
bpnt := self breakpointAtLine:lineOrNilForAnywhere. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
443 |
bpnt notNil ifTrue:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
444 |
^ true. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
445 |
] |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
446 |
]. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
447 |
|
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
448 |
((topView := codeView topView) class == DebugView) ifTrue:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
449 |
"/ can only create new breakpoints in the debugger, |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
450 |
"/ iff we are in a wrapped method's prolog |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
451 |
topView selectedContextIsWrapped ifTrue:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
452 |
topView selectedContext lineNumber == 1 ifTrue:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
453 |
^ true |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
454 |
]. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
455 |
]. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
456 |
^ false. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
457 |
]. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
458 |
^ true. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
459 |
! |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
460 |
|
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
461 |
hasBreakpoints |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
462 |
^ breakpoints notEmptyOrNil |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
463 |
! ! |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
464 |
|
9982 | 465 |
!BreakpointService methodsFor:'redrawing'! |
466 |
||
11607 | 467 |
drawLine:lineNo in:view atX:x y:y width:w height:h from:startCol to:endColOrNil with:fg and:bg |
9982 | 468 |
"Called by both gutterView and textView (well, not yet) to |
469 |
allow services to draw custom things on text view. |
|
470 |
Ask JV what the args means if unsure (I'm lazy to document |
|
471 |
them, now it is just an experiment...)" |
|
11607 | 472 |
|
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
473 |
|mthd bpnt icon dx dy| |
9982 | 474 |
|
13191
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
475 |
"/ these tests make the breakpointService unusable for other applications (which are mote |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
476 |
"/ based on smalltalk methods. They are not really needed: if there is a breakpoint, |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
477 |
"/ I can show it. Period. |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
478 |
|
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
479 |
"/ (mthd := codeView methodHolder value) isNil ifTrue:[ |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
480 |
"/ ^ self |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
481 |
"/ ]. |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
482 |
"/ currentMethodClass isNil ifTrue:[ |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
483 |
"/ "/ hack: ouch - was wrapped in the meantime; |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
484 |
"/ ^ self. "/ wait for the real update |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
485 |
"/ "/ hurry up and update. Should be done elsewhere (in codeView) |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
486 |
"/ "/ self updateCurrentMethod. |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
487 |
"/ ]. |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
488 |
|
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
489 |
view == gutterView ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
490 |
bpnt := self breakpointAtLine:lineNo. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
491 |
bpnt isNil ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
492 |
^ self |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
493 |
]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
494 |
icon := bpnt icon. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
495 |
icon isNil ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
496 |
^ self |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
497 |
]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
498 |
dx := ((w - icon width) / 2) rounded. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
499 |
dy := ((h - icon height) / 2) rounded. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
500 |
icon |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
501 |
displayOn:view |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
502 |
x:x + dx |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
503 |
y:y - h + dy + 4. "TODO: Magic constant" |
9982 | 504 |
]. |
505 |
||
506 |
"Created: / 17-06-2011 / 13:52:52 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
10182 | 507 |
"Modified (format): / 05-07-2011 / 22:14:33 / cg" |
9982 | 508 |
! ! |
509 |
||
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
510 |
!BreakpointService methodsFor:'testing'! |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
511 |
|
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
512 |
isBreakpointService |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
513 |
^ true |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
514 |
! ! |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
515 |
|
9982 | 516 |
!BreakpointService class methodsFor:'documentation'! |
517 |
||
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
518 |
version |
13201
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
519 |
^ '$Header: /cvs/stx/stx/libtool/Tools__BreakpointService.st,v 1.33 2013-07-22 11:39:55 cg Exp $' |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
520 |
! |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
521 |
|
9982 | 522 |
version_CVS |
13201
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
523 |
^ '$Header: /cvs/stx/stx/libtool/Tools__BreakpointService.st,v 1.33 2013-07-22 11:39:55 cg Exp $' |
9982 | 524 |
! |
525 |
||
526 |
version_SVN |
|
13201
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
527 |
^ '$Id: Tools__BreakpointService.st,v 1.33 2013-07-22 11:39:55 cg Exp $' |
9982 | 528 |
! ! |
12507
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
529 |