On Wed, 2011-11-02 at 21:33 -0700, Keith Packard wrote: > On Tue, 1 Nov 2011 21:32:36 +0000, Rui Matos <tiagoma...@gmail.com> wrote: > > > - if ((nx >= left) && (nx <= right) && (ny >= top) && (ny <= bottom)) > > { > > - if ((*x <= left) || (*x >= right)) { > > - int dx = *x - nx; > > - > > - if (dx > 0) > > - *x = right; > > - else if (dx < 0) > > - *x = left; > > - } > > - > > - if ((*y <= top) || (*y >= bottom)) { > > - int dy = *y - ny; > > - > > - if (dy > 0) > > - *y = bottom; > > - else if (dy < 0) > > - *y = top; > > - } > > Ajax: Can you please try to explain this logic and why it isn't > equivalent to the obvious replacement provided in this patch? > > I can't figure out why the fancy bits are any different, given the > surrounding constraints (nx and ny inside the CRTC, x and/or y outside > the CRTC). > > Either I'm missing some subtlety, or ajax was off his meds when he wrote > this stuff.
I would assume I was simply wrong, yeah. - ajax
signature.asc
Description: This is a digitally signed message part
_______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel