[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: push parser

From: Paul Eggert
Subject: Re: push parser
Date: Mon, 25 Sep 2006 23:15:50 -0700
User-agent: Gnus/5.1008 (Gnus v5.10.8) Emacs/21.4 (gnu/linux)

Bob Rossi <address@hidden> writes:

>> I have a small patch that get's me this far:
>>   status = yypushparse (ctx, ch, &yylval);
>> What should I do about yylval?

Sorry, I don't understand the question; perhaps you meant yyloc?

>> I can attempt to make yypushparse have an extra parameter when locations
>> are enabled. Does that sound correct?

Yes, that sounds fine.

> Here, is a patch I'm posting just for you to tell me if this is the
> appropriate solution.

Sorry, I'm a bit lost.  That patch modifies yacc.c and puts a bunch of
push-parser stuff in it; but yacc.c differs from push.c afterwards.
Is this intended?  I thought the patch would just affect push.c.

The push.c patch looks OK, except there still seems to be some
confusion about YYLTYPE_IS_TRIVIAL versus b4_locations_if.
YYLTYPE_IS_TRIVIAL is defined if it is known that YYLTYPE is a trivial
type (in the C++ sense: it's just a bunch of bytes).  But in the patch
YYLTYPE_IS_TRIVIAL seems to be used for some other purpose.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]