Skip to content

Commit

Permalink
Debugger now requires to define all variables when compiling code fro…
Browse files Browse the repository at this point in the history
…m the debugger code pane.
  • Loading branch information
StevenCostiou committed Aug 12, 2020
1 parent d531da6 commit 721b5fe
Showing 1 changed file with 10 additions and 8 deletions.
18 changes: 10 additions & 8 deletions src/NewTools-Debugger/StDebugger.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -683,15 +683,16 @@ StDebugger >> initializeBreakpointInspector [

{ #category : #initialization }
StDebugger >> initializeCode [
self flag: #DBG_IMPROVE.
"Add metalink information in the gutter!"

self flag: #DBG.
"It is wrong here that I have to use code adapter widget.
It is expected down in the compilation call chain that the notified requestor answers to textMorph.
This is for modifying the source code when declaring temps"
code := self newCode.
code whenSubmitDo: [ :text |
self
recompileMethodTo: text string
inContext: self stackTable selection selectedItem
notifying: nil ].
notifying: code adapter widget ].
code whenResetDo: [ self updateCodeFromContext ]
]

Expand Down Expand Up @@ -863,10 +864,11 @@ StDebugger >> proceedDebugSession [
{ #category : #actions }
StDebugger >> recompileMethodTo: aString inContext: aContext notifying: aNotifyer [

^ aContext
ifNil: [ false ]
ifNotNil: [
self session recompileMethodTo: aString inContext: aContext notifying: aNotifyer ]
aContext ifNil: [ ^ self ].
self session
recompileMethodTo: aString
inContext: aContext
notifying: aNotifyer
]

{ #category : #'updating session' }
Expand Down

0 comments on commit 721b5fe

Please sign in to comment.