[Lldb-commits] [PATCH] D88513: [lldb-vscode] Allow an empty 'breakpoints' field to clear breakpoints.

2020-09-30 Thread Jordan Rupprecht via Phabricator via lldb-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. rupprecht marked an inline comment as done. Closed by commit rGad865d9d10b8: [lldb-vscode] Allow an empty breakpoints field to clear breakpoints. (authored by

[Lldb-commits] [PATCH] D88513: [lldb-vscode] Allow an empty 'breakpoints' field to clear breakpoints.

2020-09-29 Thread walter erquinigo via Phabricator via lldb-commits
wallace accepted this revision. wallace added a comment. This revision is now accepted and ready to land. Very good! Thank you Comment at: lldb/test/API/tools/lldb-vscode/breakpoint/TestVSCode_setBreakpoints.py:228 + empty one.''' +lines =

[Lldb-commits] [PATCH] D88513: [lldb-vscode] Allow an empty 'breakpoints' field to clear breakpoints.

2020-09-29 Thread Jordan Rupprecht via Phabricator via lldb-commits
rupprecht created this revision. rupprecht added reviewers: wallace, clayborg, labath. Herald added a reviewer: JDevlieghere. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. rupprecht requested review of this revision. Per the DAP spec for SetBreakpoints [1], the way to