[Lldb-commits] [PATCH] D69425: [lldb] Fix broken -D option for breakpoint set command

Martin Svensson via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Oct 25 02:47:03 PDT 2019


poya created this revision.
poya added a reviewer: jingham.
Herald added subscribers: lldb-commits, JDevlieghere.
Herald added a project: LLDB.

BreakpointDummyOptionGroup was using //g_breakpoint_modify_options// rather than //g_breakpoint_dummy_options// causing the -D option for //breakpoint set// command to be incorrectly parsed.


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D69425

Files:
  lldb/source/Commands/CommandObjectBreakpoint.cpp


Index: lldb/source/Commands/CommandObjectBreakpoint.cpp
===================================================================
--- lldb/source/Commands/CommandObjectBreakpoint.cpp
+++ lldb/source/Commands/CommandObjectBreakpoint.cpp
@@ -198,7 +198,7 @@
   Status SetOptionValue(uint32_t option_idx, llvm::StringRef option_arg,
                           ExecutionContext *execution_context) override {
     Status error;
-    const int short_option = g_breakpoint_modify_options[option_idx].short_option;
+    const int short_option = g_breakpoint_dummy_options[option_idx].short_option;
 
     switch (short_option) {
       case 'D':


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69425.226387.patch
Type: text/x-patch
Size: 633 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20191025/574df0e6/attachment.bin>


More information about the lldb-commits mailing list