xiaoxiang781216 commented on PR #12303: URL: https://github.com/apache/nuttx/pull/12303#issuecomment-2100895046
> I thought that the internal assertions, like `ASSERT()` and `VERIFY()` (not the standard `assert()`), were controlled by `CONFIG_DEBUG_ASSERTIONS` and not `NDEBUG`. > No, DEBUGASSERT is controlled by CONFIG_DEBUG_ASSERTIONS. ASSERT, VERIFY and assert are controlled by NDEBUG. > Also before `ASSERT()` was always active, and only `DEBUGASSERT()` was enabled/disabled depending on the build type. > > Is this changed now? this patch doesn't change the assertion behavior, only the warning of unused variable may generate in the new patch if you use assert, but it's required by spec. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
