diff options
| author | wm4 <wm4@nowhere> | 2013-11-01 13:00:15 +0100 |
|---|---|---|
| committer | wm4 <wm4@nowhere> | 2013-11-01 13:00:15 +0100 |
| commit | dd344b43e87ef5a24a4ba739821998605e38a015 (patch) | |
| tree | c0066b9e73fbb6f7ba5a46fed65db9a3a53a9695 /configure | |
| parent | 2696148d64364b40e7b0d1d3de017d6afce4b43d (diff) | |
| download | mpv-dd344b43e87ef5a24a4ba739821998605e38a015.tar.bz2 mpv-dd344b43e87ef5a24a4ba739821998605e38a015.tar.xz | |
Enable -Wshadow
This one really did bite me hard (see previous commit), so enable it by
default.
Fix some cases of shadowing throughout the codebase. None of these
change behavior, and all of these were correct code, and just tripped up
the warning.
Diffstat (limited to 'configure')
| -rwxr-xr-x | configure | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -1153,14 +1153,14 @@ if test -z "$CFLAGS" ; then WARNFLAGS="-wd167 -wd556 -wd144" elif test "$cc_vendor" = "clang"; then CFLAGS="$_opt $_debug $_pipe" - WARNFLAGS="-Wall -Wno-switch -Wno-logical-op-parentheses -Wpointer-arith -Wundef -Wno-pointer-sign -Wmissing-prototypes" + WARNFLAGS="-Wall -Wno-switch -Wno-logical-op-parentheses -Wpointer-arith -Wundef -Wno-pointer-sign -Wmissing-prototypes -Wshadow" ERRORFLAGS="-Werror=implicit-function-declaration -Wno-error=deprecated-declarations -Wno-error=unused-function" elif test "$cc_vendor" != "gnu" ; then CFLAGS="$_opt $_debug $_pipe" else CFLAGS="$_opt $_debug $_pipe" WARNFLAGS="-Wall -Wno-switch -Wno-parentheses -Wpointer-arith -Wredundant-decls" - ERRORFLAGS="-Werror-implicit-function-declaration -Wno-error=deprecated-declarations -Wno-error=unused-function" + ERRORFLAGS="-Werror-implicit-function-declaration -Wno-error=deprecated-declarations -Wno-error=unused-function -Wshadow" extra_ldflags="$extra_ldflags" fi else |