summaryrefslogtreecommitdiffstats
path: root/configure
diff options
context:
space:
mode:
authorUoti Urpala <uau@mplayer2.org>2011-07-06 10:53:41 +0300
committerUoti Urpala <uau@mplayer2.org>2011-07-06 11:01:37 +0300
commitb7f574405350f93083fb423c34a21788684e10d4 (patch)
tree60ebcf3f128491f1ab6d980440f3789bd31af9f5 /configure
parent6075b4d4b2c7741171ce6a67de89143a8989660d (diff)
downloadmpv-b7f574405350f93083fb423c34a21788684e10d4.tar.bz2
mpv-b7f574405350f93083fb423c34a21788684e10d4.tar.xz
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-xconfigure5
1 files changed, 3 insertions, 2 deletions
diff --git a/configure b/configure
index 6d54e95f78..3b011a9a5f 100755
--- a/configure
+++ b/configure
@@ -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