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 not > >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.
agreed -mike
signature.asc
Description: Digital signature
