Hello.

On 02/16/2017 12:14 AM, Carsten Haitzler (The Rasterman) wrote:
> On Wed, 15 Feb 2017 10:08:29 -0800 Cedric BAIL <[email protected]> said:
>
>> On Wed, Feb 15, 2017 at 3:04 AM, Stefan Schmidt <[email protected]>
>> wrote:
>>> On 14/02/17 16:11, Stefan Schmidt wrote:
>>> emile_image.h, libemile.so.1.19.0
>>> emile_image_register ( Emile_Image* image, Emile_Action_Cb callback,
>>> Emile_Action action, void const* data )
>>>
>>> Maybe rename this to emile_image_cb_register?
>>
>> Hum, we usually use callback_add and callback_del for the name. I have
>> not implemented a callback_del and we only handle one callback at the
>> moment and not a list (As it wasn't necessary for the specific
>> behavior of that callback). I guess and I should go with our more
>> classic approach ?

If you plan to add a callback delete operation to this at some point you 
could go ahead and name it:

emile_image_callback_add

> callback_set?

Use this if you do not plan to add the delete functionality:

emile_image_callback_set

If we assume just one callback this is fine and a better name compared 
to what we have right now.

regards
Stefan Schmidt

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to