[[ Bug 22137 ]] Fix conditional breakpoint evaluation

montegoulding · montegoulding · commit db8f27307da0 · 2019-06-03T14:32:32.000+10:00
This patch ensures that conditional breakpoints that evaluate to false are reevaluated
if the same breakpoint is encountered again. For example, if a conditional breakpoint
is in a repeat loop it should be evaluated each iteration. A previous patch indended to
ensure that errors encountered while debugging a previous event were not re-entered
caused conditional breakpoints to not re-evaluate each iteration if no other breakpoint
was encountered.
diff --git a/Toolset/libraries/revdebuggerlibrary.livecodescript b/Toolset/libraries/revdebuggerlibrary.livecodescript
@@ -2455,6 +2455,7 @@ on traceBreak pHandler, pLine
       set the traceStack to the short name of __TargetStack(the long id of the target)
       __UpdateScriptEditor tTarget, pHandler, pLine, "debug"
    else
+      delete variable sLastBreakInfo
       set the traceReturn to true
    end if
 end traceBreak
diff --git a/notes/bugfix-22137.md b/notes/bugfix-22137.md
@@ -0,0 +1 @@
+# Fix issue causing conditional breakpoints in repeat loops to only be evaluated once

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+# Fix issue causing conditional breakpoints in repeat loops to only be evaluated once`