[Lldb-commits] [PATCH] D129490: [lldb/libc++] Simplify the libc++ string formatter

[Pavel Labath via Phabricator via lldb-commits]

labath marked an inline comment as done.
labath added inline comments.


================
Comment at: lldb/source/Plugins/Language/CPlusPlus/LibCxx.cpp:609
   } else {
-    ValueObjectSP size_mode(dataval_sp->GetChildAtIndexPath({1, 0, 0}));
-    if (!size_mode)
+    if (ValueObjectSP size_mode = short_sp->GetChildMemberWithName(
+            ConstString("__size_"), /*can_create=*/true)) {
----------------
mib wrote:
> Nit: In the if statement it's called `size_mode` but it the other branch (line `603`) it's called `size_member`. Would be nice to make it more consistent.
Done. I've also moved the common statement before the branch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129490