[PATCH] D28515: [Support] - Introduce zlib::toString(zlib::Status)

[Davide Italiano via Phabricator via llvm-commits]

davide added a comment.

In https://reviews.llvm.org/D28515#645037, @grimar wrote:

> In https://reviews.llvm.org/D28515#644772, @davide wrote:
>
> > As an alternative, you can merge the patch and the patch where you plan to use this code together (which would make more sense, IMHO).
> >  In fact, I'm still confused why you splitted this in the first place.
>
>
> I wanted to use it in LLD. That would mean 2 patches anyways, LLD and this one.
>
> Previously Rafael suggested to return Error for methods of this API instead of status.
>  I think that is good idea, result will be single patch affecting users of this (Decomressor class, something else probably) and changing API itself in a single patch, just like you suggesting.


Yes, that would be better.


https://reviews.llvm.org/D28515