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