commit 4145bf05ae1fae259593ef8f025795d77db5a4fc
author Urvang Joshi <urvang@google.com> Mon Oct 17 14:53:33 2016 -0700
committer Yaowu Xu <yaowu@google.com> Wed Oct 19 07:56:53 2016 -0700
tree 0b58135a053a530570d311dde1f5e44d50f48cbe
parent 88a03bb68f24c8d1c2469dc3d2956bcfe515c3b7

Fix warnings reported by -Wshadow: Part4: main directory

Now that all warnings are taken care of, add warning flag -Wshadow to
configure.

Note: Enabling this flag for C++ generates some useless warnings about
some function parameters shadowing class member function names. So, only
enabling this warning for C code.

Cherry-picked from aomedia/master: b96cbc4

Change-Id: I3922dea2e6976b16519c4aa4d1bd395c198134f1

configure

diff --git a/configure b/configure
index d911d9a..53ba10e 100755
--- a/configure
+++ b/configure
@@ -611,6 +611,10 @@
         check_add_cflags -Wuninitialized
         check_add_cflags -Wunused-variable
         check_add_cflags -Wsign-compare
+        # Enabling the following warning for C++ generates some useless warnings
+        # about some function parameters shadowing class member function names.
+        # So, only enable this warning for C code.
+        check_cflags "-Wshadow" && add_cflags_only "-Wshadow"
         case ${CC} in
           *clang*) ;;
           *) check_add_cflags -Wunused-but-set-variable ;;