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

[Jason Molenda via lldb-commits]

jasonmolenda wrote:

> I'd return an `SBValueList` as well: in the `bad_addr` case, we don't return an error which is wrong imo. 

An SBValueList is another valid way to express a read failure.  Note that users are expected to check SBValue::GetError() to see if a value was able to be fetched.  SBValue::IsValid() only tells you if there is a ValueObject backing this SB object, and there is, to hold the error value/message.

https://github.com/llvm/llvm-project/pull/172026