[Lldb-commits] [PATCH] D94937: [lldb] change SBStructuredData GetStringValue signature

[Pedro Tammela via Phabricator via lldb-commits]

tammela added a comment.

In D94937#2511222 <https://reviews.llvm.org/D94937#2511222>, @JDevlieghere wrote:

> I replied before I actually tried to understand what your'e trying to achieve, and it's still not entirely clear to me. I think what you need is something similar like `python-typemaps.swig` which helps swig understand `char**` types and such.
>
> For context, originally SWIG wasn't great at handling `std::string`s (which has since been fixed) and the designers of the SB API were worried about the ABI stability of the `std::string` class. None of these are concerns anymore today, but the ABI being stable means we're stuck with it for now. If we ever do an SB API v2, this is definitely one of the things we'd fix, but currently there are no plans for that.

That's exactly what I need, thanks. I will adjust accordingly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94937