Re: [E-devel] E SVN: englebass trunk/eina/src/modules/mp/chained_pool

[André D. <and...@gm...>]

If DEBUG=0, won't the name INF be undefined? Maybe in this case it should be
defined to empty.

On Thu, Feb 11, 2010 at 6:20 PM, Enlightenment SVN <
no-...@en...> wrote:

> Log:
>  Use eina_log
> Author:       englebass
> Date:         2010-02-11 13:20:52 -0800 (Thu, 11 Feb 2010)
> New Revision: 46094
>
> Modified:
>  trunk/eina/src/modules/mp/chained_pool/eina_chained_mempool.c
>
> Modified: trunk/eina/src/modules/mp/chained_pool/eina_chained_mempool.c
> ===================================================================
> --- trunk/eina/src/modules/mp/chained_pool/eina_chained_mempool.c
> 2010-02-11 21:05:31 UTC (rev 46093)
> +++ trunk/eina/src/modules/mp/chained_pool/eina_chained_mempool.c
> 2010-02-11 21:20:52 UTC (rev 46094)
> @@ -38,6 +38,17 @@
>
>  #include "eina_private.h"
>
> +#ifdef DEBUG
> +#include "eina_log.h"
> +
> +static int _eina_mempool_log_dom = -1;
> +
> +#ifdef INF
> +#undef INF
> +#endif
> +#define INF(...) EINA_LOG_DOM_INFO(_eina_mempool_log_dom, __VA_ARGS__)
> +#endif
> +
>  typedef struct _Chained_Mempool Chained_Mempool;
>  struct _Chained_Mempool
>  {
> @@ -227,7 +238,7 @@
>
>  #ifdef DEBUG
>        if (p->usage > 0)
> -         EINA_ERROR_PINFO("Bad news we are destroying not an empty mempool
> [%s]\n", mp->name);
> +         INF("Bad news we are destroying not an empty mempool [%s]\n",
> mp->name);
>  #endif
>
>        mp->first = eina_inlist_remove(mp->first, mp->first);
> @@ -254,12 +265,20 @@
>
>  Eina_Bool chained_init(void)
>  {
> -       return eina_mempool_register(&_eina_chained_mp_backend);
> +   _eina_mempool_log_dom = eina_log_domain_register("eina_mempool",
> EINA_LOG_COLOR_DEFAULT);
> +   if (_eina_mempool_log_dom < 0)
> +     {
> +       EINA_LOG_ERR("Could not register log domain: eina_mempool");
> +       return EINA_FALSE;
> +     }
> +   return eina_mempool_register(&_eina_chained_mp_backend);
>  }
>
>  void chained_shutdown(void)
>  {
> -       eina_mempool_unregister(&_eina_chained_mp_backend);
> +   eina_mempool_unregister(&_eina_chained_mp_backend);
> +   eina_log_domain_unregister(_eina_mempool_log_dom);
> +   _eina_mempool_log_dom = -1;
>  }
>
>  #ifndef EINA_STATIC_BUILD_CHAINED_POOL
>
>
>
> ------------------------------------------------------------------------------
> SOLARIS 10 is the OS for Data Centers - provides features such as DTrace,
> Predictive Self Healing and Award Winning ZFS. Get Solaris 10 NOW
> http://p.sf.net/sfu/solaris-dev2dev
> _______________________________________________
> enlightenment-svn mailing list
> enl...@li...
> https://lists.sourceforge.net/lists/listinfo/enlightenment-svn
>