[Lldb-commits] [PATCH] D144230: [lldb] Make persisting result variables configurable

[Adrian Prantl via Phabricator via lldb-commits]

aprantl added inline comments.


================
Comment at: lldb/source/Commands/CommandObjectExpression.cpp:149
 
+  case 'C': {
+    // 'C' for "caching", since both 'P' and 'p' for persist are taken. Both 'R'
----------------
Do we expect people to actually use this flag or to just use an alias that has this turned on? If the latter, maybe we should only provide a long option, since the single-letter name space is precious?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144230