jimingham wrote:

> Сonfusion arises between `Status error` and `llvm::Error error`. Consider 
> renaming to `Status status` or such.

The lldb class that is now called Status used to be called Error.  But then 
llvm wanted to use Error and we had to change Error to Status but we didn't 
rename all the variables because that would have been a lot more work.  So:

$ cd llvm-project/lldb
$  find . -type f -exec grep "Status error" {} \+ | wc -l
    1546

So either we change those everywhere or agree that this is a common lldb 
pattern...

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

Reply via email to