[MPlayer-dev-eng] Re: [PATCHES] iec958 and unichrome

[Remus Koos]

  Hi Diego !

> version.h is included by mistake, there are some cosmetics in the patch,
> plus
> 
>   #if defined(HAVE_XVMC)|defined(HAVE_XVMC_VLD)
> 
> is incorrect syntax, use
> 
>   #if defined(HAVE_XVMC) || defined(HAVE_XVMC_VLD)
> 
> instead and yes, please also add the spaces, it makes it more readable.

  Ok, it's fixed. Try again at the same location. It applies and compiles. I
haven't tested the binary yet. 
http://remus.gotdns.com/the2nd//mplayer-1.0pre8-unichrome-0.5.patch

  Please note that I'm not the maintainer of the patch. I barely ported it to
1.0pre8. The code that was there in the pre6 patch ended up in the ported
version ;-) ...

> Dunno about the rest of the patch, Ivan should review it.  I remember
> that Guillaume was interested in Via xvmc support as well.

  What about the iec958 patch ? Could this be included in the mainstream
mplayer ?

    remus

__________________________________________________
Do You Yahoo!?
Tired of spam?  Yahoo! Mail has the best spam protection around 
http://mail.yahoo.com