diff options
| author | Akim Demaille <akim@lrde.epita.fr> | 2012-12-30 09:28:42 +0100 |
|---|---|---|
| committer | Akim Demaille <akim@lrde.epita.fr> | 2012-12-31 16:32:12 +0100 |
| commit | b8a8cc42c6cf43c1a66b8efdcde2a2791c99d93a (patch) | |
| tree | 35932da80b7a9e86bc84c0818ca34f43b1e49523 | |
| parent | a565e60eefd0cfe3fd982410f5573272a7004dc4 (diff) | |
| download | bison-b8a8cc42c6cf43c1a66b8efdcde2a2791c99d93a.tar.gz | |
todo: remove erroneous task
* tests/input.at: Check that there are no warnings about stray $ and @
in the epilogue.
* TODO: Remove the correponding task.
| -rw-r--r-- | TODO | 4 | ||||
| -rw-r--r-- | tests/input.at | 9 |
2 files changed, 6 insertions, 7 deletions
@@ -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 diff --git a/tests/input.at b/tests/input.at index 385d0f3a..44a24770 100644 --- a/tests/input.at +++ b/tests/input.at @@ -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 |