This is the mail archive of the
java-patches@gcc.gnu.org
mailing list for the Java project.
Re: gtk-peer compile fixes for gcc-2.95
- From: Tom Tromey <tromey at redhat dot com>
- To: Mark Wielaard <mark at klomp dot org>
- Cc: Michael Koch <konqueror at gmx dot de>, java-patches at gcc dot gnu dot org,classpath at gnu dot org
- Date: 05 Apr 2004 11:43:12 -0600
- Subject: Re: gtk-peer compile fixes for gcc-2.95
- References: <1080236148.18716.161.camel@elsschot.wildebeest.org><87wu58qpmw.fsf@fleche.redhat.com><1080295746.18716.463.camel@elsschot.wildebeest.org><200403261128.55981.konqueror@gmx.de><1080301292.18716.480.camel@elsschot.wildebeest.org><871xn8h9mt.fsf@fleche.redhat.com><1081118952.7548.253.camel@elsschot.wildebeest.org>
- Reply-to: tromey at redhat dot com
>>>>> "Mark" == Mark Wielaard <mark@klomp.org> writes:
Mark> The attached patch does that and enables -Wall for GNU Classpath.
Nice. I skimmed it and it generally looks fine to me.
Mark> It fixes all issues found when compiling with -std=c89. Lots in the gtk
Mark> peers I am afraid. So please yell and scream if this is to intrusive. It
Mark> would be good if the gtk-peer stuff in libgcj was also compiled with
Mark> this flag.
I think it is a requirement, not just nice to have. Otherwise we're
going to end up reintroducing code that isn't C89.
Here's an untested patch for the GUI branch.
Tom
Index: ChangeLog
from Tom Tromey <tromey@redhat.com>
* Makefile.am (WARNINGS): Added -std=c89.
Index: Makefile.am
===================================================================
RCS file: /cvs/gcc/gcc/libjava/Makefile.am,v
retrieving revision 1.361.2.6
diff -u -r1.361.2.6 Makefile.am
--- Makefile.am 4 Mar 2004 17:31:02 -0000 1.361.2.6
+++ Makefile.am 5 Apr 2004 17:50:07 -0000
@@ -105,7 +105,7 @@
GCC_UNWIND_INCLUDE = @GCC_UNWIND_INCLUDE@
-WARNINGS = -W -Wall
+WARNINGS = -W -Wall -std=c89
## We need _GNU_SOURCE defined for some Linux builds. It doesn't hurt
## to always define it.
AM_CXXFLAGS = -fno-rtti -fnon-call-exceptions $(THREADCXXFLAGS) \