Tools__BreakpointService.st
author Claus Gittinger <cg@exept.de>
Mon, 20 Jan 2020 21:02:47 +0100
changeset 19422 c6ca1c3e0fd7
parent 19356 2b2dbaa0751a
child 19439 3f67162018e1
permissions -rw-r--r--
#REFACTORING by exept class: MultiViewToolApplication added: #askForFile:default:forSave:thenDo: changed: #askForFile:default:thenDo: #askForFile:thenDo: #menuSaveAllAs #menuSaveAs
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
19336
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
     1
"{ Encoding: utf8 }"
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
     2
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
     3
"
10071
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
     4
 COPYRIGHT (c) 2010 by Jan Vrany, SWING Research Group. CTU in Prague
11607
3cd59e465a6a *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 11601
diff changeset
     5
	      All Rights Reserved
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
     6
10071
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
     7
Permission is hereby granted, free of charge, to any person
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
     8
obtaining a copy of this software and associated documentation
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
     9
files (the 'Software'), to deal in the Software without
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    10
restriction, including without limitation the rights to use,
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    11
copy, modify, merge, publish, distribute, sublicense, and/or sell
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    12
copies of the Software, and to permit persons to whom the
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    13
Software is furnished to do so, subject to the following
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    14
conditions:
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    15
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    16
The above copyright notice and this permission notice shall be
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    17
included in all copies or substantial portions of the Software.
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    18
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    19
THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    20
EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    21
OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    22
NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    23
HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    24
WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    25
FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    26
OTHER DEALINGS IN THE SOFTWARE.
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    27
"
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    28
"{ Package: 'stx:libtool' }"
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    29
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    30
"{ NameSpace: Tools }"
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    31
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    32
CodeViewService subclass:#BreakpointService
13572
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
    33
	instanceVariableNames:'breakpoints currentMethod currentMethodClass methodHolder'
19330
2d85a45f2ad2 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19287
diff changeset
    34
	classVariableNames:'NeedDoubleClickToToggleBreakpoints'
11607
3cd59e465a6a *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 11601
diff changeset
    35
	poolDictionaries:''
3cd59e465a6a *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 11601
diff changeset
    36
	category:'Interface-CodeView'
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    37
!
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    38
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    39
!BreakpointService class methodsFor:'documentation'!
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    40
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    41
copyright
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    42
"
10071
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    43
 COPYRIGHT (c) 2010 by Jan Vrany, SWING Research Group. CTU in Prague
11607
3cd59e465a6a *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 11601
diff changeset
    44
	      All Rights Reserved
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    45
10071
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    46
Permission is hereby granted, free of charge, to any person
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    47
obtaining a copy of this software and associated documentation
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    48
files (the 'Software'), to deal in the Software without
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    49
restriction, including without limitation the rights to use,
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    50
copy, modify, merge, publish, distribute, sublicense, and/or sell
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    51
copies of the Software, and to permit persons to whom the
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    52
Software is furnished to do so, subject to the following
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    53
conditions:
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    54
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    55
The above copyright notice and this permission notice shall be
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    56
included in all copies or substantial portions of the Software.
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    57
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    58
THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    59
EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    60
OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    61
NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    62
HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    63
WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    64
FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
1bf516aac0b8 changed: #copyright
Claus Gittinger <cg@exept.de>
parents: 9982
diff changeset
    65
OTHER DEALINGS IN THE SOFTWARE.
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    66
"
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
    67
!
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
    68
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
    69
documentation
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
    70
"
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
    71
    somewhat badly designed for non-reusability:
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
    72
       I should not care for recompiling methods here,
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
    73
       but instead delegate this task to a breakPointInstaller.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
    74
       The way it is currently designed makes it hard to reuse this
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
    75
       component in non-smalltalk setups (i.e. groovy-, node- or C editors).
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
    76
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
    77
    Fiddling around here breaks it almost for sure, as this is highly obfuscated code.   
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
    78
"
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    79
! !
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    80
13101
d0b3e4e52cba class: Tools::LintService
Claus Gittinger <cg@exept.de>
parents: 13080
diff changeset
    81
!BreakpointService class methodsFor:'accessing'!
d0b3e4e52cba class: Tools::LintService
Claus Gittinger <cg@exept.de>
parents: 13080
diff changeset
    82
d0b3e4e52cba class: Tools::LintService
Claus Gittinger <cg@exept.de>
parents: 13080
diff changeset
    83
label
d0b3e4e52cba class: Tools::LintService
Claus Gittinger <cg@exept.de>
parents: 13080
diff changeset
    84
    "Answers a short label - for UI"
d0b3e4e52cba class: Tools::LintService
Claus Gittinger <cg@exept.de>
parents: 13080
diff changeset
    85
d0b3e4e52cba class: Tools::LintService
Claus Gittinger <cg@exept.de>
parents: 13080
diff changeset
    86
    ^ 'Breakpoints'
d0b3e4e52cba class: Tools::LintService
Claus Gittinger <cg@exept.de>
parents: 13080
diff changeset
    87
! !
d0b3e4e52cba class: Tools::LintService
Claus Gittinger <cg@exept.de>
parents: 13080
diff changeset
    88
13206
9dc79fafe94c class: Tools::XXXService
Claus Gittinger <cg@exept.de>
parents: 13201
diff changeset
    89
!BreakpointService class methodsFor:'testing'!
9dc79fafe94c class: Tools::XXXService
Claus Gittinger <cg@exept.de>
parents: 13201
diff changeset
    90
9dc79fafe94c class: Tools::XXXService
Claus Gittinger <cg@exept.de>
parents: 13201
diff changeset
    91
isUsefulFor:aCodeView
9dc79fafe94c class: Tools::XXXService
Claus Gittinger <cg@exept.de>
parents: 13201
diff changeset
    92
    "this filters useful services.
9dc79fafe94c class: Tools::XXXService
Claus Gittinger <cg@exept.de>
parents: 13201
diff changeset
    93
     Redefined to return true for myself - not for subclasses"
9dc79fafe94c class: Tools::XXXService
Claus Gittinger <cg@exept.de>
parents: 13201
diff changeset
    94
9dc79fafe94c class: Tools::XXXService
Claus Gittinger <cg@exept.de>
parents: 13201
diff changeset
    95
    ^ self == Tools::BreakpointService
9dc79fafe94c class: Tools::XXXService
Claus Gittinger <cg@exept.de>
parents: 13201
diff changeset
    96
9dc79fafe94c class: Tools::XXXService
Claus Gittinger <cg@exept.de>
parents: 13201
diff changeset
    97
    "Created: / 22-07-2013 / 14:01:17 / cg"
9dc79fafe94c class: Tools::XXXService
Claus Gittinger <cg@exept.de>
parents: 13201
diff changeset
    98
! !
9dc79fafe94c class: Tools::XXXService
Claus Gittinger <cg@exept.de>
parents: 13201
diff changeset
    99
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   100
!BreakpointService methodsFor:'accessing'!
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   101
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   102
breakpointAtLine:line
14845
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   103
    "return the breakpoint at line (may be disabled) or nil, if there is none"
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   104
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   105
    |pos|
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   106
17620
b7346dd44eb5 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 17500
diff changeset
   107
    breakpoints isEmptyOrNil ifTrue:[^ nil].
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   108
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   109
    pos := textView characterPositionOfLine:line col:1.
19019
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   110
    "/ breakpoints are sorted by line 
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   111
    "/ (or even by position within a line, but those are currently not supported by the GUI).
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   112
    "/ We can stop searching early
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   113
    breakpoints do:[:each |
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   114
        |bpLine|
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   115
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   116
        bpLine := each line.
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   117
        bpLine == line ifTrue:[^ each].
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   118
        bpLine > line ifTrue:[^ nil].
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   119
    ].
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   120
    ^ nil.
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   121
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   122
"/    ^ breakpoints 
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   123
"/        detect:[:each | each position = pos ] 
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   124
"/        ifNone:[ 
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   125
"/            breakpoints 
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   126
"/                detect:[:each | each line == line and:[each position isNil ]] 
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   127
"/                ifNone:[ 
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   128
"/                    self halt.
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   129
"/                    nil 
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   130
"/                ]
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   131
"/        ]
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   132
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   133
    "Modified: / 17-06-2011 / 13:59:17 / Jan Vrany <jan.vrany@fit.cvut.cz>"
17620
b7346dd44eb5 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 17500
diff changeset
   134
    "Modified: / 16-08-2017 / 08:29:51 / cg"
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   135
!
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   136
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   137
breakpoints
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   138
    self fixupBreakpointPositions.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   139
    ^ breakpoints
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   140
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   141
    "Modified: / 08-05-2014 / 14:02:25 / Jan Vrany <jan.vrany@fit.cvut.cz>"
18599
e416923e5f14 #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 18565
diff changeset
   142
    "Modified (comment): / 18-02-2019 / 20:11:07 / Claus Gittinger"
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   143
!
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   144
18565
9bc3833f43e9 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17914
diff changeset
   145
firstBreakpointAtLine:line
9bc3833f43e9 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17914
diff changeset
   146
    "return the breakpoint at line (may be disabled) or nil, if there is none"
9bc3833f43e9 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17914
diff changeset
   147
9bc3833f43e9 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17914
diff changeset
   148
    breakpoints isEmptyOrNil ifTrue:[^ nil].
9bc3833f43e9 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17914
diff changeset
   149
9bc3833f43e9 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17914
diff changeset
   150
    ^ breakpoints 
9bc3833f43e9 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17914
diff changeset
   151
        detect:[:each | each line == line ] 
9bc3833f43e9 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17914
diff changeset
   152
        ifNone:[ nil ]
9bc3833f43e9 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17914
diff changeset
   153
9bc3833f43e9 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17914
diff changeset
   154
    "Created: / 26-01-2019 / 18:49:33 / Claus Gittinger"
9bc3833f43e9 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17914
diff changeset
   155
!
9bc3833f43e9 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17914
diff changeset
   156
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   157
removeAllBreakpoints
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   158
    breakpoints := OrderedCollection new.
14510
5e9448af6e4f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14509
diff changeset
   159
    gutterView invalidate.
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   160
!
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   161
19336
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   162
setBreakpointAtLine:line
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   163
    "unconditionally set a breakpoint (called via double click)"
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   164
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   165
    self setOrToggleBreakpointAtLine:line toggle:false
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   166
!
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   167
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   168
setOrToggleBreakpointAtLine:line
19336
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   169
    "toggle a breakpoint (called via single click)"
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   170
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   171
    self setOrToggleBreakpointAtLine:line toggle:true
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   172
!
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   173
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   174
setOrToggleBreakpointAtLine:line toggle:doToggleBoolean
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   175
    "set (via double click) or toggle (via single click) a breakpoint"
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   176
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   177
    |pos bpnt prepareFullBreakSupport mClass ok|
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   178
19336
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   179
    "/ if prepareFullBreakSupport is true, setting a single breakpoint in a method will create
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   180
    "/ a whole set of invisible (and disabled) breakpoints in that method, one for each line.
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   181
    "/ These can later be enabled in the debugger
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   182
    "/ (otherwise, the debugger's behavior is stupid, as it cannot recompile a method
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   183
    "/ to set additional breakpoints).
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   184
    "/ We accept the additional overhead, as we are in debug mode anyway.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   185
    "/ prepareFullBreakSupport := false.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   186
    prepareFullBreakSupport := true.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   187
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   188
    codeView method isNil ifTrue:[
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   189
        ^ nil
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   190
    ].
17500
7f309137beb8 #BUGFIX by cg
Claus Gittinger <cg@exept.de>
parents: 17478
diff changeset
   191
    currentMethod isNil ifTrue:[
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   192
        ^ nil
17072
1035a2331a0c #BUGFIX by cg
Claus Gittinger <cg@exept.de>
parents: 16602
diff changeset
   193
    ].
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   194
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   195
    bpnt := self breakpointAtLine:line.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   196
    bpnt isNil ifTrue:[
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   197
        "/ no breakpoint there - create a new one as required (i.e. recompile)
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   198
        ok := (self canCreateOrToggleBreakpointAtLine:line).
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   199
        ok ifFalse:[
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   200
            (currentMethod isMethodWithBreakpoints and:[ prepareFullBreakSupport ]) ifFalse:[
17158
fda0a732e85e #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17157
diff changeset
   201
                (codeView topView notNil and:[codeView topView isDebugView]) ifTrue:[
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   202
                    (Dialog 
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   203
                        confirm:'Sorry, in an active method, I can only add new breakpoints in an already breakpointed method.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   204
(i.e. a method stopped at a method breakpoint or one which already has statement breakpoints)
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   205
The reason is that the method needs to be recompiled for the breakpoint, which would not affect the method being currently executed.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   206
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   207
You can proceed to set the breakpoint, but it will only affect the next call into this method, not the current invocation.'
16334
66976ce59ba9 #UI_ENHANCEMENT by mawalch
mawalch
parents: 15983
diff changeset
   208
                        yesLabel:'Set Breakpoint for Next Call' noLabel:'OK') ifTrue:[
14950
29261c456545 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14849
diff changeset
   209
"/ self halt.
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   210
                            ok := true.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   211
                        ]
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   212
                ] ifFalse:[
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   213
                    Dialog warn:'Sorry, cannot add a new breakpoint here.'.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   214
                ].
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   215
            ]
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   216
        ].
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   217
        ok ifTrue:[
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   218
            prepareFullBreakSupport ifTrue:[
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   219
                "/ add a (disabled) breakpoint for every source line. This
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   220
                "/ allows for breakpoints to be enabled/disabled in the debugger...
19287
9945d4454c58 #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 19194
diff changeset
   221
                breakpoints notNil ifTrue:[ self sortBreakpoints ].
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   222
                1 to:textView numberOfLines do:[:eachLine |
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   223
                    |oldBPnt eachPos otherBpnt|
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   224
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   225
                    oldBPnt := self breakpointAtLine:eachLine.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   226
                    oldBPnt isNil ifTrue:[
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   227
                        eachPos := textView characterPositionOfLine:eachLine col:1.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   228
                        breakpoints isNil ifTrue:[ breakpoints := OrderedCollection new].
15303
a5083c63035a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 15045
diff changeset
   229
                        breakpoints add:((otherBpnt := Breakpoint new method:codeView method) position:eachPos line:eachLine).
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   230
                        eachLine == line ifTrue:[
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   231
                            bpnt := otherBpnt.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   232
                        ] ifFalse:[
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   233
                            otherBpnt beInvisible.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   234
                        ]
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   235
                    ].
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   236
                ].
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   237
            ] ifFalse:[
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   238
                pos := textView characterPositionOfLine:line col:1.
15303
a5083c63035a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 15045
diff changeset
   239
                breakpoints add:((bpnt := Breakpoint new method:codeView method) position:pos line:line).
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   240
            ].
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   241
            Screen current shiftDown ifTrue:[
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   242
                "/ trace
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   243
                bpnt beTracepoint
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   244
            ].
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   245
            self assert: breakpoints notEmptyOrNil.
19287
9945d4454c58 #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 19194
diff changeset
   246
            self sortBreakpoints.
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   247
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   248
            "/ recompile the method with breakpoints
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   249
            self recompile.
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   250
        ]
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   251
    ] ifFalse:[
19336
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   252
        (doToggleBoolean or:[bpnt isEnabled not]) ifTrue:[
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   253
            "/ breakpoint already there - just enable/disable
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   254
            Screen current shiftDown ifTrue:[
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   255
                bpnt toggleTracing
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   256
            ] ifFalse:[
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   257
                bpnt toggle.
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   258
            ].
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   259
            (currentMethod isNil or:[(mClass := currentMethod mclass) isNil]) ifTrue:[
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   260
                "/ hack: ouch - was wrapped in the meantime;
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   261
                "/ hurry up and update. Should be done elsewhere (in codeView)
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   262
                self updateCurrentMethod.
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   263
                currentMethod notNil ifTrue:[ mClass := currentMethod mclass ].
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   264
            ].
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   265
            bpnt method:currentMethod.
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   266
            mClass notNil ifTrue:[
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   267
                Smalltalk changed:#methodTrap with:(MethodTrapChangeNotificationParameter changeClass:mClass changeSelector:currentMethod selector).
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   268
            ].
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   269
        ].
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   270
    ].
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   271
18644
f4e359c06bef #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 18620
diff changeset
   272
    bpnt notNil ifTrue:[
19336
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   273
        "/ if it was ignored in the debugger, reenable
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   274
        Debugger notNil ifTrue:[
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   275
            Debugger stopIgnoringHaltsFor:currentMethod atLineNr:bpnt line
18644
f4e359c06bef #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 18620
diff changeset
   276
        ].    
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   277
    ].
18644
f4e359c06bef #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 18620
diff changeset
   278
    
f4e359c06bef #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 18620
diff changeset
   279
    gutterView redrawLine:line.
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   280
    ^ bpnt
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   281
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   282
    "Created: / 17-06-2011 / 13:45:22 / Jan Vrany <jan.vrany@fit.cvut.cz>"
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   283
    "Modified: / 21-02-2014 / 17:36:11 / Jan Vrany <jan.vrany@fit.cvut.cz>"
17500
7f309137beb8 #BUGFIX by cg
Claus Gittinger <cg@exept.de>
parents: 17478
diff changeset
   284
    "Modified: / 13-04-2017 / 09:59:36 / cg"
18644
f4e359c06bef #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 18620
diff changeset
   285
    "Modified: / 02-03-2019 / 11:32:07 / Claus Gittinger"
14509
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   286
! !
763bb2f56f2f class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14456
diff changeset
   287
10208
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   288
!BreakpointService methodsFor:'change & update'!
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   289
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   290
update: aspect with: param from: sender
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   291
    aspect == #visibility ifTrue:[^ self].
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   292
    aspect == #sizeOfView ifTrue:[^ self].
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   293
    aspect == #classHolder ifTrue:[^ self].
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   294
    aspect == #languageHolder ifTrue:[^ self].
18620
b80e38590f3a #BUGFIX by cg
Claus Gittinger <cg@exept.de>
parents: 18606
diff changeset
   295
    (codeView notNil and:[sender == codeView modifiedChannel]) ifTrue:[^ self].
10208
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   296
13572
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   297
    "/ ATTENTION: I thought that fetching the methodHolder once in initialize would
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   298
    "/ be ok. However, it seems not (codeView gives me something which it does not use/change later).
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   299
    "/ so this fetch is called in update as well.
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   300
    "/ maybe s.o. (ie. Jan) fixes CodeView2
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   301
    (aspect == #methodHolder or:[sender == self fetchMethodHolder]) ifTrue:[
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   302
        self updateCurrentMethod.
10208
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   303
    ].
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   304
    super update: aspect with: param from: sender
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   305
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   306
    "Created: / 06-07-2011 / 15:21:08 / cg"
18620
b80e38590f3a #BUGFIX by cg
Claus Gittinger <cg@exept.de>
parents: 18606
diff changeset
   307
    "Modified: / 26-02-2019 / 18:06:23 / Claus Gittinger"
10208
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   308
!
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   309
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   310
updateBreakPointsFor:aMethod
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   311
    |methodsBreakPoints|
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   312
13216
d1db940d42fb class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13209
diff changeset
   313
    "/ Transcript show:'update breakpoints for method: '; showCR:aMethod.
10208
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   314
    aMethod notNil ifTrue:[
15383
671638959b16 Use breakpointsDo: instead of enumerating over literals.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 15303
diff changeset
   315
        aMethod breakpointsDo:[:eachLiteral |
671638959b16 Use breakpointsDo: instead of enumerating over literals.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 15303
diff changeset
   316
            methodsBreakPoints isNil ifTrue:[
671638959b16 Use breakpointsDo: instead of enumerating over literals.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 15303
diff changeset
   317
                methodsBreakPoints := OrderedCollection new.
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   318
            ].
15383
671638959b16 Use breakpointsDo: instead of enumerating over literals.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 15303
diff changeset
   319
            methodsBreakPoints add:eachLiteral copy.
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   320
        ].
13106
00e8489b6674 codeView2 fixes
Claus Gittinger <cg@exept.de>
parents: 13101
diff changeset
   321
        currentMethodClass := aMethod mclass.
00e8489b6674 codeView2 fixes
Claus Gittinger <cg@exept.de>
parents: 13101
diff changeset
   322
    ] ifFalse:[
00e8489b6674 codeView2 fixes
Claus Gittinger <cg@exept.de>
parents: 13101
diff changeset
   323
        currentMethodClass := nil
10208
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   324
    ].
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   325
    breakpoints := methodsBreakPoints.
17620
b7346dd44eb5 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 17500
diff changeset
   326
    breakpoints notEmptyOrNil ifTrue:[ 
14456
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   327
        "/ Nil out breakpoint's position. It's invalid as soon as
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   328
        "/ user edits the code. Instead, depend on line information.
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   329
        "/ Breakpoint character positions are fixed up just before
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   330
        "/ passing a breakpoints to the compiler, see
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   331
        "/ #fixupBreakpointPositions
15642
239bbe1c7328 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 15383
diff changeset
   332
        breakpoints do:[:each | each method:aMethod. each position: nil ].
14456
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   333
    ].
10208
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   334
    currentMethod := aMethod.
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   335
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   336
    "Created: / 06-07-2011 / 15:24:09 / cg"
10226
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   337
    "Modified: / 06-07-2011 / 17:32:54 / jv"
15383
671638959b16 Use breakpointsDo: instead of enumerating over literals.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 15303
diff changeset
   338
    "Modified: / 23-02-2015 / 14:48:03 / Jan Vrany <jan.vrany@fit.cvut.cz>"
17620
b7346dd44eb5 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 17500
diff changeset
   339
    "Modified: / 16-08-2017 / 08:31:18 / cg"
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   340
!
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   341
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   342
updateCurrentMethod
13101
d0b3e4e52cba class: Tools::LintService
Claus Gittinger <cg@exept.de>
parents: 13080
diff changeset
   343
    |method realMethod oldBreakPoints|
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   344
13106
00e8489b6674 codeView2 fixes
Claus Gittinger <cg@exept.de>
parents: 13101
diff changeset
   345
    "/ codeView methodHolder class == BlockValue ifTrue:[self breakPoint:#cg].
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   346
13201
3b6f308a7280 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13191
diff changeset
   347
    method := realMethod := codeView method.
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   348
    (method notNil and:[method mclass isNil]) ifTrue:[
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   349
        "/ hack: ouch - was wrapped in the meantime;
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   350
        "/ hurry up and update. Should be done elsewhere (in codeView)
14533
b0ab75a1f038 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14510
diff changeset
   351
        realMethod := MethodWithBreakpoints allBreakpointedMethods detect:[:m | m originalMethod == method and:[m mclass notNil]] ifNone:nil.
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   352
        realMethod isNil ifTrue:[
14533
b0ab75a1f038 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14510
diff changeset
   353
            realMethod := WrappedMethod allWrappedMethods detect:[:m | m originalMethod == method and:[m mclass notNil]] ifNone:nil.
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   354
        ].
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   355
    ].
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   356
    realMethod ~~ currentMethod ifTrue:[
13573
70135af4ddeb class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13572
diff changeset
   357
        "/ self fetchMethodHolder setValue:realMethod.
13101
d0b3e4e52cba class: Tools::LintService
Claus Gittinger <cg@exept.de>
parents: 13080
diff changeset
   358
        oldBreakPoints := breakpoints.
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   359
        self updateBreakPointsFor:realMethod.
13101
d0b3e4e52cba class: Tools::LintService
Claus Gittinger <cg@exept.de>
parents: 13080
diff changeset
   360
        oldBreakPoints ~= breakpoints ifTrue:[
d0b3e4e52cba class: Tools::LintService
Claus Gittinger <cg@exept.de>
parents: 13080
diff changeset
   361
            gutterView invalidate.
d0b3e4e52cba class: Tools::LintService
Claus Gittinger <cg@exept.de>
parents: 13080
diff changeset
   362
        ]
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   363
    ].
13201
3b6f308a7280 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13191
diff changeset
   364
3b6f308a7280 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13191
diff changeset
   365
    "Modified: / 22-07-2013 / 13:33:28 / cg"
10208
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   366
! !
5f0f44bd199a class definition
Claus Gittinger <cg@exept.de>
parents: 10182
diff changeset
   367
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   368
!BreakpointService methodsFor:'event handling'!
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   369
14845
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   370
buttonMultiPress:button x:x y:y in:view
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   371
    view == gutterView ifTrue:[
13152
778f0a1b39c2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13126
diff changeset
   372
        button == 1 ifTrue:[
19336
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   373
            self setBreakpointAtX:x y:y.
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   374
            "/ self setOrToggleBreakpointAtX:x y:y.
13152
778f0a1b39c2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13126
diff changeset
   375
            ^ true.
778f0a1b39c2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13126
diff changeset
   376
        ].
778f0a1b39c2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13126
diff changeset
   377
        button == 3 ifTrue:[
19071
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   378
            "/ cg:huch - why that?
13152
778f0a1b39c2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13126
diff changeset
   379
            ^ true.
778f0a1b39c2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13126
diff changeset
   380
        ]
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   381
    ].
10182
9ce79271722f added: #recompile
Claus Gittinger <cg@exept.de>
parents: 10071
diff changeset
   382
    ^ false
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   383
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   384
    "Created: / 17-06-2011 / 13:05:22 / Jan Vrany <jan.vrany@fit.cvut.cz>"
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   385
    "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
   386
    "Modified: / 19-09-2011 / 14:41:00 / cg"
10226
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   387
!
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   388
14845
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   389
buttonPress:button x:x y:y in:view
19071
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   390
    "Handles an event in given view (a subview of codeView).
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   391
     If the method returns true, it has eaten the event and it will not be processed
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   392
     by the view or other services.
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   393
     Notice, that this is called both for events in the gutter AND for events in the text."
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   394
18664
8c2338820773 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18644
diff changeset
   395
    |lineNr currentHelpListener|
14845
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   396
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   397
    "now disabled: need a double click (like in other editors);
18644
f4e359c06bef #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 18620
diff changeset
   398
     also this allows toggling breakpoints even if there are service-annotations"
14845
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   399
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   400
    view == gutterView ifTrue:[
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   401
        button == 1 ifTrue:[
19331
f0c28eeaf3e5 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19330
diff changeset
   402
            "/ see hack in CodeView2 >> buttonPress:x:y:in:
f0c28eeaf3e5 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19330
diff changeset
   403
            ^ false.
f0c28eeaf3e5 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19330
diff changeset
   404
"/            lineNr := textView yVisibleToLineNr:y.
f0c28eeaf3e5 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19330
diff changeset
   405
"/            (self singleClickToToggle) ifFalse:[
f0c28eeaf3e5 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19330
diff changeset
   406
"/                (currentHelpListener := FlyByHelp currentHelpListener) notNil ifTrue:[
f0c28eeaf3e5 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19330
diff changeset
   407
"/                    "/ show a message that a double click is now needed    
f0c28eeaf3e5 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19330
diff changeset
   408
"/                    currentHelpListener initiateHelpFor:view at:x@y now:true
f0c28eeaf3e5 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19330
diff changeset
   409
"/                ].
f0c28eeaf3e5 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19330
diff changeset
   410
"/                ^ false.
f0c28eeaf3e5 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19330
diff changeset
   411
"/            ].
f0c28eeaf3e5 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19330
diff changeset
   412
"/            lineNr notNil ifTrue:[ self setOrToggleBreakpointAtLine:lineNr ].
f0c28eeaf3e5 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19330
diff changeset
   413
"/            ^ true.
14845
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   414
        ].
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   415
    ].
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   416
    ^ false
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   417
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   418
    "Created: / 17-06-2011 / 13:05:22 / Jan Vrany <jan.vrany@fit.cvut.cz>"
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   419
    "Modified: / 19-09-2011 / 14:41:00 / cg"
14971
a33058120f3d Make service not eat click event - it does not install breakpoint on signle-click!
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14950
diff changeset
   420
    "Modified: / 03-12-2014 / 10:22:52 / Jan Vrany <jan.vrany@fit.cvut.cz>"
18664
8c2338820773 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18644
diff changeset
   421
    "Modified: / 12-03-2019 / 13:01:22 / Claus Gittinger"
14845
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   422
!
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   423
14326
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   424
keyPress:key x:x y:y in: view
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   425
    "Handles an event in given view (a subview of codeView).
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   426
     If the method returns true, it has eaten the event and it will not be processed
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   427
     by the view."
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   428
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   429
    key == #Accept ifTrue:[ 
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   430
        textView undoableDo:[
14328
9510671ea784 Fixes for breakpoint management when editing the code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14326
diff changeset
   431
            BreakpointQuery answer: self breakpoints do:[
14326
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   432
                textView accept.
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   433
            ].
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   434
        ].
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   435
        ^ true
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   436
    ].
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   437
    ^ false
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   438
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   439
    "Created: / 08-05-2014 / 10:42:56 / Jan Vrany <jan.vrany@fit.cvut.cz>"
14328
9510671ea784 Fixes for breakpoint management when editing the code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14326
diff changeset
   440
    "Modified: / 08-05-2014 / 13:52:38 / Jan Vrany <jan.vrany@fit.cvut.cz>"
14326
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   441
!
8b59bf2da03e When accepting a modified code, tell the compiler to reinsert breakpoints if any.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 13977
diff changeset
   442
10226
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   443
linesDeletedFrom: start to: end
14456
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   444
    | breakpointsToRemove |
10226
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   445
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   446
    breakpoints isEmptyOrNil ifTrue:[^self].
14845
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   447
14456
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   448
    breakpointsToRemove := Set new.
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   449
    start to: end do:[:line | 
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   450
        | breakpointToRemove |
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   451
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   452
        breakpointToRemove := self breakpointAtLine: line.
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   453
        breakpointToRemove notNil ifTrue:[ breakpointsToRemove add: breakpointToRemove ].
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   454
    ].
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   455
    breakpoints removeAll: breakpointsToRemove.
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   456
    self moveBreakpointsAfterLine: end by: (end - start + 1) negated
10226
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   457
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   458
    "Created: / 06-07-2011 / 17:16:27 / jv"
14456
218e0f00e309 Workaround for disappearing breakpoint icons after editing source code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14454
diff changeset
   459
    "Modified: / 04-06-2014 / 17:56:56 / Jan Vrany <jan.vrany@fit.cvut.cz>"
10226
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   460
!
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   461
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   462
linesInsertedFrom: start to: end
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   463
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   464
    breakpoints isEmptyOrNil ifTrue:[^self].
14845
7c4a218df889 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14533
diff changeset
   465
10226
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   466
    self moveBreakpointsAfterLine: start - 1 by: (end - start + 1)
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   467
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   468
    "Created: / 06-07-2011 / 17:16:36 / jv"
19071
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   469
!
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   470
19336
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   471
setBreakpointAtX:x y:y
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   472
    |lineNr|
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   473
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   474
    lineNr := textView yVisibleToLineNr:y.
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   475
    lineNr notNil ifTrue:[ self setBreakpointAtLine:lineNr ].
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   476
!
e63f494b9f68 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19332
diff changeset
   477
19071
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   478
setOrToggleBreakpointAtX:x y:y
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   479
    |lineNr|
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   480
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   481
    lineNr := textView yVisibleToLineNr:y.
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   482
    lineNr notNil ifTrue:[ self setOrToggleBreakpointAtLine:lineNr ].
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   483
! !
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   484
13126
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   485
!BreakpointService methodsFor:'help'!
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   486
18844
2556c2dcb567 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18828
diff changeset
   487
basicHelpTextAtLine:ignoredLineNr
13126
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   488
    |topView|
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   489
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   490
    (self canCreateOrToggleBreakpointAtLine:nil) ifFalse:[
17157
bf9addfe2aa2 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 17092
diff changeset
   491
        ((topView := codeView topView) notNil and:[topView isDebugView]) ifTrue:[
13126
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   492
            self hasBreakpoints ifFalse:[
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   493
                ^ '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
   494
            ].
13420
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   495
            ^ '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
   496
        ].
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   497
        ^ 'Cannot add breakpoint when modified. Please accept first.'
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   498
    ].
19101
d2161c8d9307 #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19097
diff changeset
   499
    self singleClickToToggle ifTrue:[
19356
2b2dbaa0751a #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19336
diff changeset
   500
        ^ 'Click to toggle breakpoint.\For tracepoint, click with SHIFT pressed.\\Double click, if other info is shown here (lint info)'
19101
d2161c8d9307 #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19097
diff changeset
   501
    ].
19104
64e63da7d061 #UI_ENHANCEMENT by exept
Claus Gittinger <cg@exept.de>
parents: 19102
diff changeset
   502
    ^ 'Double click to toggle breakpoint.\For tracepoint, click with SHIFT pressed.'
13126
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   503
18844
2556c2dcb567 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18828
diff changeset
   504
    "Created: / 26-06-2019 / 19:50:56 / Claus Gittinger"
2556c2dcb567 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18828
diff changeset
   505
!
2556c2dcb567 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18828
diff changeset
   506
2556c2dcb567 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18828
diff changeset
   507
helpTextAtLine:ignoredLineNr
19102
107ac0f21f00 #UI_ENHANCEMENT by exept
Claus Gittinger <cg@exept.de>
parents: 19101
diff changeset
   508
    |helpText|
107ac0f21f00 #UI_ENHANCEMENT by exept
Claus Gittinger <cg@exept.de>
parents: 19101
diff changeset
   509
19071
58aedf69e507 #OTHER by exept
Claus Gittinger <cg@exept.de>
parents: 19019
diff changeset
   510
    (codeView method isNil and:[currentMethod isNil]) ifTrue:[
19102
107ac0f21f00 #UI_ENHANCEMENT by exept
Claus Gittinger <cg@exept.de>
parents: 19101
diff changeset
   511
        helpText := 'No current method. Cannot set breakpoint.'
107ac0f21f00 #UI_ENHANCEMENT by exept
Claus Gittinger <cg@exept.de>
parents: 19101
diff changeset
   512
    ] ifFalse:[
107ac0f21f00 #UI_ENHANCEMENT by exept
Claus Gittinger <cg@exept.de>
parents: 19101
diff changeset
   513
        helpText := self basicHelpTextAtLine:ignoredLineNr
18844
2556c2dcb567 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18828
diff changeset
   514
    ].
19102
107ac0f21f00 #UI_ENHANCEMENT by exept
Claus Gittinger <cg@exept.de>
parents: 19101
diff changeset
   515
    ^ self resources stringWithCRs:helpText.
18844
2556c2dcb567 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18828
diff changeset
   516
13126
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   517
    "Created: / 27-01-2012 / 14:04:52 / cg"
17478
f02e9da087b3 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 17441
diff changeset
   518
    "Modified: / 19-03-2017 / 01:15:48 / cg"
18844
2556c2dcb567 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18828
diff changeset
   519
    "Modified: / 26-06-2019 / 19:52:18 / Claus Gittinger"
13126
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   520
! !
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   521
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   522
!BreakpointService methodsFor:'initialization'!
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   523
13572
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   524
fetchMethodHolder
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   525
    "redefinable in subclasses, which do not have a codeView2/methodHolder"
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   526
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   527
    "/ ATTENTION: I thought that fetching the methodHolder once in initialize would
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   528
    "/ be ok. However, it seems not (codeView gives me something which it does not use/change later).
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   529
    "/ so this fetch is called in update as well.
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   530
    "/ maybe s.o. (ie. Jan) fixes CodeView2
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   531
    methodHolder := codeView methodHolder.
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   532
    ^ methodHolder
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   533
!
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   534
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   535
initialize
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   536
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   537
    super initialize.
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   538
    breakpoints := OrderedCollection new.
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   539
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   540
    "Created: / 17-06-2011 / 13:49:12 / Jan Vrany <jan.vrany@fit.cvut.cz>"
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   541
! !
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   542
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   543
!BreakpointService methodsFor:'private'!
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   544
14328
9510671ea784 Fixes for breakpoint management when editing the code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14326
diff changeset
   545
fixupBreakpointPositions
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   546
    "computes the character positions of all line breakpoints"
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   547
    
19019
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   548
    breakpoints notEmptyOrNil ifTrue:[
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   549
        "/ self assert:(breakpoints contains:[:b | b line < 0]) not.
14450
60b7e5f93d5a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14328
diff changeset
   550
        breakpoints do:[:each |  
60b7e5f93d5a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14328
diff changeset
   551
            | pos |
14328
9510671ea784 Fixes for breakpoint management when editing the code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14326
diff changeset
   552
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   553
            pos := textView characterPositionOfLine:(each line) col:1.
18828
89ad6f8245e2 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 18664
diff changeset
   554
            each position:pos line:(each line).  
14450
60b7e5f93d5a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14328
diff changeset
   555
        ].
19019
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   556
        "/ self assert:(breakpoints contains:[:b | b line < 0]) not.
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   557
        breakpoints := breakpoints select:[:b | b line >= 0].
19019
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   558
        self sortBreakpoints.
14328
9510671ea784 Fixes for breakpoint management when editing the code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14326
diff changeset
   559
   ].
9510671ea784 Fixes for breakpoint management when editing the code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14326
diff changeset
   560
9510671ea784 Fixes for breakpoint management when editing the code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14326
diff changeset
   561
    "Created: / 08-05-2014 / 14:02:25 / Jan Vrany <jan.vrany@fit.cvut.cz>"
17366
de7bda272450 #BUGFIX by cg
Claus Gittinger <cg@exept.de>
parents: 17158
diff changeset
   562
    "Modified: / 07-02-2017 / 23:12:06 / cg"
18828
89ad6f8245e2 #FEATURE by cg
Claus Gittinger <cg@exept.de>
parents: 18664
diff changeset
   563
    "Modified: / 21-06-2019 / 07:47:19 / Claus Gittinger"
14328
9510671ea784 Fixes for breakpoint management when editing the code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14326
diff changeset
   564
!
9510671ea784 Fixes for breakpoint management when editing the code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14326
diff changeset
   565
10226
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   566
moveBreakpointsAfterLine:line by: delta
19019
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   567
    "/ self assert:(breakpoints contains:[:b | b line < 0]) not.
10226
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   568
    breakpoints do:[:bpnt|
11719
f4c9090f21af changed:
Claus Gittinger <cg@exept.de>
parents: 11607
diff changeset
   569
        bpnt line >= line ifTrue:[
14328
9510671ea784 Fixes for breakpoint management when editing the code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14326
diff changeset
   570
            "/ Note that position will be fixed up in BreakpointService>>breakpoints
9510671ea784 Fixes for breakpoint management when editing the code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14326
diff changeset
   571
            bpnt position:nil line:(bpnt line + delta). 
11719
f4c9090f21af changed:
Claus Gittinger <cg@exept.de>
parents: 11607
diff changeset
   572
        ]
10226
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   573
    ].
19019
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   574
    "/ self assert:(breakpoints contains:[:b | b line < 0]) not.
14454
fdaebd3224b8 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 14450
diff changeset
   575
    breakpoints := breakpoints reject:[:bpnt | bpnt line <= 0].
10226
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   576
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   577
    "/gutterView redrawLinesFrom: line.
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   578
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   579
    "Created: / 17-06-2011 / 13:45:22 / Jan Vrany <jan.vrany@fit.cvut.cz>"
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   580
    "Created: / 06-07-2011 / 17:26:30 / jv"
14328
9510671ea784 Fixes for breakpoint management when editing the code.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 14326
diff changeset
   581
    "Modified: / 08-05-2014 / 14:01:17 / Jan Vrany <jan.vrany@fit.cvut.cz>"
17092
f2ba8e0ae8cb #BUGFIX by cg
Claus Gittinger <cg@exept.de>
parents: 17072
diff changeset
   582
    "Modified: / 20-11-2016 / 19:09:12 / cg"
10226
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   583
!
b647aff39f33 More improvements in BreakpointService
vrany
parents: 10209
diff changeset
   584
10182
9ce79271722f added: #recompile
Claus Gittinger <cg@exept.de>
parents: 10071
diff changeset
   585
recompile
9ce79271722f added: #recompile
Claus Gittinger <cg@exept.de>
parents: 10071
diff changeset
   586
    "recompile the current method for changed breakpoints"
9ce79271722f added: #recompile
Claus Gittinger <cg@exept.de>
parents: 10071
diff changeset
   587
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   588
    |oldMethod|
10182
9ce79271722f added: #recompile
Claus Gittinger <cg@exept.de>
parents: 10071
diff changeset
   589
13201
3b6f308a7280 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13191
diff changeset
   590
    oldMethod := codeView method.
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   591
    (oldMethod isNil or:[oldMethod hasPrimitiveCode]) ifTrue:[^ self].
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   592
    "/ be careful: if the text has been edited/modified, do not compile
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   593
    textView modified ifTrue:[
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   594
        self breakPoint: #cg.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   595
        self breakPoint: #jv.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   596
        ^self.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   597
    ].
10182
9ce79271722f added: #recompile
Claus Gittinger <cg@exept.de>
parents: 10071
diff changeset
   598
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   599
    self recompileMethod:oldMethod.
10182
9ce79271722f added: #recompile
Claus Gittinger <cg@exept.de>
parents: 10071
diff changeset
   600
9ce79271722f added: #recompile
Claus Gittinger <cg@exept.de>
parents: 10071
diff changeset
   601
    "Created: / 05-07-2011 / 21:33:13 / cg"
15383
671638959b16 Use breakpointsDo: instead of enumerating over literals.
Jan Vrany <jan.vrany@fit.cvut.cz>
parents: 15303
diff changeset
   602
    "Modified: / 23-02-2015 / 14:49:04 / Jan Vrany <jan.vrany@fit.cvut.cz>"
17777
3a3e57e36d2c #DOCUMENTATION by cg
Claus Gittinger <cg@exept.de>
parents: 17620
diff changeset
   603
    "Modified: / 21-11-2017 / 13:09:36 / cg"
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   604
    "Modified: / 20-02-2019 / 17:26:57 / Claus Gittinger"
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   605
!
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   607
recompileMethod:oldMethod
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   608
    "recompile oldMethod for changed breakpoints"
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   609
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   610
    | newMethod compilerClass compiler class selector|
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   611
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   612
    "/ prepare to get reachable bpts
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   613
    self fixupBreakpointPositions.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   614
    breakpoints do:[:bp | bp isReached:false].
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   615
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   616
    class := oldMethod mclass.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   617
    class isNil ifTrue:[
19194
c1c0ae2cb0ab #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19104
diff changeset
   618
        class := oldMethod getMclass.
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   619
        class isNil ifTrue:[
19194
c1c0ae2cb0ab #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19104
diff changeset
   620
            class := oldMethod originalMethodIfWrapped mclass.
c1c0ae2cb0ab #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19104
diff changeset
   621
            class isNil ifTrue:[
c1c0ae2cb0ab #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19104
diff changeset
   622
                class := oldMethod originalMethodIfWrapped getMclass.
c1c0ae2cb0ab #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19104
diff changeset
   623
                class isNil ifTrue:[
c1c0ae2cb0ab #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19104
diff changeset
   624
                    class := codeView classHolder value.
c1c0ae2cb0ab #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19104
diff changeset
   625
                    class isNil ifTrue:[
c1c0ae2cb0ab #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19104
diff changeset
   626
                        self breakPoint:#jv.
c1c0ae2cb0ab #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19104
diff changeset
   627
                        Dialog warn:'oops - lost the method''s class'.
c1c0ae2cb0ab #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19104
diff changeset
   628
                        ^ self.
c1c0ae2cb0ab #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19104
diff changeset
   629
                    ]
c1c0ae2cb0ab #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19104
diff changeset
   630
                ]
c1c0ae2cb0ab #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19104
diff changeset
   631
            ]
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   632
        ]
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   633
    ].
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   634
    selector := oldMethod selector.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   635
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   636
    compilerClass := oldMethod programmingLanguage compilerWithBreakpointSupportClass.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   637
    compilerClass isNil ifTrue:[
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   638
        Dialog warn:'No breakpoint support for this programming language'.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   639
        ^ self.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   640
    ].
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   641
    compiler := compilerClass new.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   642
    compiler breakpoints:breakpoints.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   643
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   644
    Class withoutUpdatingChangesDo:[
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   645
        newMethod := compiler
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   646
                    compile:oldMethod source
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   647
                    forClass:class
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   648
                    inCategory:oldMethod category
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   649
                    notifying:nil
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   650
                    install:false
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   651
                    skipIfSame:false
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   652
                    silent:true
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   653
                    foldConstants:true
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   654
                    ifFail:[ Transcript showCR:'BreakpointService: failed to recompile for breakpoint' ].
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   655
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   656
        selector isNil ifTrue:[
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   657
            "/ May happen as the selector is not stored in the method but
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   658
            "/ searches through method's mclass methodDictionary.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   659
            "/ Following should be save as breakpoint is not installed when
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   660
            "/ the code is modified...
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   661
            selector := compiler selector.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   662
        ].
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   663
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   664
        oldMethod isWrapped ifTrue:[
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   665
            "/ update the wrapped method - do not install
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   666
            newMethod originalMethod: oldMethod originalMethod.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   667
            oldMethod replaceOriginalMethodWith:newMethod.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   668
        ] ifFalse:[
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   669
            "/ install
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   670
            newMethod originalMethod: oldMethod.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   671
            (class primAddSelector: selector withMethod:newMethod) ifFalse:[
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   672
                oldMethod mclass:class.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   673
                self breakPoint: #cg.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   674
                self breakPoint: #jv.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   675
                ^ self
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   676
            ].
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   677
        ].
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   678
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   679
        breakpoints := breakpoints species new.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   680
        newMethod breakpointsDo:[ :each | each method:newMethod. breakpoints add: each ].
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   681
        "/ this assertion occasionally failed - and lead to wrong breakpoint handling in the compiler.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   682
        "/ for now, as a q&d hack, sort them to make sure.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   683
        "/ self assert:(breakpoints isSortedBy:[:a :b |a position <= b position]).
19019
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   684
        self sortBreakpoints.
18606
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   685
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   686
        "/ must update breakpoints BEFORE the following, because it leads to a change
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   687
        "/ notification, which may clear the breakpoints collection!!
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   688
        self fetchMethodHolder value:newMethod.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   689
        oldMethod mclass isNil ifTrue:[
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   690
            "/ although this is not strictly true, not doing this
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   691
            "/ would confuse a lot of other tools (such as the browser)
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   692
            oldMethod mclass:class.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   693
        ].
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   694
        currentMethod := newMethod.
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   695
        class changed:#methodTrap with:selector. "/ tell browsers
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   696
        Smalltalk changed:#methodTrap with:(MethodTrapChangeNotificationParameter changeClass:class changeSelector:selector).
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   697
    ].
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   698
42261a0570c0 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18599
diff changeset
   699
    "Created: / 20-02-2019 / 17:26:24 / Claus Gittinger"
19019
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   700
!
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   701
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   702
sortBreakpoints
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   703
    "breakpoints are sorted by line 
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   704
     (or even by position within a line, but those are currently not supported by the GUI)"
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   705
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   706
    breakpoints := breakpoints 
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   707
                    asOrderedCollection 
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   708
                        sort:[:a :b|
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   709
                            |pA pB|
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   710
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   711
                            pA := a position.
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   712
                            pB := b position.
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   713
                            (pA notNil and:[pB notNil]) ifTrue:[
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   714
                                "/ self assert:(pA <= pB) = (a line <= b line).
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   715
                                pA < pB
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   716
                            ] ifFalse:[
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   717
                                a line < b line
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   718
                            ]    
88062aff617b #BUGFIX by exept
Claus Gittinger <cg@exept.de>
parents: 18844
diff changeset
   719
                        ].
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   720
! !
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   721
12902
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   722
!BreakpointService methodsFor:'queries'!
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   723
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   724
canCreateOrToggleBreakpointAtLine:lineOrNilForAnywhere
13420
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   725
    "is it possible to place a breakpoint here and now?"
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   726
13126
6c0fc3e2986a class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13106
diff changeset
   727
    |bpnt topView|
12902
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   728
19332
5d98903bd2f8 #BUGFIX by cg
Claus Gittinger <cg@exept.de>
parents: 19331
diff changeset
   729
    textView isNil ifTrue:[
5d98903bd2f8 #BUGFIX by cg
Claus Gittinger <cg@exept.de>
parents: 19331
diff changeset
   730
        "/ happens due to a bug, after disabling the service;
5d98903bd2f8 #BUGFIX by cg
Claus Gittinger <cg@exept.de>
parents: 19331
diff changeset
   731
        "/ it seems to still be asked 
5d98903bd2f8 #BUGFIX by cg
Claus Gittinger <cg@exept.de>
parents: 19331
diff changeset
   732
        ^ false
5d98903bd2f8 #BUGFIX by cg
Claus Gittinger <cg@exept.de>
parents: 19331
diff changeset
   733
    ].
5d98903bd2f8 #BUGFIX by cg
Claus Gittinger <cg@exept.de>
parents: 19331
diff changeset
   734
12902
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   735
    textView reallyModified ifTrue:[
13420
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   736
        "/ 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
   737
        "/ are while editing and shift them as required.
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   738
        "/ (another idea worth a try would be
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   739
        "/ to match the original parsetree (enumerating nodes with the breakpoints)
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   740
        "/ against the new parsetree (walking in sync?) when finally compiling,
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   741
        "/  and placing new breakpoints on matching tree nodes. 
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   742
        "/ (too much work, for a quick solution, I guess)
12902
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   743
        ^ false
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   744
    ].
13420
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   745
12902
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   746
    "/ can always toggle existing breakpoints...
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   747
    lineOrNilForAnywhere notNil ifTrue:[
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   748
        bpnt := self breakpointAtLine:lineOrNilForAnywhere.
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   749
        bpnt notNil ifTrue:[
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   750
            ^ true.
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   751
        ]
13420
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   752
    ] ifFalse:[
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   753
        (currentMethod notNil and:[currentMethod isMethodWithBreakpoints]) ifTrue:[
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   754
            ^ true.
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   755
        ]
12902
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   756
    ].
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   757
13420
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   758
    "/ ok, the method has no breakpoints yet.
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   759
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   760
    "/ 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
   761
    "/ I guess, we have to move code around a bit...
12902
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   762
    ((topView := codeView topView) class == DebugView) ifTrue:[
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   763
        "/ can only create new breakpoints in the debugger,
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   764
        "/ iff we are in a wrapped method's prolog
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   765
        topView selectedContextIsWrapped ifTrue:[
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   766
            topView selectedContext lineNumber == 1 ifTrue:[
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   767
                ^ true
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   768
            ].
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   769
        ].
13420
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   770
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   771
        "/ well, if the debugger's code has already been modified,
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   772
        "/ 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
   773
        "/ a breakpoint on the fly...
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   774
        topView showingAlreadyModifiedCode ifTrue:[^ true].
12902
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   775
        ^ false.
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   776
    ].
13420
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   777
3f8f012f121b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13395
diff changeset
   778
    "/ in a non-debugger, we can do it.
12902
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   779
    ^  true.
18844
2556c2dcb567 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18828
diff changeset
   780
2556c2dcb567 #REFACTORING by cg
Claus Gittinger <cg@exept.de>
parents: 18828
diff changeset
   781
    "Modified: / 26-06-2019 / 19:52:07 / Claus Gittinger"
12902
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   782
!
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   783
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   784
hasBreakpoints
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   785
    ^ breakpoints notEmptyOrNil
19097
b9bee641194f #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19071
diff changeset
   786
!
b9bee641194f #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19071
diff changeset
   787
b9bee641194f #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19071
diff changeset
   788
singleClickToToggle
b9bee641194f #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19071
diff changeset
   789
    "/ not yet working correctly: must ignore double clicks then,
b9bee641194f #FEATURE by exept
Claus Gittinger <cg@exept.de>
parents: 19071
diff changeset
   790
    "/ but only some...
19330
2d85a45f2ad2 #UI_ENHANCEMENT by cg
Claus Gittinger <cg@exept.de>
parents: 19287
diff changeset
   791
    ^ (NeedDoubleClickToToggleBreakpoints ? false) not
12902
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   792
! !
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   793
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   794
!BreakpointService methodsFor:'redrawing'!
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   795
16602
c8023736aa45 #OTHER by cg
Claus Gittinger <cg@exept.de>
parents: 16334
diff changeset
   796
drawLine:lineNo in:view atX:x y:yBaseline width:w height:hFont ascent:aFont
c8023736aa45 #OTHER by cg
Claus Gittinger <cg@exept.de>
parents: 16334
diff changeset
   797
    from:startCol to:endColOrNil with:fg and:bg
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   798
    "Called by both gutterView and textView (well, not yet) to
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   799
     allow services to draw custom things on text view.
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   800
     Ask JV what the args means if unsure (I'm lazy to document
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   801
     them, now it is just an experiment...)"
11607
3cd59e465a6a *** empty log message ***
Claus Gittinger <cg@exept.de>
parents: 11601
diff changeset
   802
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   803
    |mthd bpnt icon dx dy|
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   804
16602
c8023736aa45 #OTHER by cg
Claus Gittinger <cg@exept.de>
parents: 16334
diff changeset
   805
    "/ these tests make the breakpointService unusable for other applications 
c8023736aa45 #OTHER by cg
Claus Gittinger <cg@exept.de>
parents: 16334
diff changeset
   806
    "/ (which are not based on smalltalk methods). 
c8023736aa45 #OTHER by cg
Claus Gittinger <cg@exept.de>
parents: 16334
diff changeset
   807
    "/ They are not really needed: if there is a breakpoint,
13572
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   808
    "/ I can show it. Period.
13191
9745eed50f03 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13157
diff changeset
   809
13573
70135af4ddeb class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13572
diff changeset
   810
    "/    (mthd := self fetchMethodHolder value) isNil ifTrue:[
13572
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   811
    "/        ^ self
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   812
    "/    ].
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   813
    "/    currentMethodClass isNil ifTrue:[
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   814
    "/        "/ hack: ouch - was wrapped in the meantime;
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   815
    "/        ^ self. "/ wait for the real update
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   816
    "/        "/ hurry up and update. Should be done elsewhere (in codeView)
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   817
    "/        "/ self updateCurrentMethod.
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   818
    "/    ].
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   819
13572
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   820
    view == gutterView ifTrue:[
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   821
        bpnt := self breakpointAtLine:lineNo.
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   822
        bpnt isNil ifTrue:[
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   823
            ^ self
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   824
        ].
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   825
        icon := bpnt icon.
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   826
        icon isNil ifTrue:[
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   827
            ^ self
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   828
        ].
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   829
        dx := ((w - icon width) / 2) rounded.
16602
c8023736aa45 #OTHER by cg
Claus Gittinger <cg@exept.de>
parents: 16334
diff changeset
   830
        dy := ((hFont - icon height) / 2) rounded.
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   831
        icon
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   832
            displayOn:view
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   833
            x:x + dx
16602
c8023736aa45 #OTHER by cg
Claus Gittinger <cg@exept.de>
parents: 16334
diff changeset
   834
            y:yBaseline - hFont + dy + 4. "TODO: Magic constant"
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   835
    ].
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   836
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   837
    "Created: / 17-06-2011 / 13:52:52 / Jan Vrany <jan.vrany@fit.cvut.cz>"
10182
9ce79271722f added: #recompile
Claus Gittinger <cg@exept.de>
parents: 10071
diff changeset
   838
    "Modified (format): / 05-07-2011 / 22:14:33 / cg"
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   839
! !
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   840
13572
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   841
!BreakpointService methodsFor:'registering'!
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   842
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   843
registerIn:aCodeView
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   844
    super registerIn:aCodeView.
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   845
    self fetchMethodHolder.
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   846
! !
7b7d06d0d564 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 13420
diff changeset
   847
12902
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   848
!BreakpointService methodsFor:'testing'!
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   849
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   850
isBreakpointService
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   851
    ^ true
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   852
! !
724e09dfd9a2 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12855
diff changeset
   853
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   854
!BreakpointService class methodsFor:'documentation'!
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   855
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   856
version
15983
Claus Gittinger <cg@exept.de>
parents: 15642
diff changeset
   857
    ^ '$Header$'
12949
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   858
!
320a22d3c70b class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 12940
diff changeset
   859
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   860
version_CVS
15983
Claus Gittinger <cg@exept.de>
parents: 15642
diff changeset
   861
    ^ '$Header$'
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   862
!
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   863
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   864
version_SVN
15983
Claus Gittinger <cg@exept.de>
parents: 15642
diff changeset
   865
    ^ '$Id$'
9982
527ab4879a77 initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   866
! !
12507
3fa9ae668d96 class: Tools::BreakpointService
Claus Gittinger <cg@exept.de>
parents: 11987
diff changeset
   867