[Lldb-commits] [PATCH] D44081: [lldb] Add synthetic frontend for _NSCallStackArray

[Kuba (Brecka) Mracek via Phabricator via lldb-commits]

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


================
Comment at: source/Plugins/Language/ObjC/NSArray.cpp:446-451
+  } else if (class_name == g_NSCallStackArray) {
+    Status error;
+    value = process_sp->ReadUnsignedIntegerFromMemory(
+        valobj_addr + 2 * ptr_size, ptr_size, 0, error);
+    if (error.Fail())
+      return false;
----------------
davide wrote:
> Can you add a comment here?
> this also seems exactly the same as `NSArrayCF`, any chance we can avoid the duplication?
Added comment and de-duplicated the branch.


https://reviews.llvm.org/D44081