bison-patches
[Top][All Lists]
Advanced

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

Re: bison 2.5 on sparc solaris 10


From: Joel E. Denny
Subject: Re: bison 2.5 on sparc solaris 10
Date: Mon, 5 Sep 2011 15:29:55 -0400 (EDT)
User-agent: Alpine 2.00 (DEB 1167 2008-08-23)

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 <address@hidden>
> 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 <address@hidden>
> 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.



reply via email to

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