On Sat, 16 Sep 2006 20:13:53 -0400 (EDT) Enlightenment CVS
<[EMAIL PROTECTED]> babbled:

note e_object_free() frees with no respect for # of ref counts - so if someone
still has it referenced... you will be removing the object  from under them
without warning or safety...

> Enlightenment CVS committal
> 
> Author  : onefang
> Project : e17
> Module  : apps/e
> 
> Dir     : e17/apps/e/src/bin
> 
> 
> Modified Files:
>       e_apps.c e_border.c e_fm.c 
> 
> 
> Log Message:
> Tweak the E_App object reference counting.  I think this is correct, but
> I suspect the dir scanning in e_apps.c still needs an unref.
> 
> ===================================================================
> RCS file: /cvs/e/e17/apps/e/src/bin/e_apps.c,v
> retrieving revision 1.189
> retrieving revision 1.190
> diff -u -3 -r1.189 -r1.190
> --- e_apps.c  16 Sep 2006 22:48:01 -0000      1.189
> +++ e_apps.c  17 Sep 2006 00:13:52 -0000      1.190
> @@ -220,7 +220,7 @@
>        {
>           if(st.st_mtime > a->mtime)
>           {
> -            e_object_del(E_OBJECT(a));
> +            e_object_free(E_OBJECT(a));
>              a = NULL;
>              stated = 1;
>           }
> @@ -278,6 +278,9 @@
>       _e_apps_list = evas_list_prepend(_e_apps_list, a);
>  #endif
>       }
> +   else
> +      e_object_ref(E_OBJECT(a));
> +
>     return a;
>  
>  error:
> ===================================================================
> RCS file: /cvs/e/e17/apps/e/src/bin/e_border.c,v
> retrieving revision 1.543
> retrieving revision 1.544
> diff -u -3 -r1.543 -r1.544
> --- e_border.c        16 Sep 2006 05:38:09 -0000      1.543
> +++ e_border.c        17 Sep 2006 00:13:52 -0000      1.544
> @@ -2552,7 +2552,7 @@
>                 if (o)   evas_object_del(o);
>                 o = e_app_icon_add(evas, a);
>                 bd->app = a;
> -               e_object_ref(E_OBJECT(bd->app));
> +//             e_object_ref(E_OBJECT(bd->app));
>              }
>            else
>              {
> ===================================================================
> RCS file: /cvs/e/e17/apps/e/src/bin/e_fm.c,v
> retrieving revision 1.61
> retrieving revision 1.62
> diff -u -3 -r1.61 -r1.62
> --- e_fm.c    16 Sep 2006 13:07:55 -0000      1.61
> +++ e_fm.c    17 Sep 2006 00:13:52 -0000      1.62
> @@ -1611,7 +1611,10 @@
>                    app = e_app_new(buf, 0);
>                 if (app)
>                   {
> -                    ic->obj_icon = e_app_icon_add(evas_object_evas_get
> (ic->sd->obj), app); +/* FIXME: Actually, I think it's the icon searching
> that is slowing things down a lot.
> + * That will be fixed by moving it to thumbnailing as planned.  Commented
> out for now.
> + */
> +//                  ic->obj_icon = e_app_icon_add(evas_object_evas_get
> (ic->sd->obj), app); e_object_unref(E_OBJECT(app));
>                   }
>              }
> @@ -1698,7 +1701,10 @@
>                    app = e_app_new(buf, 0);
>                 if (app)
>                   {
> -                    ic->obj_icon = e_app_icon_add(evas_object_evas_get
> (ic->sd->obj), app); +/* FIXME: Actually, I think it's the icon searching
> that is slowing things down a lot.
> + * That will be fixed by moving it to thumbnailing as planned.  Commented
> out for now.
> + */
> +//                  ic->obj_icon = e_app_icon_add(evas_object_evas_get
> (ic->sd->obj), app); e_object_unref(E_OBJECT(app));
>                   }
>                 edje_object_part_swallow(ic->obj, "e.swallow.icon",
> ic->obj_icon);
> 
> 
> 
> -------------------------------------------------------------------------
> 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
> 


-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler)    [EMAIL PROTECTED]
裸好多
Tokyo, Japan (東京 日本)

-------------------------------------------------------------------------
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-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to