[Lldb-commits] [lldb] [lldb] Document the behaviour of IsValid for SBError (PR #170862)

[Jacob Lalonde via lldb-commits]

Jlalond wrote:

@DavidSpickett LGTM, but I have a follow up.

We're describing it as empty, but the documentation does imply that empty can equal success. This confuses me a bit because if I pass an SBError to a function that is expected to return an Status it should populate the underlying status and thus `IsValid()` would return true if there is is an error or a success correct?

Thus that makes SBError kinda weird where IsValid() only returns false if it hasn't been exercised?

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