[cfe-commits] [PATCH] Fix for VarDecl::getSourceRange() when static array has initializer.

Nico Weber thakis at chromium.org
Tue Jan 22 09:00:45 PST 2013


r173170, thanks for the patch!

On Tue, Jan 22, 2013 at 8:43 AM, Will Wilson <will at indefiant.com> wrote:
> Just didn't want it to slip thought the net - can anyone commit it?
>
> Cheers!
> Will.
>
>
> On 16 January 2013 21:22, Richard Smith <richard at metafoo.co.uk> wrote:
>>
>> LGTM
>>
>> On Wed, Jan 16, 2013 at 7:14 AM, Will Wilson <will at indefiant.com> wrote:
>> >
>> >>> All changes should have an associated test case (where possible).
>> >>> Please also add a comment, this is pretty subtle!
>> >>
>> >>
>> >> Sorry for the delay, only just had a chance to knock up the patch and
>> >> test
>> >> case (attached). Let me know if it fits the bill.
>> >
>> >
>> >  Scrub that, the original comment I wrote was pretty vague. New patch
>> > attached with a (hopefully) clearer comment.
>
>
>
>
> --
> Indefiant Ltd.
>
> Firsby Lodge, New Main Road, Scamblesby, Louth, Lincs LN11 9XH UK
> Tel: +44 20 8123 7663 England Registered No. 07936820 VAT No. 128556202
>
> _______________________________________________
> 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