mib added a comment.

In D95710#3118346 <https://reviews.llvm.org/D95710#3118346>, @labath wrote:

> Sorry for grave digging, but my compiler just became smart enough to start 
> complaining about this.

Haha, not at all! This wasn't called when I implemented it, but I stumbled into 
it in following patches and fixed it in D112109 
<https://reviews.llvm.org/D112109>.

The fix hasn't landed yet because I'm still trying to understand why would the 
python-initialized `SBStructuredData` be pointing the corrupted memory when 
trying accessing it in C++, instead of deserializing it into a std::string, and 
serializing it back, which is more a workaround that an actual fix.


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

https://reviews.llvm.org/D95710

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
  • [Lldb-commits] [PATCH]... Pavel Labath via Phabricator via lldb-commits
    • [Lldb-commits] [P... Med Ismail Bennani via Phabricator via lldb-commits

Reply via email to