[MPlayer-dev-eng] [PATCH] dsize negative scale values

[Trent Piepho]

On Sun, 3 Jul 2005, RC wrote:
> On Sun, 3 Jul 2005 22:36:32 +0300
> Oded Shimon <ods15 at ods15.dyndns.org> wrote:
> 
> > But, it's impossible to do the 'keepaspect' thing with -1 or -2:
> > say I have this - I want a "bounding box" of 800x600 - which would
> > work for ANY aspect, WITHOUT distorting. It's IMPOSSIBLE currently,
> > and would still be impossible with your patch. 
> 
> Well, with mine: "dsize=800:-2" would keep the dimentions under 800x600
> for any videos with any aspect of 4/3 (1.333...) or above.  I've never
> seen any videos with an aspect below 4/3, so I'd say you're very safe.

What if you had a 1280x720 widescreen LCD monitor?  You want the resolution as
large as possible but under 1280x720.  I've seen videos with apsects both less
than and greater than 16/9.

> > and with my patch you can still do anything offered in your patch:
> 
> I wish I could argue with you, but it will take me weeks to figure out
> how your dsize parameters are supposed to work.  

I do agree that all those dsize parameters were very complex.  Why would you
want something other than 0 = preserve aspect, 1 = don't preserve aspet?