Thanks for catching those.  I prefer the 'const' after the type that
it modifies, so I installed the following into gnulib.  (For coreutils
I guess I need to come up to speed on its new version control system.)

2006-10-24  Paul Eggert  <[EMAIL PROTECTED]>

        * lib/getdate.y (yyerror): Make the arguments pointer-to-const,
        to pacify GCC with some -W flags enabled.  Problem reported by
        Bruno Haible.

--- lib/getdate.y       13 Sep 2006 22:38:14 -0000      1.106
+++ lib/getdate.y       24 Oct 2006 19:54:26 -0000
@@ -195,7 +195,7 @@ typedef struct
 
 union YYSTYPE;
 static int yylex (union YYSTYPE *, parser_control *);
-static int yyerror (parser_control *, char *);
+static int yyerror (parser_control const *, char const *);
 static long int time_zone_hhmm (textint, long int);
 
 %}
@@ -1106,7 +1106,8 @@ yylex (YYSTYPE *lvalp, parser_control *p
 
 /* Do nothing if the parser reports an error.  */
 static int
-yyerror (parser_control *pc ATTRIBUTE_UNUSED, char *s ATTRIBUTE_UNUSED)
+yyerror (parser_control const *pc ATTRIBUTE_UNUSED,
+        char const *s ATTRIBUTE_UNUSED)
 {
   return 0;
 }


Reply via email to