[Lldb-commits] [PATCH] D112222: [LLDB] libcxx summary formatters for std::string_view

[Ben Jackson via Phabricator via lldb-commits]

puremourning updated this revision to Diff 381372.
puremourning added a comment.

Add some tests for dodgy inputs.
Handle invalid cases more consistently.
Factor out some copy paste code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112222

Files:
  lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp
  lldb/source/Plugins/Language/CPlusPlus/LibCxx.cpp
  lldb/source/Plugins/Language/CPlusPlus/LibCxx.h
  lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/string_view/Makefile
  lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/string_view/TestDataFormatterLibcxxStringView.py
  lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/string_view/main.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D112222.381372.patch
Type: text/x-patch
Size: 28386 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20211021/584d75fa/attachment-0001.bin>