From: Theophile Ranquet <ranquet@lrde.epita.fr>
Date: Tue, 15 Jan 2013 17:03:39 +0000 (+0100)
Subject: tests: better silencing of unused argument warnings
X-Git-Tag: v2.7.90~169
X-Git-Url: https://git.saurik.com/bison.git/commitdiff_plain/32f4c0a1b29ed97f82ed3c41156ac6ca856a60f2

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 ad08b8ad..0bebe8a5 100644
--- 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()
 {}