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

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Mon May 22 23:41:57 PDT 2017


>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.

?Personally I like this new naming more, because it makes clear about what happens inside, but naming of things is a matter of taste first of all. May be "decompressTo" would sound better ? Though I think I am fine with just removing prefix if that cause any problems.

George.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170523/8b3900d8/attachment.html>


More information about the llvm-commits mailing list