[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