Cool, thanks. My reading of those pages indicates that, were I to follow the recommendations contained therein, I would always use the EINA_LOG_DOM_*() macros and never the EINA_LOG_*() macros, because they use EINA_LOG_DOMAIN_GLOBAL, and as a good developer I should have my own domain(s) for my project.
I think it would be helpful to allow developers to specify a logging domain for an entire source file, which the EINA_LOG_*() macros would use. This would cut down on typing and errors in consistency, and increase the SNR of my code. My suggestion: the EINA_LOG_*() macros always log to EINA_LOG_DOMAIN_DEFAULT, which would initially be set to EINA_LOG_DOMAIN_GLOBAL, and may be overridden by a developer in a particular file. The complication is that it would have to be reset for the next file. On the other hand, it could encourage better developer habits regarding header files and whatnot. pete (My apologies if this has already been discussed.) On Thu, Feb 25, 2010 at 07:48:29PM +0100, Vincent Torri wrote: > > > On Thu, 25 Feb 2010, Peter Johnson wrote: > > >>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 :-/ > > > >Where is this documented? > > http://docs.enlightenment.org/auto/eina/group__Eina__Log__Group.html > > http://docs.enlightenment.org/auto/eina/tutorial_log_page.html > > it's in the documentation page of the site > > Vincent ------------------------------------------------------------------------------ Download Intel® 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
