On Wed, Aug 26, 2009 at 3:52 PM, Vincent Torri <[email protected]> wrote:

>
> hey,
>
> eina_log defines EINA_LOG_LEVEL_UNKNOWN to INT32_MIN (btw, why _MIN ?).
> The problem is that, stdint.h, INT32_MIN is defined if the compiler is not
> a c++ one. Which, btw, that the compilation on Windows fails.
>

It is INT32_MIN (negative) so that the compiler won't set the enum typedef
as unsigned int and forbid users to provide negative logging levels.



>
> why not just putting EINA_LOG_LEVEL_UNKNOWN as the last value of the enum
> ? Is that value really needed ?


That value is used for logging domains that haven't got their level set yet,
which will tell eina_log to set it to the global logger level. We need a
special value for the unknown level.

Maybe #ifdef would fix it?



>
> Vincent
>
>
> ------------------------------------------------------------------------------
> Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day
> trial. Simplify your report design, integration and deployment - and focus
> on
> what you do best, core application coding. Discover what's new with
> Crystal Reports now.  http://p.sf.net/sfu/bobj-july
> _______________________________________________
> enlightenment-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>



-- 
André Dieb Martins

Embedded Systems and Pervasive Computing Lab (Embedded)
Electrical Engineering Department (DEE)
Center of Electrical Engineering and Informatics (CEEI)
Federal University of Campina Grande (UFCG)

Blog: http://genuinepulse.blogspot.com/
------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to