This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


Michael Snyder writes:
 > 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.


No, no problem. It's fine.

Elena


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]