On Wed, Jun 15, 2011 at 9:38 PM, Ulrich Eckhardt <dooms...@knuut.de> wrote:
> On Wednesday 15 June 2011 18:06:03 Cedric BAIL wrote:
>> On Wed, Jun 15, 2011 at 5:53 PM, Mike Blumenkrantz <m...@zentific.com>
> wrote:
>> > On Wed, 15 Jun 2011 08:46:11 -0700
>> >
>> >> /**
>> >> + * @brief Redefine the callback that clean the data of a hash
>> >> + *
>> >> + * @param hash The given hash table
>> >> + * @param data_free_cb The function called on each value when the hash
>> >> + * table is freed. @c NULL can be passed as callback.
>> >> +EAPI void
>> >> +eina_hash_free_set(Eina_Hash *hash, Eina_Free_Cb data_free_cb)
>> >> +{
>> >> + EINA_MAGIC_CHECK_HASH(hash);
>> >> + EINA_SAFETY_ON_NULL_RETURN(hash);
>> >> +
>> >> + hash->data_free_cb = data_free_cb;
>> >> +}
>> >> +
> [...]
>> > I feel like this name is somewhat misleading...
>>
>> I didn't like it either, if you have a better proposal...
>
> Call the thing "deallocator".
>
>
> Question:
> - If that thing is NULL, does that mean the memory isn't released or does it
> use free() as a default deallocation function? Is there a default function
> already set?
> - Have you considered a void pointer as context to the deallocation function?
It's not a new feature, it just give the possibility to set the
free_cb after the creation of the Eina_Hash. So nothing really fancy
here.
--
Cedric BAIL
------------------------------------------------------------------------------
EditLive Enterprise is the world's most technically advanced content
authoring tool. Experience the power of Track Changes, Inline Image
Editing and ensure content is compliant with Accessibility Checking.
http://p.sf.net/sfu/ephox-dev2dev
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
