CMake: enable WITH_ASSERT_ABORT by default
authorCampbell Barton <ideasman42@gmail.com>
Thu, 28 Jun 2018 08:48:06 +0000 (10:48 +0200)
committerCampbell Barton <ideasman42@gmail.com>
Thu, 28 Jun 2018 08:48:06 +0000 (10:48 +0200)
Assert from BLI_assert by default in debug builds
(instead of just printing a warning).

Some developers ignored this, causing errors for others.
Better debug builds cause hard error so code isn't ignored.

Disabling is still useful when bisecting or testing outdated code.

CMakeLists.txt

index d1b35b1c5621e4e9003cec524f618817962a69db..46d9128646750ab0f5625e7458179ea294d15e72 100644 (file)
@@ -444,7 +444,7 @@ mark_as_advanced(WITH_MEM_VALGRIND)
 option(WITH_CXX_GUARDEDALLOC "Enable GuardedAlloc for C++ memory allocation tracking (only enable for development)" OFF)
 mark_as_advanced(WITH_CXX_GUARDEDALLOC)
 
-option(WITH_ASSERT_ABORT "Call abort() when raising an assertion through BLI_assert()" OFF)
+option(WITH_ASSERT_ABORT "Call abort() when raising an assertion through BLI_assert()" ON)
 mark_as_advanced(WITH_ASSERT_ABORT)
 
 option(WITH_BOOST                                      "Enable features depending on boost" ON)