[Lldb-commits] [lldb] Add the ability to define a Python based command that uses CommandObjectParsed (PR #70734)

Med Ismail Bennani via lldb-commits lldb-commits at lists.llvm.org
Sat Feb 10 22:32:38 PST 2024


================
@@ -805,19 +805,25 @@ let Command = "script add" in {
   def script_add_function : Option<"function", "f">, Group<1>,
     Arg<"PythonFunction">,
     Desc<"Name of the Python function to bind to this command name.">;
-  def script_add_class : Option<"class", "c">, Group<2>, Arg<"PythonClass">,
-  Desc<"Name of the Python class to bind to this command name.">;
+  def script_add_class : Option<"class", "c">, Groups<[2,3]>, 
----------------
medismailben wrote:

Let's swap the short-option between `script_add_class` & `script_add_completion_type`. Scripted Process uses `-C` (capital C) in `process launch` & `process attach` to specify the python class. Same for scripted thread plans with `thread step-scripted`.

Let's try to stay consistent for our users.

https://github.com/llvm/llvm-project/pull/70734


More information about the lldb-commits mailing list