[MPlayer-dev-eng] autoexpand patch

[D Richard Felker III]

On Thu, Nov 04, 2004 at 09:18:10AM -0600, Joey Parrish wrote:
> On Thu, Nov 04, 2004 at 11:48:51AM +0100, Arpi wrote:
> > > > It might seem like a hack, but how about just just using the "w" value 
> > > > as a decimal value?
> > > > For instance, -vf expand=1.33 would expand to a 4:3 ratio aspect...
> > > > 
> > > > How about that?
> > > 
> > > it's a pain. it could allow fraction (4/3, etc.) and that might be
> > > more useful.
> > 
> > why not add a new (6th) option to expand, so someone could do things
> > like:
> > -vf expand=:600:::4/3  to get the video expanded to 600 height, and
> > the width calculated from the given 4:3 aspect ratio.
> > also if someone only wants aspect set, do -vf expand=aspect=4/3
> > (it's handled by subconfig parser, equal to expand=:::::4/3)
> 
> Why not -vf dsize=4/3,expand=-2:600 ?  Isn't this the same?

nope, totally the opposite. it will mess up the aspect. you want to
expand the _destination_ to 4:3, not force the source to be
interpreted as 4:3 already.

also the meaning of -2 for vf_expand is to add a 2 pixel border...

rich