[Lldb-commits] [lldb] [lldb] Add ReadCStrings API to Process (PR #172026)

[David Spickett via lldb-commits]

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