]> git.saurik.com Git - bison.git/commitdiff
todo: remove erroneous task
authorAkim Demaille <akim@lrde.epita.fr>
Sun, 30 Dec 2012 08:28:42 +0000 (09:28 +0100)
committerAkim Demaille <akim@lrde.epita.fr>
Mon, 31 Dec 2012 15:32:12 +0000 (16:32 +0100)
* tests/input.at: Check that there are no warnings about stray $ and @
in the epilogue.
* TODO: Remove the correponding task.

TODO
tests/input.at

diff --git a/TODO b/TODO
index e8509e3cfcf74651f579d6d79f472d7ef339c935..2a08164a4dc44edc6ace5d05714819eb1525488a 100644 (file)
--- a/TODO
+++ b/TODO
@@ -43,10 +43,6 @@ back-ported.
     yytoken = yytranslate_ (yychar);
 
 
-** $ and others in epilogue
-A stray $ is a warning in the actions, but an error in the epilogue.
-IMHO, it should not even be a warning in the epilogue.
-
 ** stack.hh
 Get rid of it.  The original idea is nice, but actually it makes
 the code harder to follow, and uselessly different from the other
index 385d0f3a9c1953a9d1949be63cee0d86e0c6d278..44a2477010a44b1b3f83b1e0f80c7ad7ddb14d45 100644 (file)
@@ -1668,19 +1668,22 @@ AT_DATA_GRAMMAR([[input.y]],
 %destructor     { $%; @%; } <*> exp TOK;
 %initial-action { $%; @%; };
 %printer        { $%; @%; } <*> exp TOK;
+%{ $ @ %} // Should not warn.
 %%
 exp: TOK        { $%; @%; $$ = $1; };
+%%
+$ @ // Should not warn.
 ]])
 
-AT_BISON_CHECK([[input.y]], 0, [],
+AT_BISON_CHECK([[-Wall input.y]], 0, [],
 [[input.y:11.19: warning: stray '$' [-Wother]
 input.y:11.23: warning: stray '@' [-Wother]
 input.y:12.19: warning: stray '$' [-Wother]
 input.y:12.23: warning: stray '@' [-Wother]
 input.y:13.19: warning: stray '$' [-Wother]
 input.y:13.23: warning: stray '@' [-Wother]
-input.y:15.19: warning: stray '$' [-Wother]
-input.y:15.23: warning: stray '@' [-Wother]
+input.y:16.19: warning: stray '$' [-Wother]
+input.y:16.23: warning: stray '@' [-Wother]
 ]])
 
 AT_CLEANUP