[PATCH] D25816: Use descriptive message if list initializer is incorrectly parenthesized.

Alex L via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 11 02:57:22 PST 2016


I think you can go ahead and commit it, yes.

Alex

On 11 November 2016 at 05:28, Serge Pavlov <sepavloff at gmail.com> wrote:

> Is it OK to commit this patch?
>
> Thanks,
> --Serge
>
> 2016-10-21 18:21 GMT+07:00 Alex Lorenz <arphaman at gmail.com>:
>
>> arphaman added a subscriber: rsmith.
>> arphaman added a comment.
>>
>> LGTM, I added Richard in case he has something to add.
>>
>> > I chose to retain current clang behavior and reject questionable code.
>> GCC patch that introduced this message explains using warning by some
>> uncertainty, 5 years passed, I think the standard is stable in viewpoint on
>> such usage.
>>
>> Thanks for the detailed explanation, I wasn't sure if GCC's was behaving
>> according to the standard or not.
>>
>>
>> https://reviews.llvm.org/D25816
>>
>>
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161111/c01236e4/attachment.html>


More information about the cfe-commits mailing list