[MPlayer-dev-eng] autoexpand patch

Joey Parrish joey at nicewarrior.org
Fri Nov 5 03:07:23 CET 2004


On Thu, Nov 04, 2004 at 12:30:18PM -0500, D Richard Felker III wrote:
> 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...

Right, right.  I was thinking of scale.  My mistake.  Is there any way
to use dsize to get the intended behavior, though?  Does expand have any
way of doing the automatic sizing like scale does?

--Joey

-- 
"Are disorders necessarily bad?" --Ike




More information about the MPlayer-dev-eng mailing list