================
@@ -2076,7 +2077,11 @@ bool CommandInterpreter::HandleCommand(const char 
*command_line,
     }
 
     ElapsedTime elapsed(execute_time);
-    cmd_obj->Execute(remainder.c_str(), result);
+    size_t nchar = real_original_command_string.find(remainder);
+    std::optional<uint16_t> pos_in_cmd;
+    if (nchar != std::string::npos)
+      pos_in_cmd = nchar + GetDebugger().GetPrompt().size();
+    cmd_obj->Execute(remainder.c_str(), pos_in_cmd, result);
----------------
medismailben wrote:

Instead of updating every `DoExecute` implementations, it would be reasonable 
to have make this an optional member of the `CommandObject` base class so it 
can be accessed by the derived `CommandObject` classes that gonna use it.


The optional could be set in the non-virtual `CommandObject::Execute` command, 
since that's the one that calls the derived class `DoExecute` or may be by 
adding a setter to the `CommandObject` base class. 

This would greatly reduce the size of this patch.

https://github.com/llvm/llvm-project/pull/106470
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to