On Mon, Nov 10, 2008 at 10:55 PM, Gustavo Sverzut Barbieri <[EMAIL PROTECTED]> wrote: > On Mon, Nov 10, 2008 at 12:32 PM, Enlightenment SVN > <[EMAIL PROTECTED]> wrote: >> Log: >> Fix order of free during hash free. >> >> Author: cedric >> Date: 2008-11-10 06:32:14 -0800 (Mon, 10 Nov 2008) >> New Revision: 37572 >> >> Modified: >> trunk/eina/src/lib/eina_hash.c >> >> Modified: trunk/eina/src/lib/eina_hash.c >> =================================================================== >> --- trunk/eina/src/lib/eina_hash.c 2008-11-09 19:32:12 UTC (rev 37571) >> +++ trunk/eina/src/lib/eina_hash.c 2008-11-10 14:32:14 UTC (rev 37572) >> @@ -304,9 +304,9 @@ >> static void >> _eina_hash_el_free(Eina_Hash_El *el, Eina_Hash *hash) >> { >> - if (el->begin == EINA_FALSE) free(el); >> if (hash->data_free_cb) >> hash->data_free_cb(el->tuple.data); >> + if (el->begin == EINA_FALSE) free(el); >> } > > see how I changed eina_stringshare, you can save some space with this > bit there as well, or no (maybe you have other flags as well)... I'd > remove the flag anyway.
Yep, it's on the TODO, just no time to get at it :-) -- Cedric BAIL ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel