[MPlayer-dev-eng] [PATCH] vidix with fbdev

[Felix Buenemann]

On Tuesday 30 July 2002 20:47, Arpi wrote:
> > +             if(zoom > 1 || fs)
> >               {
> >                       aspect_save_orig(width,height);
> >                       aspect_save_prescale(d_width,d_height);
> >                       aspect_save_screenres(fb_xres,fb_yres);
> >                       aspect(&image_width,&image_height,A_ZOOM);
> >               }
>
> ok it was bad, and is still bad, i've fixed it will commit soon
> please someone test it...
>
> ok, now the reason of this mail... :)
>
> i've looked at Atmos' aspect() code, and found that its parameter xoom
> (A_ZOOM or A_NOZOOM) is a bit confusing - or at least i'm confused :)
>
> if i understood that code right, A_ZOOM means fullscreen and A_NOZOOM means
> d_width/d_height size. so, NOZOOM also zooms, but to the requested size
> instead of fullscreen... this way it should be named A_ZOOM and
> A_FULLSCREEN or so... right?
A_ZOOM means zoom to fullscreen while aspect correcting, A_NOZOOM means only 
aspect correct, but do not zoom to full screen area, eg. when displaying a 
movie on a 16:9 screen (-monitoraspect 16:9), A_NOZOOM would only scale the 
movie to still be 4:3, but A_ZOOM would additionally zoom it to full screen 
width or height (depending on what fits better).
-- 
Best Regards,
        Atmos
____________________________________________
- MPlayer Developer - http://mplayerhq.hu/ -
____________________________________________