[MPlayer-dev-eng] patch for set right aspect ratio on playing with "-fs -vo vesa"

[Sascha Sommer]

On Tuesday 21 September 2004 12:12, Sascha Sommer wrote:
> On Sunday 19 September 2004 06:47, Attila Kinali wrote:
> > On Thu, Jul 22, 2004 at 10:19:34AM +0300, Alexander Buloichik wrote:
> > > Index: vo_vesa.c
> > > ===================================================================
> > > RCS file: /cvsroot/mplayer/main/libvo/vo_vesa.c,v
> > > retrieving revision 1.100
> > > diff -u -r1.100 vo_vesa.c
> > > --- vo_vesa.c	25 Jun 2004 17:29:18 -0000	1.100
> > > +++ vo_vesa.c	22 Jul 2004 07:16:08 -0000
> > > @@ -803,6 +803,11 @@
> > >  		      {
> > >  			dstW = video_mode_info.XResolution;
> > >  			dstH = video_mode_info.YResolution;
> > > +
> > > +		        aspect_save_orig(width,height);
> > > +			aspect_save_prescale(d_width,d_height);
> > > +			aspect_save_screenres(video_mode_info.XResolution,video_mode_info.Y
> > >Re solution); +			aspect(&dstW,&dstH,A_ZOOM);
> >
> > Anyone out there who would like to maintain vo vesa ?
>
> The one should also take into account that vo vesa can't scale without
> swscaler which gets activated with -zoom. In that case the nonvisible
> part of this if else construction gets executed which does exactly the same
> thing as in the patch, but maybe this patch is for use with vidix in this
> case maybe a simple
>
> 		if(use_scaler>1
> #ifdef CONFIG_VIDIX
>
> 		|| vidix_name
>
> #endif
> 		)
>
> would do? I'll commit this if noone complains.
>
> Better would be to remove swscaler from vo vesa all together, as it is
> still used for conversion. Not speaking about the console hiding mess in
> vbelib.c
>

Commited my solution for the aspect problem.

Sascha