Enlightenment CVS committal

Author  : kwo
Project : e16
Module  : e

Dir     : e16/e/src


Modified Files:
        eobj.c x.c 


Log Message:
Cleanups, issue warning on missing unregistration.

===================================================================
RCS file: /cvs/e/e16/e/src/eobj.c,v
retrieving revision 1.73
retrieving revision 1.74
diff -u -3 -r1.73 -r1.74
--- eobj.c      29 Apr 2006 19:39:21 -0000      1.73
+++ eobj.c      2 May 2006 20:06:32 -0000       1.74
@@ -216,7 +216,11 @@
       Eprintf("EobjFini: %#lx %s\n", EobjGetXwin(eo), eo->name);
 
    EobjListStackDel(eo);
-   if (!eo->external)
+   if (eo->external)
+     {
+       EUnregisterWindow(eo->win);
+     }
+   else
      {
        EDestroyWindow(eo->win);
        eo->gone = 1;
===================================================================
RCS file: /cvs/e/e16/e/src/x.c,v
retrieving revision 1.129
retrieving revision 1.130
diff -u -3 -r1.129 -r1.130
--- x.c 29 Apr 2006 19:39:21 -0000      1.129
+++ x.c 2 May 2006 20:06:32 -0000       1.130
@@ -216,15 +216,7 @@
 
    xid = EXidFind(win);
    if (!xid)
-     {
-       ERegisterWindow(Xwin(win));     /* FIXME - We shouldn't go here */
-       xid = EXidFind(win);
-       if (!xid)
-         {
-            Eprintf("EventCallbackRegister win=%#lx ???\n", Xwin(win));
-            return;
-         }
-     }
+      return;
 #if 0
    Eprintf("EventCallbackRegister: %p %#lx\n", xid, win);
 #endif
@@ -674,8 +666,14 @@
    EXID               *xid;
 
    xid = EXidLookup(xwin);
-   if (xid)
-      EXidDel(xid);            /* FIXME - We shouldn't go here */
+   if (!xid)
+      return;
+
+   /* FIXME - We shouldn't go here */
+   EXidDel(xid);
+#if 1                          /* Debug - Fix code if we get here */
+   Eprintf("*** FIXME - EUnregisterXwin %#lx\n", xwin);
+#endif
 }
 
 void




-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
enlightenment-cvs mailing list
enlightenment-cvs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-cvs

Reply via email to