On Mon, Mar 11, 2013 at 03:58:05PM +0200, Tuomas Räsänen wrote: > Assertions imply programming invariants which must be fulfilled, > otherwise there is a programming error. > > Regarding this commit, it's a programming error to pass NULL to > append_serve(). And now it's explicitly stated with the assert(3) > statement. > > If-then-exit -constructs could be interpreted as a valid and > semantically meaningful code: "it's a feature of append_serve() to > exit the process if NULL is passed", which does not make any sense. On > the other hand, assert(s != NULL) makes it very clear that NULL must > not be passed, or the contract will be terminated along with the > process.
applied, thanks > This commit is part of the desensitization treatment for author's > assertion allergy. *g* :-) -- Copyshops should do vouchers. So that next time some bureaucracy requires you to mail a form in triplicate, you can mail it just once, add a voucher, and save on postage. ------------------------------------------------------------------------------ Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the endpoint security space. For insight on selecting the right partner to tackle endpoint security challenges, access the full report. http://p.sf.net/sfu/symantec-dev2dev _______________________________________________ Nbd-general mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/nbd-general
