On Wed, Dec 13, 2006 at 09:59:28PM -0500, Joel E. Denny wrote:
> 2. There's something wrong with the m4 in the header file.  You don't 
> generate the push parser declarations unless locations are turned on.

Huh. I don't understand how you are finding so many obvious problems
with my patches. Good job, seriously.

Is this OK?

Thanks,
Bob Rossi

2006-12-14  Bob Rossi  <[EMAIL PROTECTED]>

        * data/push.c (yypush_parse): Declare prototype regardless of
        %locations option.
Index: data/push.c
===================================================================
RCS file: /sources/bison/bison/data/push.c,v
retrieving revision 1.14
diff -u -r1.14 push.c
--- data/push.c 14 Dec 2006 20:45:47 -0000      1.14
+++ data/push.c 15 Dec 2006 01:24:45 -0000
@@ -1685,6 +1685,10 @@
 # define YYLTYPE_IS_TRIVIAL 1
 #endif
 
+]b4_pure_if([],
+          [extern YYLTYPE b4_prefix[]lloc;])
+)dnl b4_locations_if
+
 ]b4_push_if([struct ]b4_prefix[pstate;
 ]b4_c_function_decl([]b4_prefix[pstate_new], [struct ]b4_prefix[pstate *],
                     [[void], []])[
@@ -1696,9 +1700,6 @@
    [[YYSTYPE const *yynlval], [yynlval]]
    b4_locations_if([,[[YYLTYPE const *yynlloc], [yynlloc]]]))[
 enum { YYPUSH_MORE = 4 };])[
-]b4_pure_if([],
-          [extern YYLTYPE b4_prefix[]lloc;])
-)dnl b4_locations_if
 
 m4_ifdef([b4_provides],
 [[/* Copy the %provides blocks.  */

Reply via email to