Date: Tue, 7 Dec 1999 16:03:20 +0000 From: Josef Karthauser <joe@pavilion.net> To: Bruce Evans <bde@zeta.org.au> Cc: Josef Karthauser <joe@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/usr.bin/whois whois.c Message-ID: <19991207160319.B67579@florence.pavilion.net> In-Reply-To: <Pine.BSF.4.10.9912071000200.4891-100000@alphplex.bde.org> References: <199912062118.NAA04479@freefall.freebsd.org> <Pine.BSF.4.10.9912071000200.4891-100000@alphplex.bde.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Dec 07, 1999 at 10:05:55AM +1100, Bruce Evans wrote:
> On Mon, 6 Dec 1999, Josef Karthauser wrote:
>
> > joe 1999/12/06 13:18:34 PST
> >
> > Modified files:
> > usr.bin/whois whois.c
> > Log:
> > Add braces ({}) to all the places that they're missing.
> >
> > Revision Changes Path
> > 1.15 +35 -19 src/usr.bin/whois/whois.c
>
> None were missing. Adding around one-line statements is explicitly
> forbidden in style(9).
>
> Bruce
I thought that it was _explicity_ required! That way when someone visits
the code later they don't need to worry about breaking code because of missing
braces. (A major cause of software bug, is it not?)
Joe
--
Josef Karthauser FreeBSD: How many times have you booted today?
Technical Manager Viagra for your server (http://www.uk.freebsd.org)
Pavilion Internet plc. [joe@pavilion.net, joe@uk.freebsd.org, joe@tao.org.uk]
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?19991207160319.B67579>