Re: Bison yacc.c

[Paul Eggert]

twlevo <address@hidden> writes:

> in yacc.c line 558 noticed possibly better to have
> #define YYEMPTY (-2)
> #define yyclearin (yychar = YYEMPTY)

I assume you mean a patch like this:

--- old/yacc.c  2006-03-12 06:45:27.000000000 -0800
+++ new/yacc.c  2006-04-09 00:15:10.000000000 -0700
@@ -554,8 +554,8 @@ static const ]b4_int_type_for([b4_stos])
 };
 
 #define yyerrok                (yyerrstatus = 0)
-#define yyclearin      (yychar = YYEMPTY)
 #define YYEMPTY                (-2)
+#define yyclearin      (yychar = YYEMPTY)
 #define YYEOF          0
 
 #define YYACCEPT       goto yyacceptlab
#define yyclearin       (yychar = YYEMPTY)

I don't think this is needed, since the rules about expansion of
#define are quite standard.

> ps. how about a bison-2.1b test version

We're trying to generate Bison 2.2.  At this point the only holdup is
the licensing wording.