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

[Arpi]

Hi,

> +		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?

> +#ifdef CONFIG_VIDIX
> +	if(vidix_name)
> +		return (vidix_query_fourcc(format));
> +#endif

looks like a 10l bug :)


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu