Hello all,

The patch addresses two problems:
1.  Ecore Timer complaints (ECORE MAGIC failures).  I ran a simple
test program with gdb and found it's related to it->long_timer.  The
complaints happen from time to time but it is not very easy to
reproduce.  The patch addresses the only suspicious bit I found by
code inspection.
2.  When a genlist is dragged around, items that are supposed to be
off the screen often appear as though they are left behind.  The patch
removes the check for it->dragging before _item_unrealize().  I'm not
sure what the check is for.  I tested the patch and did not notice any
side effects.

Please have a look at the attached patch and correct it if you see fit.

Cheers,


brian

-- 
brian
------------------

Cool-Karaoke - The smallest recording studio, in your palm, open-sourced
http://cool-idea.com.tw/

iMaGiNaTiOn iS mOrE iMpOrTaNt tHaN kNoWlEdGe
Index: elm_genlist.c
===================================================================
--- elm_genlist.c       (revision 47380)
+++ elm_genlist.c       (working copy)
@@ -938,7 +938,11 @@
    Evas_Object *icon;
 
    if (!it->realized) return;
-   if (it->long_timer) ecore_timer_del(it->long_timer);
+   if (it->long_timer)
+     {
+       ecore_timer_del(it->long_timer);
+       it->long_timer = NULL;
+     }
    evas_object_del(it->base);
    it->base = NULL;
    evas_object_del(it->spacer);
@@ -1043,12 +1047,7 @@
    if (!itb->realized) return;
    EINA_LIST_FOREACH(itb->items, l, it)
      {
-        if (it->dragging)
-          {
-             dragging = 1;
-          }
-        else
-          _item_unrealize(it);
+       _item_unrealize(it);
      }
    if (!dragging)
      itb->realized = EINA_FALSE;
@@ -1364,7 +1363,7 @@
                                cvx, cvy, cvw, cvh))
          {
             if ((!itb->realized) || (itb->changed))
-               _item_block_realize(itb, in, 0);
+              _item_block_realize(itb, in, 0);
             _item_block_position(itb,  in);
          }
        else
------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to