raster pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=062c08fbabdd80707abe3ecfddf25f48f2d35221

commit 062c08fbabdd80707abe3ecfddf25f48f2d35221
Author: Vitor Sousa <vitorsousa@vitorsousa-ws.(none)>
Date:   Wed May 28 17:51:14 2014 +0900

    Correction for the macro EINA_CXX_DOM_LOG
    
    Summary: The previous definition of EINA_CXX_DOM_LOG could cause a problem 
if used inside an one-line "if" statement followed by an "else". Changed the 
top "if" by a "for" statement to avoid this problem.
    
    Reviewers: felipealmeida, cedric, woohyun, smohanty, raster
    
    Reviewed By: raster
    
    CC: savio, cedric
    
    Differential Revision: https://phab.enlightenment.org/D902
---
 src/bindings/eina_cxx/eina_log.hh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/bindings/eina_cxx/eina_log.hh 
b/src/bindings/eina_cxx/eina_log.hh
index fe4e3bf..87bcdb1 100644
--- a/src/bindings/eina_cxx/eina_log.hh
+++ b/src/bindings/eina_cxx/eina_log.hh
@@ -80,7 +80,7 @@ inline void _log(std::stringstream const& stream, int domain, 
::Eina_Log_Level l
 }
 
 #define EINA_CXX_DOM_LOG(DOMAIN, LEVEL)                                 \
-    if( bool run = ::eina_log_domain_level_check((DOMAIN), LEVEL) )     \
+    for( bool run = ::eina_log_domain_level_check((DOMAIN), LEVEL); run;) \
       for(std::stringstream stream; run ;                               \
           ::efl::eina::_log(std::move(stream), (DOMAIN), LEVEL          \
                             , __FILE__, __FUNCTION__, __LINE__), run = false) \

-- 


Reply via email to