Re: Re: [COMMITTERS] pgsql: Don't need -Wno-error anymore, because flex is no longer - Mailing list pgsql-hackers

From Kurt Roeckx
Subject Re: Re: [COMMITTERS] pgsql: Don't need -Wno-error anymore, because flex is no longer
Date
Msg-id 20080319215534.GA13268@roeckx.be
Whole thread Raw
In response to Re: Re: [COMMITTERS] pgsql: Don't need -Wno-error anymore, because flex is no longer  (Peter Eisentraut <peter_e@gmx.net>)
List pgsql-hackers
On Tue, Mar 18, 2008 at 07:23:40PM +0100, Peter Eisentraut wrote:
> Am Dienstag, 18. März 2008 schrieb Bruce Momjian:
> > Peter Eisentraut wrote:
> > > Log Message:
> > > -----------
> > > Don't need -Wno-error anymore, because flex is no longer producing
> > > warnings.
> >
> > I see this patch only affects ecpg?  Is this related to Tom's report
> > from trying the new flex?
> 
> Yes.  There used to be a few -Wno-error uses throughout the source code, near 
> the flex calls, to work around the well-known flex warnings issue in case 
> someone wanted to use -Werror to check his builds.  The fact that there was 
> only one such use left in the ecpg subtree shows that this wasn't really 
> maintained and could never have worked reliably.

There currently are a few warnings (with gcc 4.3.0):
bootscanner.c:1461: warning: 'input' defined but not used
scan.c:6670: warning: 'input' defined but not used
guc-file.c:1233: warning: 'input' defined but not used
tuplesort.c:2475: warning: inlining failed in call to 'myFunctionCall2': --param large-stack-frame-growth limit
reached
tuplesort.c:2526: warning: called from here
tuplesort.c:2475: warning: inlining failed in call to 'myFunctionCall2': --param large-stack-frame-growth limit
reached
tuplesort.c:2526: warning: called from here
[...]
tqual.c: In function 'HeapTupleSatisfiesVacuum':
tqual.c:115: warning: inlining failed in call to 'SetHintBits': call is
unlikely and code size would grow
tqual.c:1084: warning: called from here
tqual.c:115: warning: inlining failed in call to 'SetHintBits': call is
unlikely and code size would grow
tqual.c:1088: warning: called from here
[...]
pgc.c:3507: warning: 'input' defined but not used
psqlscan.c:3853: warning: 'input' defined but not used
pl_scan.c:2458: warning: 'input' defined but not used


Kurt



pgsql-hackers by date:

Previous
From: Simon Riggs
Date:
Subject: Re: Final Thoughts for 8.3 on LWLocking and Scalability
Next
From: Magnus Hagander
Date:
Subject: Re: [COMMITTERS] pgsql: Enable probes to work with Mac OS X Leopard and other OSes that