Configure damage must include shadow region, not just window. Also, leaking

    region on each configure.

diff --git a/ChangeLog b/ChangeLog
index 946788a..45faea4 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2003-11-09  Keith Packard  <keithp@keithp.com>
 
 2003-11-09  Keith Packard  <keithp@keithp.com>
 

+       * xcompmgr.c: (configure_win):
+       Configure damage must include shadow region, not just window.
+       Also, leaking region on each configure.
+
+2003-11-09  Keith Packard  <keithp@keithp.com>
+

        * xcompmgr.c: (main):
        oops.  Left in an XSynchronize call.
 
        * xcompmgr.c: (main):
        oops.  Left in an XSynchronize call.
 

diff --git a/xcompmgr.c b/xcompmgr.c
index 02a634d..16260b4 100644 (file)
--- a/xcompmgr.c
+++ b/xcompmgr.c
@@ -666,8 +666,9 @@ configure_win (Display *dpy, XConfigureEvent *ce)
     }
     if (damage)
     {
     }
     if (damage)
     {

-       XserverRegion   border = border_size (dpy, w);
-       XFixesUnionRegion (dpy, damage, damage, 0, 0, border, 0, 0);
+       XserverRegion   extents = win_extents (dpy, w);
+       XFixesUnionRegion (dpy, damage, damage, 0, 0, extents, 0, 0);
+       XFixesDestroyRegion (dpy, extents);

        add_damage (dpy, damage);
     }
 }
        add_damage (dpy, damage);
     }
 }