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

[Martin Svensson via Phabricator via lldb-commits]

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':



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':

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits