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