Re: %destructor feedback

2005-10-30 Thread Wolfgang Spraul
Akim - > Could you please confirm that the appended patch (committed) > does address your issue? > > Index: ChangeLog > * data/c.m4 (b4_yydestruct_generate, b4_yysymprint_generate): Pass > the %parse-params. > ... Please excuse the delay. Your patch didn't cleanly apply to the bison-2

Re: %destructor feedback

2005-10-20 Thread Wolfgang Spraul
ge this, then I could retire my sed fix. Regards, Wolfgang On Thursday 20 October 2005 18:38, you wrote: > >>> "Wolfgang" == Wolfgang Spraul <[EMAIL PROTECTED]> writes: > > > > Akim - > > Hey, > > >> I'm not sure about %param-dest

Re: %destructor feedback

2005-10-20 Thread Wolfgang Spraul
Akim - > I'm not sure about %param-destructor, it is not very clean, but just > like there is %initial-action, we could introduce a %final-action > always run when leaving yyparse (whether on failure or not). When you say %param-destructor, do you mean the %destructor-param I was talking about?

Re: %destructor feedback

2005-10-18 Thread Wolfgang Spraul
Joel - > Are you now talking about a YYCLEANUP macro or YYABORT_CLEANUP and > YYERROR_CLEANUP macros? > > If you mean YYCLEANUP, it would need to return. Would a function call be > too inefficient? Sorry, I made a mistake here. You are right - if YYCLEANUP is a goto, then it cannot return. Sure

Re: %destructor feedback

2005-10-18 Thread Wolfgang Spraul
fer more cleanup flexibility (macros) Regards, Wolfgang On Tuesday 18 October 2005 05:26, you wrote: > On Tue, 18 Oct 2005, Wolfgang Spraul wrote: > >>>>Note that in the future, Bison might also consider that right hand > >>>>side members that are not m

Re: %destructor feedback

2005-10-17 Thread Wolfgang Spraul
nc2($2); YYABORT; } in bison 2.1, it now looks like this: if (error) YYABORT; Thanks for your detailed comments, I found them very helpful! Best Regards, Wolfgang On Monday 17 October 2005 19:21, you wrote: > On Mon, 17 Oct 2005, Wolfgang Spraul wrote: > > the reason I prefer the aut

Re: %destructor feedback

2005-10-17 Thread Wolfgang Spraul
On Monday 17 October 2005 08:34, Joel E. Denny wrote: > On Sun, 16 Oct 2005, Wolfgang Spraul wrote: > > Let's say you have an action that gets an object passed in which has a > > destructor. When using YYABORT inside the action, the destructor for this > > object is always c

%destructor feedback

2005-10-15 Thread Wolfgang Spraul
Under %destructor, the bison 2.1 manual says: "this feature is still experimental, as there has not been enough user feedback." Well, I can help with that: I am using %destructor for a while now and I like it. It works well. One problem I have is that there is no %destructor-param same as the