Re: [PATCH 3/7] skel: better aliasing of identifiers

[Théophile Ranquet]

2013/1/12 Akim Demaille <address@hidden>:
>
> Le 11 janv. 2013 à 17:22, Théophile Ranquet <address@hidden> a écrit :
>
>> +*** The epilogue is no longer affected by internal #defines
>> +
>> +  The glr.c skeleton uses defines such as #define yylval (yystackp->yyval) 
>> in
>> +  generated code.  These weren't properly undefined before the inclusion of
>> +  the user epilogue, so functions such as the following were butchered by 
>> the
>> +  preprocessor expansion:
>> +
>> +    int yylex (yy::parser::semantic_type *yylval);
>
> Just use YYSTYPE.
>
>> +  This is has been fixed: yylval, ynerrs, yychar, and yylloc are now valid
>                                     ^^^^^^^ typo

Sorry, fixed as follows:

commit 8b4499ad042d7ca58aac4c0d6eed1e250f8546d0
Author: Theophile Ranquet <address@hidden>
Date:   Mon Jan 14 11:02:12 2013 +0100

    news: typos

    * NEWS: Fix a typo, use YYSTYPE rather than semantic_type.

diff --git a/NEWS b/NEWS
index 461959e..d10d0f7 100644
--- a/NEWS
+++ b/NEWS
@@ -211,9 +211,9 @@ GNU Bison NEWS
   the user epilogue, so functions such as the following were butchered by the
   preprocessor expansion:

-    int yylex (yy::parser::semantic_type *yylval);
+    int yylex (YYSTYPE *yylval);

-  This is has been fixed: yylval, ynerrs, yychar, and yylloc are now valid
+  This is has been fixed: yylval, yynerrs, yychar, and yylloc are now valid
   identifiers for user-provided variables.

 ** Renamed %define variables