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 ?

Regards,
-- 
Albin Tonnerre

Attachment: 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to