Date: Wed, 19 Sep 2012 03:23:13 -0700 From: Garrett Cooper <yanegomi@gmail.com> To: Erik Cederstrand <erik@cederstrand.dk> Cc: FreeBSD Hackers <hackers@freebsd.org> Subject: Re: NDEBUG and assert() Message-ID: <CAGH67wRnCpFAO5H392Lf0mj2_pceeQ4DWA8w%2Bye_matHj6-PoQ@mail.gmail.com> In-Reply-To: <D4A468C8-82E8-4E3B-BB20-2384CE17262E@cederstrand.dk> References: <63FCC95F-7D73-423D-892D-DB5D75BCEBE7@cederstrand.dk> <D4A468C8-82E8-4E3B-BB20-2384CE17262E@cederstrand.dk>
index | next in thread | previous in thread | raw e-mail
On Wed, Sep 19, 2012 at 3:18 AM, Erik Cederstrand <erik@cederstrand.dk> wrote:
> Den 19/09/2012 kl. 11.19 skrev Erik Cederstrand <erik@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
help
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGH67wRnCpFAO5H392Lf0mj2_pceeQ4DWA8w%2Bye_matHj6-PoQ>