From 5cc99db5bae8633f85559e5cdaef4cd905a4ee9c Mon Sep 17 00:00:00 2001
From: Lasse Collin <lasse.collin@tukaani.org>
Date: Wed, 1 Jul 2009 12:21:24 +0300
Subject: [PATCH] Avoid visibility related compiler warnings on Windows.

---
 configure.ac | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/configure.ac b/configure.ac
index b1961ec..2f3af3d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -554,21 +554,19 @@ lc_CPUCORES
 # If using GCC, set some additional AM_CFLAGS:
 ###############################################################################
 
-if test "x$GCC" = xyes ; then
+if test "$GCC" = yes ; then
 	echo
 	echo "GCC extensions:"
+fi
 
-	# Avoid checking for visibility support on Windows, because the test
-	# may succeed even though visibility isn't supported. Windows has
-	# a different way to export only the required symbols from the
-	# libraries.
-	if test "$is_w32" = no; then
-		gl_VISIBILITY
-		if test -n "$CFLAG_VISIBILITY" ; then
-			AM_CFLAGS="$AM_CFLAGS $CFLAG_VISIBILITY"
-		fi
-	fi
+# Always do the visibility check but don't set AM_CFLAGS on Windows.
+# This way things get set properly even on Windows.
+gl_VISIBILITY
+if test -n "$CFLAG_VISIBILITY" && test "$is_w32" = no; then
+	AM_CFLAGS="$AM_CFLAGS $CFLAG_VISIBILITY"
+fi
 
+if test "$GCC" = yes ; then
 	# Enable as much warnings as possible. These commented warnings won't
 	# work for this package though:
 	#   * -Wunreachable-code breaks several assert(0) cases, which are
-- 
2.39.2