[MPlayer-dev-eng] [PATCH] automatically set monitor_aspect for x11 based vo's

[Alexander Strasser]

Hi,

Ivan Kalvachev wrote:
> 2006/2/13, René Rebe <rene at exactcode.de>:
> > Set the monitor aspect automatically for X displays as done by other drivers,
> > e.g. the OS X one. Formerly 4:3 was always assumted leading to obviously
> > false apsect on recent 16:9 / 16:10 desktop and laptop displays.
> >
> >   - Rene Rebe <rene at exactcode.de>
> >
> > --- MPlayer-1.0pre7try2/libvo/x11_common.c      2005-04-16 17:52:03.000000000 +0200
> > +++ MPlayer-1.0pre7try2-fixed/libvo/x11_common.c        2006-02-12 19:04:15.070424750 +0100
> > @@ -87,6 +87,8 @@
> >  static int vo_fs_flip = 0;
> >  char **vo_fstype_list;
> >
> > +extern float monitor_aspect;
> > +
> >  /* if equal to 1 means that WM is a metacity (broken as hell) */
> >  int metacity_hack = 0;
> >
> > @@ -510,6 +512,14 @@
> >             vo_screenwidth, vo_screenheight, depth, vo_depthonscreen,
> >             dispName, mLocalDisplay ? "local" : "remote");
> >
> > +    {
> > +        int w, h;
> > +        w = XDisplayWidthMM (mDisplay, mScreen);
> > +        h = XDisplayHeightMM (mDisplay, mScreen);
> > +        if (w && h)
> > +            monitor_aspect = (float) w / h;
> > +    }
> > +
> >      vo_wm_type = vo_wm_detect();
> >
> >      vo_fs_type = vo_x11_get_fs_type(vo_wm_type);
> 
> I think there was an older and much more mature patch for the same
> purpose. It handled override from the user (aka monitor-aspect
> option), and even rotation of the display...
> 
> Probably we have to rush beastd to approove it.

  Sorry, but I am in a hurry to get to work now :(

  About this aspect stuff I am really in favor of
such a change and the  spirit of the former patch
was basically ok but it had some open implementation
issues.

  However, i guess i can allocate some time on this
weekend to fix the problem at least partially.

  Alex (beastd)