delve/service/test
Alessandro Arzilli d97b471292
pkg/proc,service/debugger: do not disable unsatisfiable breakpoints (#3868)
Previously breakpoints with hitcount conditions that became
unsatisfiable
would become disabled, this was done as an optimization so that the
continue loop would no longer need to stop on them and evaluate their
conditions.
As a side effect this meant that on restart these breakpoints would
remain disabled, even though their hit condition returned satisfiable.

This commit changes Delve behavior so that breakpoints with
unsatisifiable hitcount conditions are no longer disabled but the
associated physical breakpoints are removed anyway, preserving the
optimization.

Some refactoring is done to the way conditions are represented and the
enable status is managed so that in the future it will be possible to
use hitcount conditions to implement "chained" breakpoints (also known
as dependet breakpoints), i.e. breakpoints that become active only
after a second breakpoint has been hit.
2024-12-04 19:07:56 -08:00
..
common_test.go terminal: restore breakpoints set with line offset on restart (#3425) 2023-07-20 12:29:59 +02:00
integration1_test.go *: remove redundant lines at the start/end of block (#3730) 2024-05-20 14:16:22 -07:00
integration2_test.go pkg/proc,service/debugger: do not disable unsatisfiable breakpoints (#3868) 2024-12-04 19:07:56 -08:00