Re: push parser implemenation

[Akim Demaille]

>>> "Bob" == Bob Rossi <address@hidden> writes:

 > On Fri, Sep 15, 2006 at 10:33:03AM +0200, Akim Demaille wrote:
 >> >>> "Bob" == Bob Rossi <address@hidden> writes:
 >> 
 >> > Honestly, I don't understand why a pull parser needs this feature. If
 >> > the only reason the pull parser had extra parameters via %parse-param
 >> > was to pass those parameters to the lexer, then the push parser doesn't
 >> > need this feature, since the user calls the lexer directly.
 >> 
 >> You're forgetting auxiliary routines (yyerror, yyprint, yydestruct
 >> etc.).  Actually your work is almost to mimic what lalr1.cc does, but
 >> with member functions being functions.  The parser struct must
 >> aggregate the parse-params, and provide it to the auxiliary functions.
 >> 
 >> As for yyparse, do not forget about the %initial-action either.

 > Hi Akim,

 > It seems that Paul told me not to worry about %parse-param. Unless
 > decided otherwise, I'm not going to include it in the patch.

At some point it will have to take them into account :)  Maybe not by
you, I was not requesting that you take care of them, I was pointing
out their support is needed.  Of course %lex-param is a totally
different issue (but for the wrapper named yyparse).