On 20/03/14 05:26, Gustavo Sverzut Barbieri wrote:
> On Wed, Mar 19, 2014 at 2:15 PM, Daniel Juyung Seo <seojuyu...@gmail.com> 
> wrote:
>
>>   static Eina_Bool
>> +#ifdef DEBUGON
>>   _x11_dnd_leave(void *data EINA_UNUSED, int etype EINA_UNUSED, void *ev)
>> +#else
>> +_x11_dnd_leave(void *data EINA_UNUSED, int etype EINA_UNUSED, void *ev 
>> EINA_UNUSED)
>> +#endif
>>   {
>>      cnp_debug("Leave %x\n", ((Ecore_X_Event_Xdnd_Leave *)ev)->win);
>
> better fix would be:
>
> #ifdef DEBUGON
> cnp_debug(...)
> #else
> (void)ev;
> #endif

An even better fix would be to define cnp_debug() to be an inline 
function that does nothing if DEBUGON is not defined, rather than an 
empty macro.

Mike



------------------------------------------------------------------------------
Learn Graph Databases - Download FREE O'Reilly Book
"Graph Databases" is the definitive new guide to graph databases and their
applications. Written by three acclaimed leaders in the field,
this first edition is now available. Download your free book today!
http://p.sf.net/sfu/13534_NeoTech
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to