[Lldb-commits] [PATCH] Add a --all command option to "target delete"

[Jim Ingham]

I think it is worth mentioning in the help that if you do "clean" then you will need to reparse all shared modules the next time you debug.  This can be a significant hiccup when you next debug...  Can't think of a succinct way to say this off the top of my head, though...

I try to use boolean values only when it isn't clear what the default value is (for instance if it is also controlled by a setting, or maybe situationally determined.)  That's not true for --all  or --clean so it is fine to have them be argument-less flags.

And while I would be concerned about changing commonly used CLI options since that will make aliases to fail, these are not common options, so it is okay to clean up the clean option.


http://reviews.llvm.org/D8615

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/