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

[Diego Biurrun]

On Thu, Jun 07, 2007 at 06:53:57PM +0200, Carl Eugen Hoyos wrote:
> 
> 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.

I was going to ask, oh well ...

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

Just apply it.

> --- configure   (Revision 23494)
> +++ configure   (Arbeitskopie)
> @@ -2251,7 +2251,7 @@
> @@ -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

cc_check -Wno-pointer-sign && CFLAGS="-Wno-pointer-sign $CFLAGS"

Diego