[vlc-devel] [PATCH] configure: enable asserts by default

Thomas Guillem thomas at gllm.fr
Wed Mar 28 14:35:28 CEST 2018


For 4.0, I propose to finally do the opposite:
 - Enable asserts by default (NDEBUG is not defined).
 - Disable asserts with --disable-debug for release builds (NDEBUG is defined).

Why ? Because too many developers are not using this option that should be
mandatory during the development or testing phase.
---
 configure.ac | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/configure.ac b/configure.ac
index e1a4800ccb..e1f1e8c748 100644
--- a/configure.ac
+++ b/configure.ac
@@ -952,9 +952,9 @@ dnl
 dnl  Debugging mode
 dnl
 AC_ARG_ENABLE(debug,
-  [AS_HELP_STRING([--enable-debug],
-    [build with run-time assertions (default disabled)])],,
-  [enable_debug="no"])
+  [AS_HELP_STRING([--disable-debug],
+    [disable run-time assertions (default enabled)])],,
+  [enable_debug="yes"])
 AH_TEMPLATE(NDEBUG,
             [Define to 1 if debug code should NOT be compiled])
 AS_IF([test "${enable_debug}" != "no"], [
-- 
2.11.0



More information about the vlc-devel mailing list