Re: Lists and aliasing (Re: Freeze on 27 February)

[Lennart Sorensen]

On Wed, Feb 22, 2012 at 11:25:22AM -0500, Lennart Sorensen wrote:
> That actually looks like a mistake.
> 
> HOST_CFLAGS="$HOST_CFLAGS $WARN_FLAGS"
> TARGET_CFLAGS="$TARGET_CFLAGS $WARN_FLAGS -g -Wredundant-decls 
> -Wmissing-prototypes"
> 
> but WARN_FLAGS="-Wall -W -Wshadow -Wold-style-declaration 
> -Wold-style-definition -Wpointer-arith -Wundef -Wextra -Waddress 
> -Warray-bounds -Wattributes -Wbuiltin-macro-redefined -Wcast-align 
> -Wchar-subscripts -Wclobbered -Wcomment -Wcoverage-mismatch -Wdeprecated 
> -Wdeprecated-declarations -Wdisabled-optimization -Wdiv-by-zero -Wempty-body 
> -Wendif-labels -Wfloat-equal -Wformat-contains-nul -Wformat-extra-args 
> -Wformat-security -Wformat-y2k -Wignored-qualifiers -Wimplicit 
> -Wimplicit-function-declaration -Wimplicit-int -Winit-self 
> -Wint-to-pointer-cast -Winvalid-pch -Wunsafe-loop-optimizations -Wlogical-op 
> -Wmain -Wmissing-braces -Wmissing-field-initializers 
> -Wmissing-format-attribute -Wmissing-noreturn -Wmudflap -Wmultichar -Wnonnull 
> -Woverflow -Wpacked-bitfield-compat -Wparentheses -Wpointer-arith 
> -Wpointer-to-int-cast -Wreturn-type -Wsequence-point -Wshadow -Wsign-compare 
> -Wstrict-aliasing -Wswitch -Wsync-nand -Wtrigraphs  -Wtype-limits -Wundef 
> -Wuninitialized -Wunknown-pragmas -Wunused -Wunused-function -Wun
> used-label -Wunused-parameter -Wunused-result -Wunused-value  
> -Wunused-variable -Wvariadic-macros -Wvolatile-register-var -Wwrite-strings 
> -Wmissing-declarations -Wmissing-parameter-type  -Wmissing-prototypes 
> -Wnested-externs -Wstrict-prototypes -Wpointer-sign"
> 
> So by having -Wmissing-prototypes in WARN_FLAGS it gets set twice in
> TARGET_CFLAGS and also set once in HOST_CFLAGS.  Perhaps that was
> not meant to be in there and should only have been in TARGET_CFLAGS.

So I removed -Wmissing-prototypes from configure.ac, and now instead I get:

gcc-4.6 -DHAVE_CONFIG_H -I. -I../..  -Wall -W -I./include -DGRUB_UTIL=1 
-DGRUB_LIBDIR=\"/usr/lib/grub\" -DLOCALEDIR=\"/usr/share/locale\"  
-DGRUB_MACHINE_EMU=1 -DGRUB_MACHINE=POWERPC_EMU -DGRUB_TARGET_CPU_POWERPC=1 
-DGRUB_FILE=\"util/grub-mkimage.c\" -I. -I../.. -I. -I../.. -I../../include 
-I./include -I./grub-core/gnulib -I../../grub-core/gnulib 
-DGRUB_PKGLIBROOTDIR=\"/usr/lib/grub\"   -g -Wall -O2 -Wall -W -Wshadow 
-Wold-style-declaration -Wold-style-definition -Wpointer-arith -Wundef -Wextra 
-Waddress -Warray-bounds -Wattributes -Wbuiltin-macro-redefined -Wcast-align 
-Wchar-subscripts -Wclobbered -Wcomment -Wcoverage-mismatch -Wdeprecated 
-Wdeprecated-declarations -Wdisabled-optimization -Wdiv-by-zero -Wempty-body 
-Wendif-labels -Wfloat-equal -Wformat-contains-nul -Wformat-extra-args 
-Wformat-security -Wformat-y2k -Wignored-qualifiers -Wimplicit 
-Wimplicit-function-declaration -Wimplicit-int -Winit-self 
-Wint-to-pointer-cast -Winvalid-pch -Wunsafe-loop-optimizations -Wlogical-op 
-Wmain -Wmissing-braces -Wmissing-field-initializers -Wmissing-format-attribute 
-Wmissing-noreturn -Wmudflap -Wmultichar -Wnonnull -Woverflow 
-Wpacked-bitfield-compat -Wparentheses -Wpointer-arith -Wpointer-to-int-cast 
-Wreturn-type -Wsequence-point -Wshadow -Wsign-compare -Wstrict-aliasing 
-Wswitch -Wsync-nand -Wtrigraphs  -Wtype-limits -Wundef -Wuninitialized 
-Wunknown-pragmas -Wunused -Wunused-function -Wunused-label -Wunused-parameter 
-Wunused-result -Wunused-value  -Wunused-variable -Wvariadic-macros 
-Wvolatile-register-var -Wwrite-strings -Wmissing-declarations 
-Wmissing-parameter-type -Wnested-externs -Wstrict-prototypes -Wpointer-sign 
-Wno-undef -Wno-sign-compare -Wno-unused -Wno-unused-parameter 
-Wno-redundant-decls -Wno-unreachable-code -Wno-conversion 
-Wno-old-style-definition   -c -o util/grub_mkimage-grub-mkimage.o `test -f 
'util/grub-mkimage.c' || echo '../../'`util/grub-mkimage.c
In file included from ../../util/grub-mkimage.c:45:0:
../../grub-core/gnulib/argp.h:610:1: warning: no previous declaration for 
'argp_usage' [-Wmissing-declarations]
../../grub-core/gnulib/argp.h:616:1: warning: no previous declaration for 
'_option_is_short' [-Wmissing-declarations]
../../grub-core/gnulib/argp.h:628:1: warning: no previous declaration for 
'_option_is_end' [-Wmissing-declarations]

Does -Wmissing-declarations have to go too?

-- 
Len Sorensen