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

Diego Biurrun diego at biurrun.de
Fri Mar 5 05:39:37 CET 2004


Jiri Svoboda writes:
 > > Please patch the man page and not the XML docs.  Option descriptions
 > > belong in the man page.
 > > Thanks
 > 
 > Here it goes...

thx

 > Just few coments (my fault I didn't read man page before):
 > -there is only -vo directfb and NO -vo directfb2 and never was.
 >  vo_directfb.c or vo_directfb2.c is choosen at configure/compile 
 >  time depending on the DirectFB version

Does vo_directfb.c support the same options?  A cursory glance through
the sources leads me to believe that vo_directfb.c has no suboptions..

 > -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.

Diego




More information about the MPlayer-dev-eng mailing list