From: Dana Jansens <danakj@orodu.net>
Date: Sat, 16 Oct 2010 21:13:27 +0000 (-0400)
Subject: don't use -Wextra unless --enable-super-warnings
X-Git-Url: https://git.brokenzipper.com/gitweb?a=commitdiff_plain;h=d291a9c7aa685b6d8bd6aa70cdc543926702c301;p=chaz%2Fopenbox

don't use -Wextra unless --enable-super-warnings
---

diff --git a/m4/openbox.m4 b/m4/openbox.m4
index 862c6891..0c37a584 100644
--- a/m4/openbox.m4
+++ b/m4/openbox.m4
@@ -11,6 +11,10 @@ AC_DEFUN([OB_DEBUG],
     AC_HELP_STRING([--enable-strict-ansi],[Enable strict ANSI compliance build [[default=no]]]),
     [STRICT=$enableval], [STRICT="no"])
 
+    AC_ARG_ENABLE([super-warnings],
+    AC_HELP_STRING([--enable-super-warnings],[Enable extra compiler warnings [[default=no]]]),
+    [SUPERWARN=$enableval], [SUPERWARN="no"])
+
     AC_ARG_ENABLE([debug],
     AC_HELP_STRING([--enable-debug],[build a debug version [[default=no]]]),
     [DEBUG=$enableval], [DEBUG="no"])
@@ -43,6 +47,9 @@ AC_DEFUN([OB_DEBUG],
     if test "$STRICT" = "yes"; then
 	MSG="$MSG with strict ANSI compliance"
     fi
+    if test "$SUPERWARN" = "yes"; then
+	MSG="$MSG with super warnings"
+    fi
     AC_MSG_RESULT([$MSG])
     
     test "$DEBUG" = "yes" && \
@@ -77,12 +84,14 @@ AC_DEFUN([OB_COMPILER_FLAGS],
 	    FLAGS="$FLAGS -O0 -ggdb -fno-inline -Wwrite-strings"
 	    FLAGS="$FLAGS -Wall -Wsign-compare -Waggregate-return"
 	    FLAGS="$FLAGS -Wbad-function-cast -Wpointer-arith"
-	    FLAGS="$FLAGS -Wno-write-strings -Wextra"
-	    # glib can't handle this flag
-	    # -Wcast-qual
+	    FLAGS="$FLAGS -Wno-write-strings"
             # for Python.h
 	    #FLAGS="$FLAGS -Wno-long-long"
 	fi
+	if test "$SUPERWARN" = "yes"; then
+	    # glib can't handle -Wcast-qual
+	    FLAGS="$FLAGS -Wcast-qual -Wextra"
+	fi
 	if test "$STRICT" = "yes"; then
 	    FLAGS="$FLAGS -ansi -pedantic -D_XOPEN_SOURCE"
 	fi