On Sat, Jun 06, 2009 at 07:20:39PM +0200, Cedric BAIL wrote : > On Sat, Jun 6, 2009 at 5:48 PM, Albin Tonnerre<albin.tonne...@gmail.com> > wrote: > > On Sat, Jun 06, 2009 at 12:12:49PM -0300, Andre Dieb wrote : > >> This patch contains my other two previous patches: > >> > >> - [PATCH] Fix eina mempool leak, init error handling, remove consumed > >> TODO msg > >> - [PATCH] Add checks to eina_module_new > >> > >> Also fixes the build error with --disable-magic-debug. > > > > I still think that when ecore_magic is disabled, it would be better to use > > functions rather than preprocessor macros for things like > > eina_magic_string_init(). Indeed, that would avoid exporting different > > symbols > > depending on whether eina has magic-debug enabled or not. If you use > > macros, an > > application using magic debug compiled against eina without magic-debug will > > have to be recompiled to benefit from it if it gets enabled in eina at a > > later > > point. If you use functions, this is no longer an issue. > > > > Thoughts ? > > I strongly disagree, when you disable magic debug in eina, you want to > avoid it's cost at all. >This include the call of the function call. > And from my test, it does have a big impact to just enter an empty > function. When we are speaking about performance, I think we can > accept the little tradeoff of needing to recompile application using > eina for maximum performance.
If you're an application developer and use eina_magic, you're ready to pay that cost and more likely to rant about eina_magic being disabled than about any performance loss, as you explicitely asked for it. If you're talking about eina, nothing prevents us to do some trickery so that functions calls are avoided internally, while still exporting the symbols. Regards, -- Albin Tonnerre
signature.asc
Description: Digital signature
------------------------------------------------------------------------------ OpenSolaris 2009.06 is a cutting edge operating system for enterprises looking to deploy the next generation of Solaris that includes the latest innovations from Sun and the OpenSource community. Download a copy and enjoy capabilities such as Networking, Storage and Virtualization. Go to: http://p.sf.net/sfu/opensolaris-get
_______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
