Take -Wconversion out of BDECFLAGS. It is not particularly useful for
us anyway because it doesn't work right on the x86 and alpha. On K&R code, small ints would be promoted to int. ANSI-C doesn't require this and the small ints can be passed taking 8 or 16 bits of stack space. However, the x86 abi that we use *does* promote to 32 bit, and the alpha ABI passes them in 64 bit registers so we dont have that aspect of the problem here. Losing float precision by having it cast down to int because the funtion prototype specifies int is the least of our problems. -Wmissing-prototypes helps here anyway.
This commit is contained in:
parent
d13600a5af
commit
791da841a1
@ -50,10 +50,11 @@
|
||||
#
|
||||
# BDECFLAGS are a set of gcc warning settings that Bruce Evans has suggested
|
||||
# for use in developing FreeBSD and testing changes. They can be used by
|
||||
# putting "CFLAGS+=${BDECFLAGS}" in /etc/make.conf.
|
||||
# putting "CFLAGS+=${BDECFLAGS}" in /etc/make.conf. -Wconversion is not
|
||||
# included here due to compiler bugs, eg: mkdir()'s mode_t argument.
|
||||
#
|
||||
BDECFLAGS= -W -Wall -ansi -pedantic -Wbad-function-cast -Wcast-align \
|
||||
-Wcast-qual -Wchar-subscripts -Wconversion -Winline \
|
||||
-Wcast-qual -Wchar-subscripts -Winline \
|
||||
-Wmissing-prototypes -Wnested-externs -Wpointer-arith \
|
||||
-Wredundant-decls -Wshadow -Wstrict-prototypes -Wwrite-strings
|
||||
#
|
||||
|
@ -50,10 +50,11 @@
|
||||
#
|
||||
# BDECFLAGS are a set of gcc warning settings that Bruce Evans has suggested
|
||||
# for use in developing FreeBSD and testing changes. They can be used by
|
||||
# putting "CFLAGS+=${BDECFLAGS}" in /etc/make.conf.
|
||||
# putting "CFLAGS+=${BDECFLAGS}" in /etc/make.conf. -Wconversion is not
|
||||
# included here due to compiler bugs, eg: mkdir()'s mode_t argument.
|
||||
#
|
||||
BDECFLAGS= -W -Wall -ansi -pedantic -Wbad-function-cast -Wcast-align \
|
||||
-Wcast-qual -Wchar-subscripts -Wconversion -Winline \
|
||||
-Wcast-qual -Wchar-subscripts -Winline \
|
||||
-Wmissing-prototypes -Wnested-externs -Wpointer-arith \
|
||||
-Wredundant-decls -Wshadow -Wstrict-prototypes -Wwrite-strings
|
||||
#
|
||||
|
Loading…
Reference in New Issue
Block a user