shafik 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
----------------
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.


================
Comment at: lldb/source/Interpreter/OptionGroupValueObjectDisplay.cpp:180
     use_dynamic = target_sp->GetPreferDynamicValue();
-  else {
+    std::tie(max_depth, max_depth_is_default) =
+        target_sp->GetMaximumDepthOfChildrenToDisplay();
----------------
My kingdom for structured bindings.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123954

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to