[Lldb-commits] [PATCH] D110298: Add "command multiword add" and the ability to add script commands into a user multiword hierarchy

Greg Clayton via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Sep 27 21:17:59 PDT 2021


clayborg added a comment.

The changes look fine from a quick read. See inline comments



================
Comment at: lldb/source/Commands/CommandObjectCommands.cpp:1817
+      : CommandObjectParsed(
+            interpreter, "command multiword add",
+            "Add a multiword command to lldb.  Adding to built-"
----------------
can't this just be "command add" instead of "command multiword add"? 


================
Comment at: lldb/source/Commands/CommandObjectCommands.cpp:1959
+      : CommandObjectParsed(
+            interpreter, "command multiword delete",
+            "Delete a multiword command previously added to "
----------------
Could we just modify the existing "command delete" instead of adding a new command here?


================
Comment at: lldb/source/Commands/CommandObjectCommands.cpp:2091-2092
                  CommandObjectSP(new CommandObjectCommandsDelete(interpreter)));
+  LoadSubCommand("multiword", CommandObjectSP(new CommandObjectCommandMultiword(
+                                  interpreter)));
   LoadSubCommand(
----------------
"multiword" makes sense to us LLDB software folks, not sure it will resonate with general public. See comments above about maybe doing "command add" and then modifying "command delete" to also handle multiword duties


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110298/new/

https://reviews.llvm.org/D110298



More information about the lldb-commits mailing list