No idea ... I was just fixing a compiler warning :) Have to discuss that 
w/ who ever wrote ecore_anim :)

dh

On 05/11/2010 10:10 AM, Gustavo Sverzut Barbieri wrote:
> On Tue, May 11, 2010 at 11:01 AM, Enlightenment SVN
> <[email protected]>  wrote:
>> Log:
>>   Fix 'return' with a value in function returning void.
>>
>>
>> Author:       devilhorns
>> Date:         2010-05-11 07:01:42 -0700 (Tue, 11 May 2010)
>> New Revision: 48754
>>
>> Modified:
>>   trunk/ecore/src/lib/ecore/ecore_anim.c
>>
>> Modified: trunk/ecore/src/lib/ecore/ecore_anim.c
>> ===================================================================
>> --- trunk/ecore/src/lib/ecore/ecore_anim.c      2010-05-11 13:16:12 UTC (rev 
>> 48753)
>> +++ trunk/ecore/src/lib/ecore/ecore_anim.c      2010-05-11 14:01:42 UTC (rev 
>> 48754)
>> @@ -149,7 +149,7 @@
>>       {
>>         ECORE_MAGIC_FAIL(animator, ECORE_MAGIC_ANIMATOR,
>>                          "ecore_animator_del");
>> -       return NULL;
>> +       return;
>>       }
>>     if (animator->delete_me) return;
>>     animator->suspended = EINA_TRUE;
>> @@ -170,7 +170,7 @@
>>       {
>>         ECORE_MAGIC_FAIL(animator, ECORE_MAGIC_ANIMATOR,
>>                          "ecore_animator_del");
>> -       return NULL;
>> +       return;
>>       }
>>     if (animator->delete_me) return;
>>     animator->suspended = EINA_FALSE;
>
> for consistency with timer, maybe it should return the associated data?
>
>
>


------------------------------------------------------------------------------

_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to