Move -Wcast-qual from WARNS=2 to WARNS=3 (a new level), because it

generates too many false positives that require gross amounts of
workaround, and the other WARNS=2 stuff is quite useful. Per discussion
with simonb & christos.
This commit is contained in:
lukem 2001-11-01 07:17:17 +00:00
parent 058fbb840d
commit 0e643f61a5
2 changed files with 10 additions and 6 deletions

View File

@ -1,4 +1,4 @@
# $NetBSD: bsd.README,v 1.81 2001/10/30 15:17:16 wiz Exp $
# $NetBSD: bsd.README,v 1.82 2001/11/01 07:17:17 lukem Exp $
# @(#)bsd.README 8.2 (Berkeley) 4/2/94
This is the README file for the new make "include" files for the BSD
@ -545,8 +545,10 @@ yacc that allow multiple lex and yacc targets to be built in parallel.
Other variables of note (incomplete list):
WARNS Crank up gcc warning options; WARNS=1 and WARNS=2 are the two
distinct levels.
WARNS Crank up gcc warning options; the distinct levels are:
WARNS=1
WARNS=2
WARNS=3
FORMAT_AUDIT If FORMAT_AUDIT is set, and WFORMAT is set and > 1, turn on
WFORMAT -Wnetbsd-format-audit for extra-stringent format checking.

View File

@ -1,4 +1,4 @@
# $NetBSD: bsd.sys.mk,v 1.55 2001/10/19 19:07:48 tv Exp $
# $NetBSD: bsd.sys.mk,v 1.56 2001/11/01 07:17:17 lukem Exp $
#
# Overrides used for NetBSD source tree builds.
@ -11,8 +11,10 @@ CFLAGS+= -Wall -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith
CFLAGS+= -Wno-uninitialized
.endif
.if ${WARNS} > 1
CFLAGS+=-Wreturn-type -Wcast-qual -Wpointer-arith -Wwrite-strings
CFLAGS+=-Wswitch -Wshadow
CFLAGS+=-Wreturn-type -Wpointer-arith -Wwrite-strings -Wswitch -Wshadow
.endif
.if ${WARNS} > 2
CFLAGS+=-Wcast-qual
.endif
.endif