Enlightenment CVS committal

Author  : kwo
Project : e16
Module  : e

Dir     : e16/e/src


Modified Files:
        ewins.c 


Log Message:
Avoid setting shape to default (unshaped) on ShapeNotify when already set.
This fixes an event looping problem on Xsun where doing so generates
Shape (and Visibility) Notify events (Xsun bug?).
- Thanks again to Mark Bowyer for help to track this down.
Additional shape debug stuff.

===================================================================
RCS file: /cvsroot/enlightenment/e16/e/src/ewins.c,v
retrieving revision 1.138
retrieving revision 1.139
diff -u -3 -r1.138 -r1.139
--- ewins.c     7 Jan 2006 15:02:15 -0000       1.138
+++ ewins.c     12 Jan 2006 22:50:09 -0000      1.139
@@ -615,10 +615,10 @@
       EShapeCopy(ewin->win_container, _EwinGetClientWin(ewin));
    EUngrabServer();
 
-#if 0                          /* Debug */
-   Eprintf("EwinUpdateShapeInfo %#lx cont=%#lx shaped=%d\n",
-          _EwinGetClientXwin(ewin), ewin->win_container, ewin->client.shaped);
-#endif
+   if (EventDebug(EX_EVENT_SHAPE_NOTIFY))
+      Eprintf("EwinUpdateShapeInfo %#lx cont=%#lx shaped=%d\n",
+             _EwinGetClientXwin(ewin), _EwinGetContainerXwin(ewin),
+             ewin->state.shaped);
 }
 
 void
@@ -630,16 +630,16 @@
    if (ewin->state.docked)
       return;
 
-#if 0
-   Eprintf("EwinPropagateShapes %#lx %#lx %s\n", EoGetWin(ewin),
-          _EwinGetClientXwin(ewin), EoGetName(ewin));
-#endif
-   if (ewin->update.shape)
-     {
-       EShapePropagate(EoGetWin(ewin));
-       EoChangeShape(ewin);
-       ewin->update.shape = 0;
-     }
+   if (!ewin->update.shape)
+      return;
+
+   if (EventDebug(EX_EVENT_SHAPE_NOTIFY))
+      Eprintf("EwinPropagateShapes %#lx frame=%#lx shaped=%d\n",
+             _EwinGetClientXwin(ewin), EoGetWin(ewin), ewin->state.shaped);
+
+   EShapePropagate(EoGetWin(ewin));
+   EoChangeShape(ewin);
+   ewin->update.shape = 0;
 }
 
 void
@@ -1298,11 +1298,19 @@
 }
 
 static void
-EwinEventShapeChange(EWin * ewin)
+EwinEventShapeChange(EWin * ewin, XEvent * ev)
 {
+#define se ((XShapeEvent *)ev)
+   if (EventDebug(EX_EVENT_SHAPE_NOTIFY))
+      Eprintf("EwinEventShapeChange %#lx %s: state.shaped=%d ev->shaped=%d\n",
+             _EwinGetClientXwin(ewin), EoGetName(ewin), ewin->state.shaped,
+             se->shaped);
+   if (!se->shaped && !ewin->state.shaped)
+      return;
    EwinUpdateShapeInfo(ewin);
    ewin->update.shape = 1;
    EwinPropagateShapes(ewin);
+#undef se
 }
 
 static void
@@ -1933,7 +1941,7 @@
        break;
 
      case EX_EVENT_SHAPE_NOTIFY:
-       EwinEventShapeChange(ewin);
+       EwinEventShapeChange(ewin, ev);
        break;
 
      default:




-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
_______________________________________________
enlightenment-cvs mailing list
enlightenment-cvs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-cvs

Reply via email to