kastiglione 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' ---------------- aprantl wrote: > 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? Initially I was going to have only the long option, but I reconsidered because I don't know how frequently persistent results are used by users. In the future, if/when we make the persistent results opt-in (instead of opt-out), will people want to use persistent results sporadically? If so maybe they'd want a short flag to quickly opt-in. > since the single-letter name space is precious this is a compelling argument. I'll go with long option only to start. It's easier to add a short option later, than to remove it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144230/new/ https://reviews.llvm.org/D144230 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits