tests: better silencing of unused argument warnings

input.yy:35:44: error: unused parameter 'msg' [-Werror,-Wunused-parameter]
void yy::parser::error (std::string const& msg)
                                           ^

* tests/c++.at (C++ GLR parser identifier shadowing): Don't name unused
argument, use YYUSE instead of a direct cast to void.

diff --git a/tests/c++.at b/tests/c++.at
index ad08b8ad9c125b7e2ce4fc16222271a9bf995f3e..0bebe8a55ef78aac678e66fd0a8f8f61f1d783c8 100644 (file)
--- a/tests/c++.at
+++ b/tests/c++.at
@@ -781,14 +781,15 @@ exp: ZERO
 
 int yylex (yy::parser::semantic_type *yylval)
 {
-  (void) *yylval;
+  // Note: this argument is unused, but named on purpose.  There used to be a
+  // bug with a macro that erroneously expanded this identifier to
+  // yystackp->yyval.
+  YYUSE (yylval);
   return yy::parser::token::ZERO;
 }
 
-void yy::parser::error (std::string const& msg)
-{
-  (void) msg;
-}
+void yy::parser::error (std::string const&)
+{}
 
 int main()
 {}