[Lldb-commits] [PATCH] D103349: [lldb] Don't print script output twice in HandleCommand

Shafik Yaghmour via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Jun 1 11:43:45 PDT 2021

shafik added inline comments.

Comment at: lldb/include/lldb/Interpreter/CommandReturnObject.h:169
+  /// as a temporary.
+  bool m_hermetic;
JDevlieghere wrote:
> aprantl wrote:
> > bool m_hermetic = false;
> This is the second time someone has suggested to use in-class member initializers. It's not what we've been doing in LLDB so far, but I am a fan of it personally, so I'm going to interpret this as what we want to do moving forward. I believe it's even a C++ core guideline. Maybe we can have a clang tidy check to enforce this. 
This is idiomatic C++ and we should be using this feature going forward. It is more readable then a crowded `mem-init-list` and really makes a big difference when there are multiple constructors. 



More information about the lldb-commits mailing list