[MPlayer-cvslog] CVS: main/libmpcodecs vf_scale.c,1.63,1.64

[Oded Shimon]

On Fri, Feb 24, 2006 at 04:23:01PM +0100, Michael Niedermayer CVS wrote:
> CVS change done by Michael Niedermayer CVS
> 
> Update of /cvsroot/mplayer/main/libmpcodecs
> In directory mail:/var2/tmp/cvs-serv11714
> 
> Modified Files:
> 	vf_scale.c 
> Log Message:
> 10l (dont limit dimension components independantly if noup)

I didn't really consider it a bug, just behavior. it still doesn't fix the 
issue with aspect ratio mpeg-2, or intentional breaking of aspect ratio...

The only way I see for this is by looking at the aspect ratio of the 
requested width/height, and if they are upscaling, downsize both of them to 
keep the correct aspect. But I really don't care about any of this..

> +        if((vf->priv->w > width) + (vf->priv->h > height) >= vf->priv->noup){

obfuscation or feature?..

noup=1 means just one dimention needs to be upscaled to reverse
noup=2 means BOTH have to be upscaled to reverse...

funny code

- ods15