G++ patch to handle scoping correctly

Geoff Keating geoffk@geoffk.org
Fri Dec 1 14:07:00 GMT 2000


Will Cohen <wcohen@redhat.com> writes:

> 2000-12-01  Will Cohen  <wcohen@redhat.com>
> 
> 	* netgcc/gcc/cp/decl.c (define_label): Cleared more_cleanups_ok in
> 	surrounding scopes.

The testcase is OK.

For the remaining code,

> !   for (p = current_binding_level; p; p = p->level_chain)
> !     p->more_cleanups_ok = 0;

Is there a reason this goes all the way to the topmost binding level?

I would think it could stop at the binding level which is a parent of
both the current and the label's binding level.

-- 
- Geoffrey Keating <geoffk@geoffk.org>


More information about the Gcc-patches mailing list