[Lldb-commits] [PATCH] D95710: [lldb/Commands] Add command options for ScriptedProcess to ProcessLaunch

[Med Ismail Bennani via Phabricator via lldb-commits]

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