bison-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

tests: avoid useless variables


From: Akim Demaille
Subject: tests: avoid useless variables
Date: Sun, 31 Jul 2022 13:08:44 +0200

commit cf686be382ae70e69fc147768132e9334dc27814
Author: Akim Demaille <akim.demaille@gmail.com>
Date:   Sun Jul 31 12:56:59 2022 +0200

    tests: avoid useless variables
    
    Reported by clang's -Wunused-but-set-variable warning.
    
    * configure.ac (warn_common): Add -Wunused-but-set-variable.
    * tests/cxx-type.at: Don't emit location track code when we don't use
    locations.

diff --git a/configure.ac b/configure.ac
index 2f848879..08ea7cdf 100644
--- a/configure.ac
+++ b/configure.ac
@@ -129,6 +129,7 @@ if test "$enable_gcc_warnings" = yes; then
     -Wformat -Wimplicit-fallthrough -Wmismatched-dealloc -Wnull-dereference
     -Wno-sign-compare -Wno-tautological-constant-out-of-range-compare
     -Wpointer-arith -Wshadow -Wstrict-aliasing
+    -Wunused-but-set-variable
     -Wwrite-strings
     -wr188 -wr2259 -wr3179'
   warn_c='-Wbad-function-cast
diff --git a/tests/cxx-type.at b/tests/cxx-type.at
index 4b2d8ed3..4ebbf503 100644
--- a/tests/cxx-type.at
+++ b/tests/cxx-type.at
@@ -148,13 +148,13 @@ m4_define([_AT_TEST_GLR_CXXTYPES],
 ]AT_YYERROR_DEFINE[
 
 ]AT_YYLEX_PROTOTYPE[
-{
+{]AT_LOCATION_IF([[
   static int lineNum = 1;
   static int colNum = 0;
-
-#if YYPURE
+]])[
+#if YYPURE]AT_LOCATION_IF([[
 # undef yylloc
-# define yylloc (*llocp)
+# define yylloc (*llocp)]])[
 # undef yylval
 # define yylval (*lvalp)
 #endif
@@ -168,15 +168,15 @@ m4_define([_AT_TEST_GLR_CXXTYPES],
         {
         case EOF:
           return 0;
-        case '\t':
-          colNum = (colNum + 7) & ~7;
+        case '\t':]AT_LOCATION_IF([[
+          colNum = (colNum + 7) & ~7;]])[
           break;
-        case ' ': case '\f':
-          colNum += 1;
+        case ' ': case '\f':]AT_LOCATION_IF([[
+          colNum += 1;]])[
           break;
-        case '\n':
+        case '\n':]AT_LOCATION_IF([[
           lineNum += 1;
-          colNum = 0;
+          colNum = 0;]])[
           break;
         default:
           {
@@ -190,8 +190,8 @@ m4_define([_AT_TEST_GLR_CXXTYPES],
 
                 do
                   {
-                    buffer[i++] = YY_CAST (char, c);
-                    colNum += 1;
+                    buffer[i++] = YY_CAST (char, c);]AT_LOCATION_IF([[
+                    colNum += 1;]])[
                     assert (i != sizeof buffer - 1);
                     c = getchar ();
                   }
@@ -203,8 +203,8 @@ m4_define([_AT_TEST_GLR_CXXTYPES],
                 yylval = new_term (strdup (buffer));
               }
             else
-              {
-                colNum += 1;
+              {]AT_LOCATION_IF([[
+                colNum += 1;]])[
                 tok = c;
                 yylval = YY_NULLPTR;
               }]AT_LOCATION_IF([[




reply via email to

[Prev in Thread] Current Thread [Next in Thread]