patch: check constructor initializer list packs

Nick Lewycky nlewycky at google.com
Wed Jun 12 17:23:45 PDT 2013


On 12 June 2013 16:35, Richard Smith <richard at metafoo.co.uk> wrote:

> Is there a reason not to just build a single vector of unexpanded
> packs, and check it once?
>

Nope! Updated patch attached.

Nick


> On Wed, Jun 12, 2013 at 4:20 PM, Nick Lewycky <nlewycky at google.com> wrote:
> > This patch fixes a slight oversight with a pack expansion of constructor
> > initialization lists of the form "S() : T<Pack1>(Pack2)... {}" not
> checking
> > the arity of the unexpanded packs in the initializer expression.
> >
> > Fixes PR16303. Please review!
> >
> > Nick
> >
> > _______________________________________________
> > cfe-commits mailing list
> > cfe-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130612/851e7365/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pr16303-2.patch
Type: application/octet-stream
Size: 1630 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130612/851e7365/attachment.obj>


More information about the cfe-commits mailing list