Date: Wed, 21 Feb 2001 20:16:16 +0000 From: Ben Smithurst <csxbcs@comp.leeds.ac.uk> To: Poul-Henning Kamp <phk@critter.freebsd.dk> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/share/man/man9 style.9 Message-ID: <20010221201615.A25231@comp.leeds.ac.uk> In-Reply-To: <1166.982760630@critter> References: <200102211210.f1LCAKN41997@freefall.freebsd.org> <1166.982760630@critter>
next in thread | previous in thread | raw e-mail | index | archive | help
Poul-Henning Kamp wrote: > Better, and more readable: > > error = function(a1, a2); > if (error != 0) Well that's a matter of opinion, but I suppose it's easily changed. :-) I figured there's enough use of 'if ((var = func()) != 0)' in FreeBSD that everyone knows exactly what it does and yours isn't really any more readable. But it can easily be changed if people generally agree with you. -- Ben Smithurst / csxbcs@comp.leeds.ac.uk / ben@FreeBSD.org To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010221201615.A25231>