[MPlayer-dev-eng] [PATCH] vo_directfb minor updates

[Jiri Svoboda]

Hi, 

>  > Yes. vo_directfb.c does not support any suboption.
> 
> OK, this should be mentioned in the docs, though.
> 
>  > It is outdated now and was designed for very old DirectFB version.
> 
> How about removing it then?  Is it still useful?

I vote for removing (and I stated this long time ago) ...

> 
>  > >  > -as I stated before most options are common between 
>  > >  >  vo_directfb and vo_dfbmga. Maybe it would be better to group 
>  > >  >  the togeather.
>  > > 
>  > > You just copied and pasted the descriptions from the 
> dfbmga section
>  > > anyway, so maybe is should point there.  Probably it 
> should be the
>  > > other way around since vo_directfb is the more general vo.
>  > 
>  > Yes and No. The most of option are same - but they have 
> other defaults.
>  > Also vo_directfb2 does not support matrox specific features.
> 
> OK, I'll think about if and how to unify them.

It is up to You :-) Of course I'll help.

		JS