From owner-freebsd-audit Tue Jun 26 23:44:39 2001 Delivered-To: freebsd-audit@freebsd.org Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by hub.freebsd.org (Postfix) with ESMTP id 90C5337B406 for ; Tue, 26 Jun 2001 23:44:36 -0700 (PDT) (envelope-from bde@zeta.org.au) Received: from bde.zeta.org.au (bde.zeta.org.au [203.2.228.102]) by mailman.zeta.org.au (8.9.3/8.8.7) with ESMTP id QAA07345; Wed, 27 Jun 2001 16:44:19 +1000 Date: Wed, 27 Jun 2001 16:42:24 +1000 (EST) From: Bruce Evans X-Sender: bde@besplex.bde.org To: Warner Losh Cc: Mike Heffner , audit@FreeBSD.ORG Subject: Re: Config patch to stable. In-Reply-To: <200106262346.f5QNkDU11463@harmony.village.org> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-freebsd-audit@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Tue, 26 Jun 2001, Warner Losh wrote: > In message Mike Heffner writes: > : Braces that aren't necessary may be left out. > > It is allowed by this statement. If they may be left out, they may be > left in too :-) This statement is a bug in style(9). To fix it, upgrade to rev.1.22 which said "Don't add braces that aren't necessary". (Some unnecessary braces are actually necessary to prevent compiler warnings. Hopefully the compiler warnings are of high quality. I like gcc's warnings about braces, but not its ones about logical and binary logical operations in combination with themselves.) The rule must give an unambigous number of braces; otherwise the code will keep changing when people who like braces and them and people who don't like them delete them. Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-audit" in the body of the message