adrian-prantl wrote: > > Status is already extensively unittested. > > The fact that you were able to pull this switch off without changing a single > line of test code tells me otherwise. Surely there is _some_ new > functionality being added here.
That's *why* the implementation has all these quirky details :-) > > > Is there something you feel is missing? > > For example, the value of GetError()/GetType() for multi-error objects. And > the ability to copy an ErrorInfo object and make sure that the copy preserves > its type and content. Maybe there's more, but these are the most obvious > new/changed behaviors of the class. I'll add a test for that. https://github.com/llvm/llvm-project/pull/106774 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits