diff options
author | Uoti Urpala <uau@mplayer2.org> | 2011-07-06 10:53:41 +0300 |
---|---|---|
committer | Uoti Urpala <uau@mplayer2.org> | 2011-07-06 11:01:37 +0300 |
commit | b7f574405350f93083fb423c34a21788684e10d4 (patch) | |
tree | 60ebcf3f128491f1ab6d980440f3789bd31af9f5 /configure | |
parent | 6075b4d4b2c7741171ce6a67de89143a8989660d (diff) |
configure: Use -Werror-implicit-function-declaration with GCC
Add -Werror-implicit-function-declaration to the default compiled
flags used with GCC. Add the option through a new variable ERRORFLAGS
instead of the existing WARNFLAGS to avoid using it in configure
tests. I think it's overall preferable not to fail tests because of
the warning - in some case this could cause a compilation failure
later, but on the other hand it could be just an unreliable test
triggering the warning and even if it does fail those are likely cases
worth closer investigation.
Diffstat (limited to 'configure')
-rwxr-xr-x | configure | 5 |
1 files changed, 3 insertions, 2 deletions
@@ -2321,6 +2321,7 @@ elif test -z "$CFLAGS" ; then else CFLAGS="-O2 $_march $_mcpu $_pipe -ffast-math -fomit-frame-pointer" WARNFLAGS="-Wall -Wno-switch -Wno-parentheses -Wpointer-arith -Wredundant-decls" + ERRORFLAGS="-Werror-implicit-function-declaration" extra_ldflags="$extra_ldflags -ffast-math" fi else @@ -6654,8 +6655,8 @@ INSTALL = $_install INSTALLSTRIP = $_install_strip WINDRES = $_windres -CFLAGS = $WARNFLAGS $WARN_CFLAGS $CFLAGS $extra_cflags -CXXFLAGS = $WARNFLAGS $CXXFLAGS $extra_cflags $extra_cxxflags +CFLAGS = $WARNFLAGS $ERRORFLAGS $WARN_CFLAGS $CFLAGS $extra_cflags +CXXFLAGS = $WARNFLAGS $ERRORFLAGS $CXXFLAGS $extra_cflags $extra_cxxflags DEPFLAGS = $DEPFLAGS CFLAGS_LIBDVDCSS = $cflags_libdvdcss |