Re: [Mesa-dev] [PATCH 2/2] configure.ac: add --enable-assertions

2015-04-14 Thread Emil Velikov
Hi Marek On 13 April 2015 at 21:06, Marek Olšák mar...@gmail.com wrote: From: Marek Olšák marek.ol...@amd.com --- configure.ac | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 6ccf3b4..f5eeb7d 100644 --- a/configure.ac +++

Re: [Mesa-dev] [PATCH 2/2] configure.ac: add --enable-assertions

2015-04-14 Thread Marek Olšák
If --enable-debug is used, --enable-assertions is ignored. --enable-assertions is basically a weaker version of --enable-debug. There are 2 assert definitions in Mesa: - The one that is enabled if DEBUG is defined (gallium only). - The one that is enabled if NDEBUG is not defined (the standard

Re: [Mesa-dev] [PATCH 2/2] configure.ac: add --enable-assertions

2015-04-14 Thread Marek Olšák
On Tue, Apr 14, 2015 at 3:12 PM, Emil Velikov emil.l.veli...@gmail.com wrote: On 14 April 2015 at 13:27, Marek Olšák mar...@gmail.com wrote: If --enable-debug is used, --enable-assertions is ignored. --enable-assertions is basically a weaker version of --enable-debug. Doesn't this make things

Re: [Mesa-dev] [PATCH 2/2] configure.ac: add --enable-assertions

2015-04-14 Thread Emil Velikov
On 14 April 2015 at 13:27, Marek Olšák mar...@gmail.com wrote: If --enable-debug is used, --enable-assertions is ignored. --enable-assertions is basically a weaker version of --enable-debug. Doesn't this make things more counter-intuitive ? I'm guilty of making --enable-debug a bit messy (and

[Mesa-dev] [PATCH 2/2] configure.ac: add --enable-assertions

2015-04-13 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com --- configure.ac | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 6ccf3b4..f5eeb7d 100644 --- a/configure.ac +++ b/configure.ac @@ -402,6 +402,13 @@ AC_ARG_ENABLE([debug],