[Mesa-dev] [PATCH] mesa: do not define NDEBUG to make sure assert work

Yuanhan Liu yuanhan.liu at linux.intel.com
Sat Oct 8 02:08:24 PDT 2011


If you include "mtypes.h", which would define macro NDEBUG, before
including "compiler.h", you would find that the assert macro will no
longer work; it just make it slient by replacing it with ((void) (0));

Signed-off-by: Yuanhan Liu <yuanhan.liu at linux.intel.com>
---
 src/mesa/main/mtypes.h |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
index 749324d..5edecf2 100644
--- a/src/mesa/main/mtypes.h
+++ b/src/mesa/main/mtypes.h
@@ -3336,9 +3336,6 @@ extern int MESA_DEBUG_FLAGS;
 # define MESA_VERBOSE 0
 # define MESA_DEBUG_FLAGS 0
 # define MESA_FUNCTION "a function"
-# ifndef NDEBUG
-#  define NDEBUG
-# endif
 #endif
 
 
-- 
1.7.4.4



More information about the mesa-dev mailing list