[Mplayer-cvslog] CVS: main cfg-mencoder.h,1.62,1.63 mencoder.c,1.203,1.204

[Diego Biurrun]

D Richard Felker III writes:
 > On Fri, Jan 24, 2003 at 10:46:58PM +0100, Diego Biurrun wrote:
 > > Richard Felker CVS writes:
 > >  > +	{"autoexpand", &auto_expand, CONF_TYPE_FLAG, 0, 0, 1, NULL},
 > >  > +	{"noautoexpand", &auto_expand, CONF_TYPE_FLAG, 0, 1, 0, NULL},
 > > 
 > > Could you please document these?
 > 
 > Perhaps, if it's appropriate. They're really only for people who know
 > exactly what they're doing... Sometimes it's desirable to render
 > subtitles prior to scaling or cropping in a filter chain, and the way
 > mencoder used to work, it always inserted an extra expand filter to
 > render subtitles at the end. This resulted in totally broken output.
 > 
 > A better fix to the problem should probably be applied eventually, but
 > for now a way to disable the extra expand filter is needed for
 > advanced mencoder usage. I'm not sure if it should be documented,
 > though.

Then flag it as an advanced - you have to know what you are doing -
option, but IMHO no option should be missing from the manual, MPlayer
and MEncoder are used by a lot of technically savvy people.

Diego