[MPlayer-dev-eng] [PATCH] support for external VIDIX

[Nico Sabbi]

Diego Biurrun wrote:

>>+else
>>+  _external_vidix=no
>>    
>>
>
>The else clause is superfluous, you've already set _external_vidix=no
>above.
>  
>
no, it handles the case --disable-internal-vidix --disable-external-vidix,
otherwise disabling vidix entirely would not be possible


>I don't particularly like the _has_vidix variable.  Instead you could
>rename _vidix to _vidix_internal and use _vidix for the common case.
>
>Diego
>  
>

updated
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vidix4.diff
Type: text/x-patch
Size: 4823 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20060326/134a9f35/attachment.bin>