From owner-freebsd-current  Sun Jan 31 21:49:13 1999
Return-Path: <owner-freebsd-current@FreeBSD.ORG>
Received: (from majordom@localhost)
          by hub.freebsd.org (8.8.8/8.8.8) id VAA25282
          for freebsd-current-outgoing; Sun, 31 Jan 1999 21:49:13 -0800 (PST)
          (envelope-from owner-freebsd-current@FreeBSD.ORG)
Received: from gatekeeper.tsc.tdk.com (gatekeeper.tsc.tdk.com [207.113.159.21])
          by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id VAA25276
          for <current@FreeBSD.ORG>; Sun, 31 Jan 1999 21:49:12 -0800 (PST)
          (envelope-from gdonl@tsc.tdk.com)
Received: from sunrise.gv.tsc.tdk.com (root@sunrise.gv.tsc.tdk.com [192.168.241.191])
	by gatekeeper.tsc.tdk.com (8.8.8/8.8.8) with ESMTP id VAA27831;
	Sun, 31 Jan 1999 21:48:40 -0800 (PST)
	(envelope-from gdonl@tsc.tdk.com)
Received: from salsa.gv.tsc.tdk.com (salsa.gv.tsc.tdk.com [192.168.241.194])
	by sunrise.gv.tsc.tdk.com (8.8.5/8.8.5) with ESMTP id VAA09450;
	Sun, 31 Jan 1999 21:48:39 -0800 (PST)
Received: (from gdonl@localhost)
	by salsa.gv.tsc.tdk.com (8.8.5/8.8.5) id VAA07713;
	Sun, 31 Jan 1999 21:48:36 -0800 (PST)
From: Don Lewis <Don.Lewis@tsc.tdk.com>
Message-Id: <199902010548.VAA07713@salsa.gv.tsc.tdk.com>
Date: Sun, 31 Jan 1999 21:48:36 -0800
In-Reply-To: Sheldon Hearn <axl@iafrica.com>
       "Re: btokup().. patch to STYLE(9) (fwd)" (Jan 29, 12:05pm)
X-Mailer: Mail User's Shell (7.2.6 alpha(3) 7/19/95)
To: Sheldon Hearn <axl@iafrica.com>, Greg Lehey <grog@lemis.com>
Subject: Re: btokup().. patch to STYLE(9) (fwd)
Cc: current@FreeBSD.ORG
Sender: owner-freebsd-current@FreeBSD.ORG
Precedence: bulk
X-Loop: FreeBSD.ORG

On Jan 29, 12:05pm, Sheldon Hearn wrote:
} Subject: Re: btokup().. patch to STYLE(9) (fwd)

} The reason I'm interested in this (now tiresome) thread is that I'd much
} rather have to read
} 
} 	/*
} 	 * Bail out if the time left to next transaction is less than
} 	 * the duration of the previous transaction.
} 	 */
} 	if (t % u - n % u < d % u) {
} 
} than
} 
} 	if (((t % u) - (n % u)) < (d % u)) {
} 
} Giving folks the go-ahead to use parens as a form of documentation is
} misguided and will end in tears. MHO.

This is a fairly trivial example, but I find the second version slightly
easier to read at a glance.  I do think it's overly parenthesized, though.
I prefer

 	if ((t % u - n % u) < (d % u)) {

or 

 	if ((t % u - n % u) < d % u) {

because they are less cluttered.

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message