Fix testsuite for ./configure --enable-gcc-warnings:

* configure.ac (gcc-warnings): Move -Wall before -Wno-sign-compare.
Otherwise, gcc 4.1.0 (at least) warns about sign comparisons in
__AT_CHECK_PRINTER_AND_DESTRUCTOR in tests/actions.at.
* test/input.at (Torturing the Scanner): #include <stdlib.h> for abort.
* test/regression.at (Diagnostic that expects two alternatives):
Likewise.

diff --git a/ChangeLog b/ChangeLog
index 4f6b5a02608abfe67481a110c66622ea8421da4c..9fed16bb1b23729575ad1ee9d444158d54f48870 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2006-10-14  Joel E. Denny  <jdenny@ces.clemson.edu>
+
+       Fix testsuite for ./configure --enable-gcc-warnings:
+       * configure.ac (gcc-warnings): Move -Wall before -Wno-sign-compare.
+       Otherwise, gcc 4.1.0 (at least) warns about sign comparisons in
+       __AT_CHECK_PRINTER_AND_DESTRUCTOR in tests/actions.at.
+       * test/input.at (Torturing the Scanner): #include <stdlib.h> for abort.
+       * test/regression.at (Diagnostic that expects two alternatives):
+       Likewise.
+
 2006-10-12  Paul Eggert  <eggert@cs.ucla.edu>
 
        * bootstrap.conf (gnulib_modules): Add config-h.

diff --git a/configure.ac b/configure.ac
index 52241669c188e7ae5375043f4be1ca14b8c89296..e71fadb431bf4fbd294c427a89545bf8d4b27625 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -46,8 +46,8 @@ if test "${enableval}" = yes; then
   gl_WARNING_CFLAGS([-Werror])
   AC_SUBST([WERROR_CFLAGS], [$WARNING_CFLAGS])
   WARNING_CFLAGS=
-  gl_WARNING_CFLAGS([-Wextra -Wno-sign-compare])
   gl_WARNING_CFLAGS([-Wall])
+  gl_WARNING_CFLAGS([-Wextra -Wno-sign-compare])
   gl_WARNING_CFLAGS([-Wcast-align])
   gl_WARNING_CFLAGS([-Wcast-qual])
   gl_WARNING_CFLAGS([-Wformat])

diff --git a/tests/input.at b/tests/input.at
index f6eb33c4d6e4f628a4462639eb2da17c3675de4c..dedd65636641113ed81ca9f41780c07767976155 100644 (file)
--- a/tests/input.at
+++ b/tests/input.at
@@ -396,6 +396,7 @@ char apostrophe = '\'';
 #endif
 
 #include <stdio.h>
+#include <stdlib.h>
 %}
 /* %{ and %} can be here too. */
 

diff --git a/tests/regression.at b/tests/regression.at
index 6a0703ae8893ea183333a4957b18bc8afa58338d..761e1bedbfd17e4ae92fd382de5d4bbdf5fee4e1 100644 (file)
--- a/tests/regression.at
+++ b/tests/regression.at
@@ -959,6 +959,7 @@ m4_define([_AT_DATA_EXPECT2_Y],
 static int yylex (AT_LALR1_CC_IF([int *], [void]));
 AT_LALR1_CC_IF([],
 [#include <stdio.h>
+#include <stdlib.h>
 static void yyerror (const char *);])
 %}
 $1