author | Claus Gittinger <cg@exept.de> |
Sat, 31 Aug 2013 00:54:49 +0200 | |
changeset 13420 | 3f8f012f121b |
parent 13395 | 64b70dc92a1c |
child 13491 | b3afe831ff0a |
child 13572 | 7b7d06d0d564 |
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 |
||
13206 | 75 |
!BreakpointService class methodsFor:'testing'! |
76 |
||
77 |
isUsefulFor:aCodeView |
|
78 |
"this filters useful services. |
|
79 |
Redefined to return true for myself - not for subclasses" |
|
80 |
||
81 |
^ self == Tools::BreakpointService |
|
82 |
||
83 |
"Created: / 22-07-2013 / 14:01:17 / cg" |
|
84 |
! ! |
|
85 |
||
10226 | 86 |
!BreakpointService methodsFor:'accessing'! |
87 |
||
88 |
breakpoints |
|
89 |
^ breakpoints |
|
90 |
! ! |
|
91 |
||
10208 | 92 |
!BreakpointService methodsFor:'change & update'! |
93 |
||
94 |
update: aspect with: param from: sender |
|
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
95 |
aspect == #visibility ifTrue:[^ self]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
96 |
aspect == #sizeOfView ifTrue:[^ self]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
97 |
aspect == #classHolder ifTrue:[^ self]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
98 |
aspect == #languageHolder ifTrue:[^ self]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
99 |
sender == codeView modifiedChannel ifTrue:[^ self]. |
10208 | 100 |
|
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
101 |
(aspect == #methodHolder or:[sender == codeView methodHolder]) ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
102 |
self updateCurrentMethod. |
10208 | 103 |
]. |
104 |
super update: aspect with: param from: sender |
|
105 |
||
106 |
"Created: / 06-07-2011 / 15:21:08 / cg" |
|
107 |
! |
|
108 |
||
109 |
updateBreakPointsFor:aMethod |
|
110 |
|methodsBreakPoints| |
|
111 |
||
13216
d1db940d42fb
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13209
diff
changeset
|
112 |
"/ Transcript show:'update breakpoints for method: '; showCR:aMethod. |
10208 | 113 |
aMethod notNil ifTrue:[ |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
114 |
aMethod literalsDo:[:eachLiteral | |
13216
d1db940d42fb
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13209
diff
changeset
|
115 |
(eachLiteral isKindOf:Breakpoint) ifTrue:[ |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
116 |
methodsBreakPoints isNil ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
117 |
methodsBreakPoints := OrderedCollection new. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
118 |
]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
119 |
methodsBreakPoints add:eachLiteral copy. |
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 |
]. |
13106 | 122 |
currentMethodClass := aMethod mclass. |
123 |
] ifFalse:[ |
|
124 |
currentMethodClass := nil |
|
10208 | 125 |
]. |
126 |
breakpoints := methodsBreakPoints. |
|
127 |
currentMethod := aMethod. |
|
128 |
||
129 |
"Created: / 06-07-2011 / 15:24:09 / cg" |
|
10226 | 130 |
"Modified: / 06-07-2011 / 17:32:54 / jv" |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
131 |
! |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
132 |
|
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
133 |
updateCurrentMethod |
13101 | 134 |
|method realMethod oldBreakPoints| |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
135 |
|
13106 | 136 |
"/ codeView methodHolder class == BlockValue ifTrue:[self breakPoint:#cg]. |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
137 |
|
13201
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
138 |
method := realMethod := codeView method. |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
139 |
(method notNil and:[method mclass isNil]) ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
140 |
"/ hack: ouch - was wrapped in the meantime; |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
141 |
"/ hurry up and update. Should be done elsewhere (in codeView) |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
142 |
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
|
143 |
realMethod isNil ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
144 |
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
|
145 |
]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
146 |
]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
147 |
realMethod ~~ currentMethod ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
148 |
"/ codeView methodHolder setValue:realMethod. |
13101 | 149 |
oldBreakPoints := breakpoints. |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
150 |
self updateBreakPointsFor:realMethod. |
13101 | 151 |
oldBreakPoints ~= breakpoints ifTrue:[ |
152 |
gutterView invalidate. |
|
153 |
] |
|
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
154 |
]. |
13201
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
155 |
|
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
156 |
"Modified: / 22-07-2013 / 13:33:28 / cg" |
10208 | 157 |
! ! |
158 |
||
9982 | 159 |
!BreakpointService methodsFor:'event handling'! |
160 |
||
11607 | 161 |
buttonPress:button x:x y:y in:view |
10714
3399fb50f42e
changed: #buttonPress:x:y:in:
Claus Gittinger <cg@exept.de>
parents:
10411
diff
changeset
|
162 |
|lineNr| |
3399fb50f42e
changed: #buttonPress:x:y:in:
Claus Gittinger <cg@exept.de>
parents:
10411
diff
changeset
|
163 |
|
9982 | 164 |
view == gutterView ifTrue:[ |
13152
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
165 |
button == 1 ifTrue:[ |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
166 |
lineNr := textView yVisibleToLineNr:y. |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
167 |
lineNr notNil ifTrue:[ self setOrToggleBreakpointAtLine:lineNr ]. |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
168 |
^ true. |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
169 |
]. |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
170 |
button == 3 ifTrue:[ |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
171 |
^ true. |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
172 |
] |
9982 | 173 |
]. |
10182 | 174 |
^ false |
9982 | 175 |
|
176 |
"Created: / 17-06-2011 / 13:05:22 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
177 |
"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
|
178 |
"Modified: / 19-09-2011 / 14:41:00 / cg" |
10226 | 179 |
! |
180 |
||
181 |
linesDeletedFrom: start to: end |
|
182 |
||
183 |
breakpoints isEmptyOrNil ifTrue:[^self]. |
|
184 |
self moveBreakpointsAfterLine: start - 1 by: (end - start + 1) negated |
|
185 |
||
186 |
"Created: / 06-07-2011 / 17:16:27 / jv" |
|
187 |
! |
|
188 |
||
189 |
linesInsertedFrom: start to: end |
|
190 |
||
191 |
breakpoints isEmptyOrNil ifTrue:[^self]. |
|
192 |
self moveBreakpointsAfterLine: start - 1 by: (end - start + 1) |
|
193 |
||
194 |
"Created: / 06-07-2011 / 17:16:36 / jv" |
|
9982 | 195 |
! ! |
196 |
||
13126
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
197 |
!BreakpointService methodsFor:'help'! |
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 |
flyByHelpText |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
200 |
|topView| |
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 |
(self canCreateOrToggleBreakpointAtLine:nil) ifFalse:[ |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
203 |
((topView := codeView topView) class == DebugView) ifTrue:[ |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
204 |
self hasBreakpoints ifFalse:[ |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
205 |
^ '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
|
206 |
]. |
13420
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
207 |
^ 'Sorry - cannot add new breakpoint if method is already entered\(i.e. if not stopped at a breakpoint).' withCRs |
13126
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
208 |
]. |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
209 |
^ 'Cannot add breakpoint when modified. Please accept first.' |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
210 |
]. |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
211 |
|
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
212 |
^ 'Click to toggle breakpoint. Shift-Click to toggle tracepoint.' |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
213 |
|
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
214 |
"Created: / 27-01-2012 / 14:04:52 / cg" |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
215 |
! ! |
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
216 |
|
9982 | 217 |
!BreakpointService methodsFor:'initialization'! |
218 |
||
219 |
initialize |
|
220 |
||
221 |
super initialize. |
|
222 |
breakpoints := OrderedCollection new. |
|
223 |
||
224 |
"Created: / 17-06-2011 / 13:49:12 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
225 |
! ! |
|
226 |
||
227 |
!BreakpointService methodsFor:'private'! |
|
228 |
||
11607 | 229 |
breakpointAtLine:line |
10182 | 230 |
|pos| |
9982 | 231 |
|
12969
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
232 |
breakpoints isNil ifTrue:[^ nil]. |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
233 |
|
9982 | 234 |
pos := textView characterPositionOfLine:line col:1. |
13191
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
235 |
^ breakpoints |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
236 |
detect:[:each | each position = pos ] |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
237 |
ifNone:[ |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
238 |
breakpoints |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
239 |
detect:[:each | each line == line and:[each position isNil ]] |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
240 |
ifNone:[ nil ] |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
241 |
] |
9982 | 242 |
|
243 |
"Modified: / 17-06-2011 / 13:59:17 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
10182 | 244 |
"Modified (format): / 05-07-2011 / 21:33:23 / cg" |
245 |
! |
|
246 |
||
10226 | 247 |
moveBreakpointsAfterLine:line by: delta |
248 |
|pos | |
|
249 |
||
250 |
breakpoints do:[:bpnt| |
|
11719 | 251 |
bpnt line >= line ifTrue:[ |
252 |
pos := textView characterPositionOfLine:bpnt line + delta col:1. |
|
253 |
bpnt position:pos line:(bpnt line + delta). |
|
254 |
] |
|
10226 | 255 |
]. |
256 |
||
257 |
"/gutterView redrawLinesFrom: line. |
|
258 |
||
259 |
"Created: / 17-06-2011 / 13:45:22 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
260 |
"Created: / 06-07-2011 / 17:26:30 / jv" |
|
11719 | 261 |
"Modified: / 02-08-2012 / 09:27:10 / cg" |
10226 | 262 |
! |
263 |
||
10182 | 264 |
recompile |
265 |
"recompile the current method for changed breakpoints" |
|
266 |
||
13209
887b3ce3c6e2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13206
diff
changeset
|
267 |
|oldMethod newMethod compilerClass compiler class selector| |
10182 | 268 |
|
13201
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
269 |
oldMethod := codeView method. |
11719 | 270 |
(oldMethod notNil and:[oldMethod hasPrimitiveCode not]) ifTrue:[ |
271 |
"/ be careful: if the text has been edited/modified, do not compile |
|
272 |
textView modified ifTrue:[ |
|
273 |
self breakPoint: #cg. |
|
274 |
self breakPoint: #jv. |
|
275 |
^self. |
|
276 |
] ifFalse:[ |
|
277 |
"/ prepare to get reachable bpts |
|
278 |
breakpoints do:[:bp | bp isReached:false]. |
|
279 |
||
280 |
class := oldMethod mclass. |
|
281 |
class isNil ifTrue:[ |
|
282 |
class := codeView classHolder value. |
|
12507
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
283 |
class isNil ifTrue:[ |
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
284 |
self breakPoint:#jv. |
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
285 |
Dialog warn:'oops - lost the methods''s class'. |
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
286 |
^ self. |
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
287 |
] |
11719 | 288 |
]. |
289 |
selector := oldMethod selector. |
|
10182 | 290 |
|
11719 | 291 |
Class withoutUpdatingChangesDo:[ |
13209
887b3ce3c6e2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13206
diff
changeset
|
292 |
"/ compilerClass := ByteCodeCompilerWithBreakpointSupport. |
887b3ce3c6e2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13206
diff
changeset
|
293 |
compilerClass := oldMethod programmingLanguage compilerWithBreakpointSupportClass. |
887b3ce3c6e2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13206
diff
changeset
|
294 |
compilerClass isNil ifTrue:[ |
887b3ce3c6e2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13206
diff
changeset
|
295 |
Dialog warn:'No breakpoint support for this programming language'. |
887b3ce3c6e2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13206
diff
changeset
|
296 |
^ self. |
887b3ce3c6e2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13206
diff
changeset
|
297 |
]. |
887b3ce3c6e2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13206
diff
changeset
|
298 |
compiler := compilerClass new. |
11719 | 299 |
compiler breakpoints:breakpoints. |
13209
887b3ce3c6e2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13206
diff
changeset
|
300 |
"/ not needed - new compilers already know it |
887b3ce3c6e2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13206
diff
changeset
|
301 |
"/ compiler methodClass:(oldMethod programmingLanguage isSTXJavaScript |
887b3ce3c6e2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13206
diff
changeset
|
302 |
"/ ifTrue:[JavaScriptFunctionWithBreakpoints] |
887b3ce3c6e2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13206
diff
changeset
|
303 |
"/ ifFalse:[MethodWithBreakpoints]). |
11719 | 304 |
newMethod := compiler |
305 |
compile:oldMethod source |
|
306 |
forClass:class |
|
307 |
inCategory:oldMethod category |
|
308 |
notifying:nil |
|
309 |
install:false |
|
310 |
skipIfSame:false |
|
311 |
silent:true |
|
312 |
foldConstants:true |
|
11987 | 313 |
ifFail:[ Transcript showCR:'BreakpointService: failed to recompile for breakpoint' ]. |
10182 | 314 |
|
11719 | 315 |
selector isNil ifTrue:[ |
316 |
"/ May happen as the selector is not stored in the method but |
|
317 |
"/ searches through method's mclass methodDictionary. |
|
318 |
"/ Following should be save as breakpoint is not installed when |
|
319 |
"/ the code is modified... |
|
320 |
selector := compiler selector. |
|
321 |
]. |
|
322 |
||
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
323 |
oldMethod isWrapped ifTrue:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
324 |
"/ update the wrapped method - do not install |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
325 |
newMethod originalMethod: oldMethod originalMethod. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
326 |
oldMethod replaceOriginalMethodWith:newMethod. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
327 |
] ifFalse:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
328 |
"/ install |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
329 |
newMethod originalMethod: oldMethod. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
330 |
(class primAddSelector: selector withMethod:newMethod) ifFalse:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
331 |
oldMethod mclass:class. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
332 |
self breakPoint: #cg. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
333 |
self breakPoint: #jv. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
334 |
^ self |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
335 |
]. |
11719 | 336 |
]. |
12969
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
337 |
|
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
338 |
breakpoints := breakpoints |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
339 |
select:[:bp | |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
340 |
"/ bp isReached ifFalse:[ |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
341 |
"/ "/ Transcript show:'remove unreached:'; showCR:bp |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
342 |
"/ ]. |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
343 |
bp isReached |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
344 |
]. |
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
345 |
|
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
346 |
"/ 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
|
347 |
"/ notification, which may clear the breakpoints collection!! |
11719 | 348 |
codeView methodHolder value:newMethod. |
349 |
oldMethod mclass isNil ifTrue:[ |
|
12507
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
350 |
"/ although this is not strictly true, not doing this |
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
351 |
"/ would confuse a lot of other tools (such as the browser) |
11719 | 352 |
oldMethod mclass:class. |
353 |
]. |
|
12507
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
354 |
class changed:#methodTrap with:selector. "/ tell browsers |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
355 |
Smalltalk changed:#methodTrap with:(MethodTrapChangeNotificationParameter changeClass:class changeSelector:selector). |
11719 | 356 |
]. |
357 |
] |
|
10182 | 358 |
] |
359 |
||
360 |
"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
|
361 |
"Modified: / 18-07-2012 / 10:53:22 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
13209
887b3ce3c6e2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13206
diff
changeset
|
362 |
"Modified: / 22-07-2013 / 16:00:13 / cg" |
9982 | 363 |
! |
364 |
||
11607 | 365 |
setOrToggleBreakpointAtLine:line |
13395
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
366 |
|pos bpnt prepareFullBreakSupport mClass ok| |
12940
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
367 |
|
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
368 |
"/ if true, setting a single breakpoint in a method will create |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
369 |
"/ a whole set of invisible (and disabled) breakpoints in that method, |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
370 |
"/ one for each line. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
371 |
"/ These can later be enabled in the debugger |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
372 |
"/ (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
|
373 |
"/ to set additional breakpoints). |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
374 |
"/ 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
|
375 |
"/ prepareFullBreakSupport := false. |
12940
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
376 |
prepareFullBreakSupport := true. |
9982 | 377 |
|
13201
3b6f308a7280
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13191
diff
changeset
|
378 |
codeView method isNil ifTrue:[ |
13152
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
379 |
^ self |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
380 |
]. |
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
381 |
|
12855
ab87c94ed5ac
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12507
diff
changeset
|
382 |
textView reallyModified ifTrue:[ |
ab87c94ed5ac
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12507
diff
changeset
|
383 |
"/ leads to ugly behavior (method no longer found), if we allow |
ab87c94ed5ac
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12507
diff
changeset
|
384 |
"/ this... |
13152
778f0a1b39c2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13126
diff
changeset
|
385 |
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
|
386 |
^ self |
ab87c94ed5ac
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12507
diff
changeset
|
387 |
]. |
ab87c94ed5ac
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12507
diff
changeset
|
388 |
|
9982 | 389 |
bpnt := self breakpointAtLine:line. |
10182 | 390 |
bpnt isNil ifTrue:[ |
13395
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
391 |
"/ no breakpoint there - create a new one as required (i.e. recompile) |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
392 |
ok := (self canCreateOrToggleBreakpointAtLine:line). |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
393 |
ok ifFalse:[ |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
394 |
codeView topView class == DebugView ifTrue:[ |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
395 |
(Dialog |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
396 |
confirm:'Sorry, in an active method, I can only add new breakpoints in an already breakpointed method. |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
397 |
(i.e. a method stopped at a method breakpoint or one which already has statement breakpoints) |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
398 |
The reason is that the method needs to be recompiled for the breakpoint, which would not affect the method being currently executed. |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
399 |
|
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
400 |
You can proceed to set the breakpoint, but it will only affect the next call into this method, not the current invocation.' |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
401 |
yesLabel:'Set Breakpoint for Next Call' noLabel:'Ok') ifTrue:[ |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
402 |
self halt. |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
403 |
ok := true. |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
404 |
] |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
405 |
] ifFalse:[ |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
406 |
Dialog warn:'Sorry, cannot add a new breakpoint here.'. |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
407 |
]. |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
408 |
]. |
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
409 |
ok ifTrue:[ |
12940
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
410 |
prepareFullBreakSupport ifTrue:[ |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
411 |
"/ add a (disabled) breakpoint for every source line. This |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
412 |
"/ allows for breakpoints to be enabled/disabled in the debugger... |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
413 |
1 to:textView numberOfLines do:[:eachLine | |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
414 |
|oldBPnt eachPos otherBpnt| |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
415 |
|
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
416 |
oldBPnt := self breakpointAtLine:eachLine. |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
417 |
oldBPnt isNil ifTrue:[ |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
418 |
eachPos := textView characterPositionOfLine:eachLine col:1. |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
419 |
breakpoints isNil ifTrue:[ breakpoints := OrderedCollection new]. |
12940
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
420 |
breakpoints add:((otherBpnt := Breakpoint new) position:eachPos line:eachLine). |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
421 |
eachLine == line ifTrue:[ |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
422 |
bpnt := otherBpnt. |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
423 |
] ifFalse:[ |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
424 |
otherBpnt beInvisible. |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
425 |
] |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
426 |
]. |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
427 |
]. |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
428 |
] ifFalse:[ |
13080
5ad43ae672b1
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12976
diff
changeset
|
429 |
pos := textView characterPositionOfLine:line col:1. |
12940
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
430 |
breakpoints add:((bpnt := Breakpoint new) position:pos line:line). |
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
431 |
]. |
11719 | 432 |
Display shiftDown ifTrue:[ |
433 |
"/ trace |
|
434 |
bpnt beTracepoint |
|
435 |
]. |
|
12969
3642a54942f0
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12949
diff
changeset
|
436 |
self assert: breakpoints notEmptyOrNil. |
13420
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
437 |
|
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
438 |
"/ recompile the method with breakpoints |
11719 | 439 |
self recompile. |
12906
58e97bbbf5a4
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12902
diff
changeset
|
440 |
] |
10182 | 441 |
] ifFalse:[ |
12940
7ef425531618
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12906
diff
changeset
|
442 |
"/ breakpoint already there - just enable/disable |
11719 | 443 |
Display shiftDown ifTrue:[ |
444 |
bpnt toggleTracing |
|
445 |
] ifFalse:[ |
|
446 |
bpnt toggle. |
|
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
447 |
]. |
13359
969a96c3a4b6
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13216
diff
changeset
|
448 |
(mClass := currentMethod mclass) isNil ifTrue:[ |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
449 |
"/ hack: ouch - was wrapped in the meantime; |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
450 |
"/ hurry up and update. Should be done elsewhere (in codeView) |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
451 |
self updateCurrentMethod. |
13420
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
452 |
currentMethod notNil ifTrue:[ mClass := currentMethod mclass ]. |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
453 |
]. |
13359
969a96c3a4b6
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13216
diff
changeset
|
454 |
mClass notNil ifTrue:[ |
969a96c3a4b6
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13216
diff
changeset
|
455 |
Smalltalk changed:#methodTrap with:(MethodTrapChangeNotificationParameter changeClass:mClass changeSelector:currentMethod selector). |
969a96c3a4b6
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13216
diff
changeset
|
456 |
]. |
10182 | 457 |
]. |
11987 | 458 |
|
9982 | 459 |
gutterView redrawLine:line. |
460 |
||
461 |
"Created: / 17-06-2011 / 13:45:22 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
10411 | 462 |
"Modified: / 27-07-2011 / 13:27:55 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
13395
64b70dc92a1c
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13359
diff
changeset
|
463 |
"Modified: / 28-08-2013 / 14:45:36 / cg" |
9982 | 464 |
! ! |
465 |
||
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
466 |
!BreakpointService methodsFor:'queries'! |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
467 |
|
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
468 |
canCreateOrToggleBreakpointAtLine:lineOrNilForAnywhere |
13420
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
469 |
"is it possible to place a breakpoint here and now?" |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
470 |
|
13126
6c0fc3e2986a
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13106
diff
changeset
|
471 |
|bpnt topView| |
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
472 |
|
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
473 |
textView reallyModified ifTrue:[ |
13420
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
474 |
"/ this is not really true - we could keep track of where the breakpoints |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
475 |
"/ are while editing and shift them as required. |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
476 |
"/ (another idea worth a try would be |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
477 |
"/ to match the original parsetree (enumerating nodes with the breakpoints) |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
478 |
"/ against the new parsetree (walking in sync?) when finally compiling, |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
479 |
"/ and placing new breakpoints on matching tree nodes. |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
480 |
"/ (too much work, for a quick solution, I guess) |
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
481 |
^ false |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
482 |
]. |
13420
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
483 |
|
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
484 |
"/ can always toggle existing breakpoints... |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
485 |
lineOrNilForAnywhere notNil ifTrue:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
486 |
bpnt := self breakpointAtLine:lineOrNilForAnywhere. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
487 |
bpnt notNil ifTrue:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
488 |
^ true. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
489 |
] |
13420
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
490 |
] ifFalse:[ |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
491 |
(currentMethod notNil and:[currentMethod isMethodWithBreakpoints]) ifTrue:[ |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
492 |
^ true. |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
493 |
] |
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
494 |
]. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
495 |
|
13420
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
496 |
"/ ok, the method has no breakpoints yet. |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
497 |
|
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
498 |
"/ this is a bad hack - looking into the debugger's state here. |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
499 |
"/ I guess, we have to move code around a bit... |
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
500 |
((topView := codeView topView) class == DebugView) ifTrue:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
501 |
"/ can only create new breakpoints in the debugger, |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
502 |
"/ iff we are in a wrapped method's prolog |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
503 |
topView selectedContextIsWrapped ifTrue:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
504 |
topView selectedContext lineNumber == 1 ifTrue:[ |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
505 |
^ true |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
506 |
]. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
507 |
]. |
13420
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
508 |
|
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
509 |
"/ well, if the debugger's code has already been modified, |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
510 |
"/ we will accept the new code anyway. So there's no problem in adding |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
511 |
"/ a breakpoint on the fly... |
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
512 |
topView showingAlreadyModifiedCode ifTrue:[^ true]. |
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
513 |
^ false. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
514 |
]. |
13420
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
515 |
|
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
516 |
"/ in a non-debugger, we can do it. |
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
517 |
^ true. |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
518 |
! |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
519 |
|
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
520 |
hasBreakpoints |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
521 |
^ breakpoints notEmptyOrNil |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
522 |
! ! |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
523 |
|
9982 | 524 |
!BreakpointService methodsFor:'redrawing'! |
525 |
||
11607 | 526 |
drawLine:lineNo in:view atX:x y:y width:w height:h from:startCol to:endColOrNil with:fg and:bg |
9982 | 527 |
"Called by both gutterView and textView (well, not yet) to |
528 |
allow services to draw custom things on text view. |
|
529 |
Ask JV what the args means if unsure (I'm lazy to document |
|
530 |
them, now it is just an experiment...)" |
|
11607 | 531 |
|
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
532 |
|mthd bpnt icon dx dy| |
9982 | 533 |
|
13191
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
534 |
"/ 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
|
535 |
"/ 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
|
536 |
"/ I can show it. Period. |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
537 |
|
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
538 |
"/ (mthd := codeView methodHolder value) isNil ifTrue:[ |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
539 |
"/ ^ self |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
540 |
"/ ]. |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
541 |
"/ currentMethodClass isNil ifTrue:[ |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
542 |
"/ "/ hack: ouch - was wrapped in the meantime; |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
543 |
"/ ^ self. "/ wait for the real update |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
544 |
"/ "/ hurry up and update. Should be done elsewhere (in codeView) |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
545 |
"/ "/ self updateCurrentMethod. |
9745eed50f03
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13157
diff
changeset
|
546 |
"/ ]. |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
547 |
|
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
548 |
view == gutterView ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
549 |
bpnt := self breakpointAtLine:lineNo. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
550 |
bpnt isNil ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
551 |
^ self |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
552 |
]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
553 |
icon := bpnt icon. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
554 |
icon isNil ifTrue:[ |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
555 |
^ self |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
556 |
]. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
557 |
dx := ((w - icon width) / 2) rounded. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
558 |
dy := ((h - icon height) / 2) rounded. |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
559 |
icon |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
560 |
displayOn:view |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
561 |
x:x + dx |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
562 |
y:y - h + dy + 4. "TODO: Magic constant" |
9982 | 563 |
]. |
564 |
||
565 |
"Created: / 17-06-2011 / 13:52:52 / Jan Vrany <jan.vrany@fit.cvut.cz>" |
|
10182 | 566 |
"Modified (format): / 05-07-2011 / 22:14:33 / cg" |
9982 | 567 |
! ! |
568 |
||
12902
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
569 |
!BreakpointService methodsFor:'testing'! |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
570 |
|
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
571 |
isBreakpointService |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
572 |
^ true |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
573 |
! ! |
724e09dfd9a2
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12855
diff
changeset
|
574 |
|
9982 | 575 |
!BreakpointService class methodsFor:'documentation'! |
576 |
||
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
577 |
version |
13420
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
578 |
^ '$Header: /cvs/stx/stx/libtool/Tools__BreakpointService.st,v 1.39 2013-08-30 22:54:49 cg Exp $' |
12949
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
579 |
! |
320a22d3c70b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
12940
diff
changeset
|
580 |
|
9982 | 581 |
version_CVS |
13420
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
582 |
^ '$Header: /cvs/stx/stx/libtool/Tools__BreakpointService.st,v 1.39 2013-08-30 22:54:49 cg Exp $' |
9982 | 583 |
! |
584 |
||
585 |
version_SVN |
|
13420
3f8f012f121b
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
13395
diff
changeset
|
586 |
^ '$Id: Tools__BreakpointService.st,v 1.39 2013-08-30 22:54:49 cg Exp $' |
9982 | 587 |
! ! |
12507
3fa9ae668d96
class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents:
11987
diff
changeset
|
588 |