patch: check constructor initializer list packs

Richard Smith richard at metafoo.co.uk
Wed Jun 12 17:37:49 PDT 2013


Looks great, thanks!

On Wed, Jun 12, 2013 at 5:23 PM, Nick Lewycky <nlewycky at google.com> wrote:
> 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
>> >
>
>



More information about the cfe-commits mailing list