On Thu, Feb 25, 2010 at 3:29 PM, Vincent Torri <vto...@univ-evry.fr> wrote:
> On Thu, 25 Feb 2010, Gustavo Sverzut Barbieri wrote:
>
>> On Thu, Feb 25, 2010 at 9:19 AM, Enlightenment SVN
>> <no-re...@enlightenment.org> wrote:
>>>
>>> Log:
>>>  add lots of logging functions - for trackign x overhead when u cant get
>>>  symbols... booo! - disabled of course.
>>
>> damn raster, if you take so much time to do all this work, why don't
>> you take 5 minutes more to check out eina's logging API?
>>
>> If you did it, it would be clear that overhead would not be added,
>> you'd benefit from configurable logging and would avoid duplication of
>> code.
>>
>> If one want to avoid overhead with eina logging, just create another
>> domain or level and use the same as eina's  "maximum log level".
>>
>> Anyway, let's try to avoid such things in future.
>
> no : raster should modify what he did. Otherwise it will be a mess. In
> addition, eina log will do exactly what he wants

not just that, but would avoid replicating __FILE__, __LINE__ and
__FUNCTION__ on every debug line ;-)

really, there is already logging infrastructure there in ecore-x, if
he wants another "domain", just create a new one and name macros
differently, but use the standards and let's not require people to
change private.h to define or not the debug :-/

-- 
Gustavo Sverzut Barbieri
http://profusion.mobi embedded systems
--------------------------------------
MSN: barbi...@gmail.com
Skype: gsbarbieri
Mobile: +55 (19) 9225-2202

------------------------------------------------------------------------------
Download Intel&#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to