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>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Sep 19, 2012 at 3:18 AM, Erik Cederstrand <erik@cederstrand.dk> wro=
te:
> 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 c=
ode, and removes the report mentioned above (and a handful others in ncurse=
s). 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 /contr=
ib:
This should be disabled [in part because it can be properly
enabled with WITH_ASSERT_DEBUG=3Dyes in /etc/src.conf].
Thanks!
-Garrett
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGH67wRnCpFAO5H392Lf0mj2_pceeQ4DWA8w%2Bye_matHj6-PoQ>