Re: [Mesa-dev] [PATCH] gallium: Guard assertions by NDEBUG instead of DEBUG

2017-11-06 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Mon, Nov 6, 2017 at 6:39 PM, Michel Dänzer wrote: > From: Michel Dänzer > > This matches the standard assert.h header. > > Signed-off-by: Michel Dänzer > --- > src/gallium/auxiliary/util/u_debug.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > d

Re: [Mesa-dev] [PATCH] gallium: Guard assertions by NDEBUG instead of DEBUG

2017-11-06 Thread Eric Engestrom
On Monday, 2017-11-06 18:39:54 +0100, Michel Dänzer wrote: > From: Michel Dänzer > > This matches the standard assert.h header. > > Signed-off-by: Michel Dänzer Reviewed-by: Eric Engestrom Guess we should audit every DEBUG in the codebase to make sure the right one is used. > --- > src/gal

[Mesa-dev] [PATCH] gallium: Guard assertions by NDEBUG instead of DEBUG

2017-11-06 Thread Michel Dänzer
From: Michel Dänzer This matches the standard assert.h header. Signed-off-by: Michel Dänzer --- src/gallium/auxiliary/util/u_debug.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/util/u_debug.h b/src/gallium/auxiliary/util/u_debug.h index 63940b7225