[Lldb-commits] [lldb] [lldb] Unify implementation of CommandReturnObject::SetError(NFC) (PR #110707)
via lldb-commits
lldb-commits at lists.llvm.org
Tue Oct 1 13:39:25 PDT 2024
jimingham wrote:
`SetError(Status error, const char* fallback_error_string)` was a badly named API. What it actually does is APPEND either the error string in the incoming `error`, or if that had no error string, the `fallback_error_string`, to whatever was already in the command error.
So far as I can tell, there are no internal uses of this fallback_error_string (I removed that argument from the API and lldb still compiles.) However, we made an equally poorly named SBCommandReturnObject API that does take this string.
So this change does change the behavior of this quite poorly named API which I bet no one was using for that purpose, but still...
https://github.com/llvm/llvm-project/pull/110707
More information about the lldb-commits
mailing list