[Lldb-commits] [PATCH] D122680: Add a setting to force overwriting commands in "command script add"

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Mar 29 12:22:44 PDT 2022


jingham created this revision.
jingham added a reviewer: JDevlieghere.
Herald added a project: All.
jingham requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.

When I added containers it seemed like it would be easier to accidentally overwrite extant commands, so I required a --overwrite option before "command script add" will overwrite an extant command.  That still strikes me as a good idea for users of packages of commands, but when you are developing the commands it's annoying since it means you have to change the "command script add" commands in your package while iteratively adding them during development.  So this patch adds a setting to turn off the overwrite requirement.  That way you can set this setting while developing your package, then turn it off again when you're acting as a consumer of packages.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D122680

Files:
  lldb/include/lldb/Interpreter/CommandInterpreter.h
  lldb/source/Commands/CommandObjectCommands.cpp
  lldb/source/Interpreter/CommandInterpreter.cpp
  lldb/source/Interpreter/InterpreterProperties.td
  lldb/test/API/commands/command/container/TestContainerCommands.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D122680.418949.patch
Type: text/x-patch
Size: 6265 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20220329/520f8e0a/attachment.bin>


More information about the lldb-commits mailing list