medismailben wrote:

> > raise an exception in the decorator if the CRO has an error ?
> 
> `GetError()` can returning strings prefixed by "warning: ", so those 
> shouldn't be raised as an exception. Should the decorator have differing 
> behavior, print for non-errors, raise for errors?

Sounds good. IMO if `warning:` shouldn't be reported as an error in the 
CommandReturnObject but rather as a regular output. This is a bit orthogonal to 
this PR but still wondering what @jimingham thinks of this.

https://github.com/llvm/llvm-project/pull/195351
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to