[Lldb-commits] [PATCH] D127481: [LLDB][formatters] Add formatter for libc++'s std::span

[Adrian Prantl via Phabricator via lldb-commits]

aprantl added inline comments.


================
Comment at: lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/span/TestDataFormatterLibcxxSpan.py:23
+                             '[4] = 12345',
+                             '}'])
+
----------------
Something you could consider for a follow-up commit:
A more robust and canonical way to test for synthetic children would be to check the SBAPI directly.
https://lldb.llvm.org/python_api.html
```
var = frame.FindVariable(var_name)
self.assertEquals(var.GetNumChildren(), 5)
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127481