Commit: 7a3122901166cc2200460a5659aa8b28dae70483
Author: Bastien Montagne
Date:   Tue May 17 15:25:03 2022 +0200
Branches: master
https://developer.blender.org/rB7a3122901166cc2200460a5659aa8b28dae70483

Cleanup: Use `BLI_assert_unreachable()` as example in comment of 
`_BLI_assert_abort`.

Using `BLI_assert(0)` should typically be avoided now that we have
`BLI_assert_unreachable`...

===================================================================

M       source/blender/blenlib/intern/BLI_assert.c

===================================================================

diff --git a/source/blender/blenlib/intern/BLI_assert.c 
b/source/blender/blenlib/intern/BLI_assert.c
index 2ebeba43dbe..96c16b47214 100644
--- a/source/blender/blenlib/intern/BLI_assert.c
+++ b/source/blender/blenlib/intern/BLI_assert.c
@@ -40,7 +40,7 @@ void _BLI_assert_abort(void)
   /* Wrap to remove 'noreturn' attribute since this suppresses missing return 
statements,
    * allowing changes to debug builds to accidentally to break release builds.
    *
-   * For example `BLI_assert(0);` at the end of a function that returns a 
value,
+   * For example `BLI_assert_unreachable();` at the end of a function that 
returns a value,
    * will hide that it's missing a return. */
 
   abort();

_______________________________________________
Bf-blender-cvs mailing list
[email protected]
List details, subscription details or unsubscribe:
https://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to