On 22 July 2015 at 08:39, Michel Dänzer <[email protected]> wrote: > On 22.07.2015 07:55, Emil Velikov wrote: >> Inspired by a identical commit for xf86-video-nouveau by Ilia Mirkin. >> >> Signed-off-by: Emil Velikov <[email protected]> >> --- >> >> Compile tested only. >> >> src/radeon_dri2.c | 8 ++++++-- >> 1 file changed, 6 insertions(+), 2 deletions(-) >> >> diff --git a/src/radeon_dri2.c b/src/radeon_dri2.c >> index 7587a0c..814d751 100644 >> --- a/src/radeon_dri2.c >> +++ b/src/radeon_dri2.c >> @@ -443,8 +443,12 @@ radeon_dri2_copy_region2(ScreenPtr pScreen, >> if (translate && drawable->type == DRAWABLE_WINDOW) { >> PixmapPtr pPix = GetDrawablePixmap(drawable); >> >> - off_x = drawable->x - pPix->screen_x; >> - off_y = drawable->y - pPix->screen_y; >> +#ifdef COMPOSITE >> + off_x = - pPix->screen_x; >> + off_y = - pPix->screen_y; >> +#endif >> + off_x += drawable->x; >> + off_y += drawable->y; >> } > > There's more code which effectively relies on COMPOSITE being defined, > because it doesn't add drawable->x/y otherwise. Either that needs to be > fixed as well, or maybe we should just bail in configure if COMPOSITE > isn't defined. > Had a look at the COMPOSITE handling around and it involves more time than what I spend atm.
Feel free to tackle this any way that feels appropriate. I won't be able to look into this anytime soon, I'm afraid :( -Emil _______________________________________________ xorg-driver-ati mailing list [email protected] http://lists.x.org/mailman/listinfo/xorg-driver-ati
