This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [4.7] Fix a couple of C++isms
- From: Andrew Pinski <pinskia at gmail dot com>
- To: Jeff Law <law at redhat dot com>
- Cc: Eric Botcazou <ebotcazou at adacore dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Sun, 21 Apr 2013 14:17:33 -0700
- Subject: Re: [4.7] Fix a couple of C++isms
- References: <7725102 dot aKVEupMyhz at polaris> <51745678 dot 7010604 at redhat dot com>
On Sun, Apr 21, 2013 at 2:13 PM, Jeff Law <law@redhat.com> wrote:
> On 04/21/2013 01:59 AM, Eric Botcazou wrote:
>>
>> Bootstrapped/regtested on x86_64-suse-linux, applied as obvious.
>>
>>
>> 2013-04-21 Eric Botcazou <ebotcazou@adacore.com>
>>
>> * cfgexpand.c (avoid_complex_debug_insns): Fix C++ism.
>> cp/
>> * parser.c (cp_parser_late_return_type_opt): Likewise.
>
> Umm, I thought declaring at first use (or near first use) was one of the
> approved C++-isms?
Yes but not for 4.7.x which is still written in C90.
Thanks,
Andrew
>
> http://gcc.gnu.org/wiki/CppConventions
>
>
> Jeff
>
On Sun, Apr 21, 2013 at 2:13 PM, Jeff Law <law@redhat.com> wrote:
> On 04/21/2013 01:59 AM, Eric Botcazou wrote:
>>
>> Bootstrapped/regtested on x86_64-suse-linux, applied as obvious.
>>
>>
>> 2013-04-21 Eric Botcazou <ebotcazou@adacore.com>
>>
>> * cfgexpand.c (avoid_complex_debug_insns): Fix C++ism.
>> cp/
>> * parser.c (cp_parser_late_return_type_opt): Likewise.
>
> Umm, I thought declaring at first use (or near first use) was one of the
> approved C++-isms?
>
> http://gcc.gnu.org/wiki/CppConventions
>
>
> Jeff
>