NDEBUG and assert()

Garrett Cooper yanegomi at gmail.com
Wed Sep 19 10:23:14 UTC 2012


On Wed, Sep 19, 2012 at 3:18 AM, Erik Cederstrand <erik at cederstrand.dk> wrote:
> Den 19/09/2012 kl. 11.19 skrev Erik Cederstrand <erik at cederstrand.dk>:
>
>> The below below patch will let the analyzer reason correctly about the code, and removes the report mentioned above (and a handful others in ncurses). It doesn't touch contrib code, but I'm not happy about changing include/assert.h since it's used so many other places. Any other ideas for how to best solve this?
>
> An alternative that doesn't touch assert.h but contains a patch to /contrib:

    This should be disabled [in part because it can be properly
enabled with WITH_ASSERT_DEBUG=yes in /etc/src.conf].
Thanks!
-Garrett


More information about the freebsd-hackers mailing list