labath wrote: I don't know about "discoverable", but I think it's more logical and fits it with the existing "noun verb" command naming scheme. You say you want to "show" "settings", and then you follow that with some modifiers/restrictions on that. Currently, the only restriction is via the setting path name (in a non-option argument), but I think an option would fit nicely there as well. Maybe that makes it more discoverable? I wouldn't have a problem finding it there...
The top level "settings" command doesn't have that many subcommands, so adding one more is not a big deal, but I also think that adding a flag makes the result more future-proof. Now, if you want to change the way that settings are rendered in this mode (e.g. by including the default value), then a separate command may make more sense. Although, for default values specifically, I can also imagine this as being controlled by another command line flag (`--verbose` or something). And then you can combine the two however you see fit... https://github.com/llvm/llvm-project/pull/152338 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
