This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][4.6] detect C++ errors to fix 2288 and 18770
- From: Gerald Pfeifer <gerald at pfeifer dot com>
- To: Jack Howarth <howarth at bromo dot med dot uc dot edu>, Janis Johnson <janisjo at codesourcery dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Richard Guenther <richard dot guenther at gmail dot com>
- Date: Sun, 22 May 2011 23:29:06 +0200 (CEST)
- Subject: Re: [PATCH][4.6] detect C++ errors to fix 2288 and 18770
- References: <1268945454.2219.13.camel@janis-laptop> <20110522193335.GA9729@bromo.med.uc.edu>
On Sun, 22 May 2011, Jack Howarth wrote:
>> 2010-03-18 Janis Johnson <janis187@us.ibm.com>
>>
>> gcc/cp
>> PR c++/2288
>> PR c++/18770
>> * name-lookup.h (enum scope_kind): Add sk_cond.
>> * name-lookup.c (pushdecl_maybe_friend): Get scope of shadowed local.
>> Detect and report error for redeclaration from for-init or if
>> or switch condition.
>> (begin_scope): Handle sk_cond.
>> * semantics.c (begin_if_stmt): Use sk_cond.
>> (begin switch_stmt): Ditto.
>>
>> gcc/testsuite/
>> PR c++/2288
>> PR c++/18770
>> * g++.old-deja/g++.jason/cond.C: Remove xfails.
>> * g++.dg/parse/pr18770.C: New test.
> Any chance we can get this committed to gcc trunk while in 4.7 stage 1
> (since it was already approved for 4.6 stage 1 and never applied...
> http://gcc.gnu.org/ml/gcc-patches/2010-05/msg00264.html). It would be
> nice to have in gcc 4.7 so that other open source projects that only
> build against g++ would be alerted to fix their sources.
It's my understanding that an approved patch can be committed by anyone
with write access unless it is retracted or there is some other reason
not to. Customary that is left to the submitter if she has write access,
of course.
In the concrete case, Janis is now reachable at janisjo@codesourcery.com
according to the gcc/MAINTAINERS file, by the way. Including her...
Gerald