[Lldb-commits] [PATCH] D123954: [lldb] Add setting for max depth of value object printing (NFC)

Dave Lee via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Apr 28 09:49:57 PDT 2022


kastiglione added inline comments.


================
Comment at: lldb/include/lldb/Interpreter/CommandInterpreter.h:718
+  /// been told.
+  ChildrenOmissionWarningStatus m_truncation_warning;
+  /// Whether we reached the maximum child nesting depth and whether the user
----------------
kastiglione wrote:
> shafik wrote:
> > Why not use in class member initialization? It looks like they have default values. I am not sure why the rest of the values where not caught the other day when I ran clang-tidy.
> I was following the convention within the file. What's the ideal change, initialize all in this change? Or initialize the two that I have edited?
I think it's best to leave these as is, and make a follow up change that uses member initialization for all of these.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123954



More information about the lldb-commits mailing list