On Sun, 21 Aug 2011, Joel E. Denny wrote: > I can get gcc to report that problem by adding the -pedantic option in the > test suite. The second patch below adds -pedantic when bison is > configured with --enable-gcc-warnings, and it fixes another minor test > suite problem reported by -pedantic. The third patch fixes a problem it > reports for data/lalr1.cc on master. I haven't pushed either of these > patches yet. Any objections from anyone?
I've pushed those. > >From 5422d56a718805a2470558808355e04182a12d65 Mon Sep 17 00:00:00 2001 > From: Joel E. Denny <[email protected]> > Date: Sun, 21 Aug 2011 13:07:16 -0400 > Subject: [PATCH 2/2] tests: add -pedantic for --enable-gcc-warnings. > > This should help to avoid some portability problems. For example, > it would have revealed the empty unions fixed by the last patch > * configure.ac (WARN_CFLAGS_TESTS, WARN_CXXFLAGS_TEST): Implement. > * tests/synclines.at (AT_TEST_SYNCLINE): Avoid -pedantic warning > about an empty translation unit. > >From f3bd3f7876b4d4c6d22b9590e1a0938ad342c91f Mon Sep 17 00:00:00 2001 > From: Joel E. Denny <[email protected]> > Date: Sun, 21 Aug 2011 17:35:26 -0400 > Subject: [PATCH] lalr1.cc: remove trailing comma from enumerator list. > > * data/lalr1.cc (yy::parser): Here. This suppresses a -pedantic > warning.
