diff options
| author | Ben Jackson <puremourning@users.noreply.github.com> | 2025-02-03 22:39:43 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-02-03 14:39:43 -0800 |
| commit | c26bb4f095120ff4741b6530b6b3c0ac5a7dacad (patch) | |
| tree | 33b65727e9fcfec126d2a5958f64e9ec4ec689d0 /lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp | |
| parent | f8fa93193b664c33c9227073b2bd123842e05ffc (diff) | |
| download | llvm-c26bb4f095120ff4741b6530b6b3c0ac5a7dacad.zip llvm-c26bb4f095120ff4741b6530b6b3c0ac5a7dacad.tar.gz llvm-c26bb4f095120ff4741b6530b6b3c0ac5a7dacad.tar.bz2 | |
[lldb] correct event when removing all watchpoints (#125312)
LLDB: correct event when removing all watchpoints
Previously we incorrectly checked for a "breakpoint changed" event
listener removing all watchpoints (e.g. via
SBTarget::DeleteAllWatchpoints()), although we would emit a "watchpoint
changed" event if there were a listener for 'breakpoint changed'.
This meant that we might not emit a "watchpoint changed" event if there
was a listener for this event.
Correct it to check for the "watchpoint changed" event.
---
Updated regression tests which were also incorrectly peeking for the
wrong event type. The 'remove' action actually triggers 2 events which
the test didn't allow, so I updated it to allow specifically what was
requested.
The test fails (expectedly) at the line following "DeleteAllWatchpoints"
prior to this patch, and passes after.
Diffstat (limited to 'lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp')
0 files changed, 0 insertions, 0 deletions