From bc69e92dc041e8fa5a0d9968ae9fcf0b281c6673 Mon Sep 17 00:00:00 2001
From: Darshit Shah <address@hidden>
Date: Sat, 1 Nov 2014 15:25:01 +0530
Subject: [PATCH 1/2] Add the -Wextra flag by default

After the Spring Cleaning on warnings in early 2014, GCC and Clang emit
only about two warnings when compiling Wget with -Wall -Wextra.

Enable both these flags by default so that new warnings in these areas
are more prominent.
---
 ChangeLog    | 4 ++++
 configure.ac | 4 +++-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 767221d..26679ae 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 2014-11-16  Darshit Shah  <address@hidden>
 
+	* configure.ac: Add the -Wextra flag when no CFLAGS are set
+
+2014-11-16  Darshit Shah  <address@hidden>
+
 	* po/stamp-po: Remove autogenerated file from checked  out sources
 
 2014-11-16  Darshit Shah  <address@hidden>
diff --git a/configure.ac b/configure.ac
index 298aede..fea3fa3 100644
--- a/configure.ac
+++ b/configure.ac
@@ -165,9 +165,11 @@ dnl   if compiler is gcc, then
 dnl     use -O2 and some warning flags
 dnl   else
 dnl     use os-specific flags or -O
+dnl
+dnl For these tests, the system considers clang to be gcc
 if test -n "$auto_cflags"; then
   if test -n "$GCC"; then
-    CFLAGS="$CFLAGS -O2 -Wall"
+    CFLAGS="$CFLAGS -O2 -Wall -Wextra"
   else
     case "$host_os" in
       *hpux*)  CFLAGS="$CFLAGS +O3"                      ;;
-- 
2.1.3