[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-wget] [Bug-Wget] Configure flag to disable assert

From: Darshit Shah
Subject: Re: [Bug-wget] [Bug-Wget] Configure flag to disable assert
Date: Sun, 16 Nov 2014 11:52:30 +0530
User-agent: Mutt/1.5.23 (2014-03-12)

On 11/15, Mike Frysinger wrote:
On 15 Nov 2014 10:24, Darshit Shah wrote:
On 11/14, Dagobert Michelsen wrote:
>> Am 14.11.2014 um 20:01 schrieb Darshit Shah:
>> Assertions are good programming practice. However, as the recent assert I 
added shows us, they should not exist on the client side installations of our 
software. Assertions are meant for development use only.
>> Hence, I've added a new configure option, --disable-assert which will 
eliminate all assertions from the codebase before compilation.
>Is this necessary? I mean if you add -DNDEBUG to CPPFLAGS asserts are already 
>compiled in.

That's very true. But as I have come to realize, not a lot of people are either:
a. Aware of that fact
b. Using the -DNDEBUG option.

On the other hand, providing this configure option and making a note in the NEWS
section, I am hoping that more maintainers will use the --disable-assert flag
and remove assertions from the compiled code.



Thanking You,
Darshit Shah

Attachment: pgpag4022DPvT.pgp
Description: PGP signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]