[MPlayer-dev-eng] [PATCH] Fix for broken vo_xmga support in GUI mode

[Dirk Porezag]

Right. I don't see the 1 pixel off here. I'm currently looking into the issue 
with VO_MGA. Thanks for pointing out the typo 2 lines below my "fix" for non GUI 
VO_XMGA, should have seen this since I'm already aware that update_xinerama* 
does the necessary stuff.

I'll work with your patch and a svn updated tree. Have to update first.



----- Ursprüngliche Mail ----
> Von: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
> An: mplayer-dev-eng at mplayerhq.hu
> Gesendet: Freitag, den 8. Oktober 2010, 20:33:28 Uhr
> Betreff: Re: [MPlayer-dev-eng] [PATCH] Fix for broken vo_xmga support in GUI 
>mode
> 
> On Fri, Oct 08, 2010 at 08:13:29PM +0200, Reimar Döffinger wrote:
> > On  Fri, Oct 08, 2010 at 10:58:10AM +0200, Diego Biurrun wrote:
> > > On Thu,  Oct 07, 2010 at 08:21:01AM +0200, Reimar Döffinger wrote:
> > > > On  Thu, Oct 07, 2010 at 08:11:44AM +0200, Reimar Döffinger wrote:
> > > >  > I can have a look now that I know where the issues are.
> > > > 
> > > > Actually, I think it would be easier if you can tell me which  code
> > > > exactly breaks witch screen width/height = 0 and  why.
> > > > Attached is an updated version of the patch that should  work
> > > > right at least if you manually specify -screenwidth  -screenheight.
> > > 
> > > I get a 1-2 pixel colorkey (green)  border at the right and bottom sides
> > > of the window with your patch  applied.
> > 
> > With GUI? Without GUI? In fullscreen also?
> 
> And  note that unless you apply the other patch as well, movements
> will not work,  so you could get this effect if e.g. due to an
> off-by-one bug your window  manager move the window slightly
> at any  time.
> _______________________________________________
> MPlayer-dev-eng  mailing list
> MPlayer-dev-eng at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
>