From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 13398 invoked by alias); 15 Jan 2006 20:40:25 -0000 Received: (qmail 13390 invoked by uid 22791); 15 Jan 2006 20:40:24 -0000 X-Spam-Check-By: sourceware.org Received: from sibelius.xs4all.nl (HELO sibelius.xs4all.nl) (82.92.89.47) by sourceware.org (qpsmtpd/0.31) with ESMTP; Sun, 15 Jan 2006 20:40:20 +0000 Received: from elgar.sibelius.xs4all.nl (root@elgar.sibelius.xs4all.nl [192.168.0.2]) by sibelius.xs4all.nl (8.13.4/8.13.4) with ESMTP id k0FKdkdH009842; Sun, 15 Jan 2006 21:39:46 +0100 (CET) Received: from elgar.sibelius.xs4all.nl (kettenis@localhost.sibelius.xs4all.nl [127.0.0.1]) by elgar.sibelius.xs4all.nl (8.13.4/8.13.3) with ESMTP id k0FKdki8005600; Sun, 15 Jan 2006 21:39:46 +0100 (CET) Received: (from kettenis@localhost) by elgar.sibelius.xs4all.nl (8.13.4/8.13.4/Submit) id k0FKdk9x002692; Sun, 15 Jan 2006 21:39:46 +0100 (CET) Date: Sun, 15 Jan 2006 20:40:00 -0000 Message-Id: <200601152039.k0FKdk9x002692@elgar.sibelius.xs4all.nl> From: Mark Kettenis To: drow@false.org CC: gdb-patches@sourceware.org In-reply-to: <20060115202601.GA12204@nevyn.them.org> (message from Daniel Jacobowitz on Sun, 15 Jan 2006 15:26:01 -0500) Subject: Re: [RFA] Turn on -Werror by default References: <200601081759.k08HxOxP007121@elgar.sibelius.xs4all.nl> <20060108222546.GA4147@nevyn.them.org> <200601082253.k08MrLY1000252@elgar.sibelius.xs4all.nl> <20060115170103.GA5830@nevyn.them.org> <200601151755.k0FHtSR9016305@elgar.sibelius.xs4all.nl> <20060115182055.GA8189@nevyn.them.org> <200601152004.k0FK41sV010809@elgar.sibelius.xs4all.nl> <20060115202601.GA12204@nevyn.them.org> Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2006-01/txt/msg00184.txt.bz2 > Date: Sun, 15 Jan 2006 15:26:01 -0500 > From: Daniel Jacobowitz > > > Since we also seem to agree that the "warn for pointer argument > > passing or assignment with different signedness" warning from GCC 4 is > > mostly pedantry, how about the attached patch, which adds > > -Wno-pointer-sign" to the mix. > > > > I can add that bit first, and then when we've fixed the few remaining > > warnings, we can enable -Werror. > > I think we do need to fix the pointer sign warnings eventually, as a > code cleanliness issue, so perhaps this is a good time to do it. > You've needled me enough this week that I'm actually rolling now :-) > Give me a few days. Heh ;-) > BTW, if we were to go with your patch, wouldn't we would need to > conditionalize the warning on an appropriate version of GCC? Or is > there pre-existing magic for this in $build_warnings? Yes, there is. > I'd gotten it into my head that there was no way to turn these off. > Nice to be wrong about that. It may very well be that -Wno-pointer-sign, wasn't there in GCC HEAD when Andrew initially started "fixing" things. Anyway, I'll keep this patch in my tree, and will bring it up again if it turns out we need it. I'll start nagging again in two weeks or so ;-). Mark