Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 29 Dec 2001 14:35:10 +0000
From:      David Malone <dwmalone@maths.tcd.ie>
To:        Bruce Evans <bde@zeta.org.au>
Cc:        "David E. O'Brien" <obrien@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/share/mk bsd.sys.mk 
Message-ID:   <200112291435.aa83853@salmon.maths.tcd.ie>
In-Reply-To: Your message of "Sun, 30 Dec 2001 01:21:29 %2B1100." <20011230011102.L5086-100000@gamplex.bde.org> 

next in thread | previous in thread | raw e-mail | index | archive | help
> > WARNS=4 doesn't seem to be the same as WARNS=2 used to be, because
> > -Wno-uninitialized isn't getting added at the end of the warnings
> > flags and so -Wuninitialized is still producing false positives.
> > Could someone review the patch below?

> Doesn't it work where it already is, since it is to cancel the
> -Wuninitialized provied by -Wall?

It doesn't seem to work - try setting WARNS=4 and then compiling
usr.bin/getcat. I've just checked and it seems that -W also sets
-Wuninitialised:

14:29:gonzo 29% gcc -O -W -c blah.c
blah.c: In function `f':
blah.c:2: warning: `i' might be used uninitialized in this function
14:29:gonzo 30% gcc -O -W -Wno-uninitialized -c blah.c
14:30:gonzo 31% cat blah.c
void f() { int i; g(i); }

Did the patch look OK, or is there some other way I should be doing
this?

	David.

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi? <200112291435.aa83853>