[RFA] Fix regexp problem with "operator* etc.

Michael Snyder msnyder@cygnus.com
Wed Dec 5 12:13:00 GMT 2001


Elena Zannoni wrote:
> 
> Michael Snyder writes:
>  >
>  > I committed this change, without Michael Chastain's worthy suggestion
>  > regarding the size of the alloca.  Will submit that separately.
>  >
>  > Michael Elizabeth Chastain wrote:
>  > >
>  > > cplusfuncs.exp:
>  > >   In my maintainer area.
>  > >   Approved unconditionally.
>  > >   No matter what gdb does, the test script should do the right thing.
>  > >
>  > > alloca change:
>  > >   Outside my maintainer area.
>  > >   Recommended for rework.
>  > >
>  > >   I think the size is long enough, but the proof of correctness is
>  > >   too delicate.  There is no space explicitly allocated for the final
>  > >   null, so the alloca is correct only because it occurs in a context
>  > >   where at least one character is being deleted.  This also relies on
>  > >   some character-class tests being the same in this function and
>  > >   in operator_chars.   How about something stupidly correct:
>  > >
>  > >     alloca (8 + fix + strlen(opname) + 1)
> 
> Ah, OK.
> 
>  > >
>  > > operator_chars change:
>  > >   Outside my maintainer area.
>  > >   Proofread and tested, works for me.
>  > >   Recommended for approval.
> 
> Whoops, I missed this one. Sorry, I'll go and look at it. Did you
> commit it already?

Yep, I guess I mistook silence for assent.  Sorry if I overstepped.
I'll back it out if it doesn't look OK to you.



More information about the Gdb-patches mailing list