[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