https://github.com/DavidSpickett commented:
In an ideal world we wouldn't need to add this to the SB API just for testing purposes. If it wasn't added to the API then we could ignore the minor issue I'm about to point out... The one problem I see with it is you cannot tell a failure to read from a genuinely zero length string. This is present in ReadCStringFromMemory too though. The extra red tape of SB types makes using a hypothetical `SBList<SBExpected<SBString>>` a lot of work too. So I'm ok ignoring that corner case. https://github.com/llvm/llvm-project/pull/172026 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
