[MPlayer-dev-eng] [PATCH] Adding -Wno-pointer-sign to configure

[Carl Eugen Hoyos]

Hi Diego!

Diego Biurrun wrote:
> On Thu, Jun 07, 2007 at 06:33:08PM +0200, Carl Eugen Hoyos wrote:
>> Following inlined patch adds Wno-pointer-sign to configure.
>>
>> Ok?
> 
> Fine by me.  CFLAGS handling in configure is in for an overhaul...

I applied it, but it was wrong, because it breaks gcc 2.95.

May I apply following patch, or should I revert and apply only the
second part (or only revert it)?

Carl Eugen

Index: configure
===================================================================
--- configure   (Revision 23494)
+++ configure   (Arbeitskopie)
@@ -2251,7 +2251,7 @@
   elif test "$cc_vendor" != "gnu" ; then
     CFLAGS="-O2 $_march $_mcpu $_pipe"
   else
-    CFLAGS="-Wall -Wno-pointer-sign -O4 $_march $_mcpu $_pipe
-ffast-math -fomit-frame-pointer"
+    CFLAGS="-Wall -O4 $_march $_mcpu $_pipe -ffast-math
-fomit-frame-pointer"
   fi
 else
   _warn_CFLAGS=yes
@@ -7397,6 +7397,9 @@
 if cc_check -Wdeclaration-after-statement ; then
   CFLAGS="-Wdeclaration-after-statement $CFLAGS"
 fi
+if cc_check -Wno-pointer-sign ; then
+  CFLAGS="-Wno-pointer-sign $CFLAGS"
+fi


 #this must be the last test to be performed or the ones following it
will likely fail