[llvm] r303331 - [lib/Object] - Minor API update for llvm::Decompressor.

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Tue May 23 10:36:56 PDT 2017


fair enough(:

On Tue, May 23, 2017 at 10:34 AM Rafael Avila de Espindola <
rafael.espindola at gmail.com> wrote:

> David Blaikie via llvm-commits <llvm-commits at lists.llvm.org> writes:
>
> > I don't know that changing the name was really necessary here - though
> I'm
> > open to hearing ideas/opinions.
> >
> > Most operations on a vector resize them - when they're intended to add
> > things to it. So I think the "resizeAnd" prefix is unnecessary and should
> > be removed/reverted.
>
> I suggested having a new name because there were two decompress
> functions that did different things.
>
> Cheers,
> Rafael
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170523/43ecc2be/attachment.html>


More information about the llvm-commits mailing list