https://github.com/labath commented:

> 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. 

>  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.

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

Reply via email to