From owner-freebsd-current Wed May 29 15:22: 5 2002 Delivered-To: freebsd-current@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 931) id 1FB3337B404; Wed, 29 May 2002 15:22:00 -0700 (PDT) Date: Wed, 29 May 2002 15:22:00 -0700 From: "J. Mallett" To: Matthew Dillon Cc: David O'Brien , Julian Elischer , John Baldwin , FreeBSD current users Subject: Re: Seeking OK to commit KSE MIII Message-ID: <20020529152159.A82752@FreeBSD.ORG> References: <20020529124434.A2156@dragon.nuxi.com> <200205292007.g4TK71YD062671@apollo.backplane.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: <200205292007.g4TK71YD062671@apollo.backplane.com>; from dillon@apollo.backplane.com on Wed, May 29, 2002 at 01:07:01PM -0700 Organisation: The FreeBSD Project X-Alternate-Addresses: , , , X-Affiliated-Projects: FreeBSD, xMach, ircd-hybrid-7 X-Towel: Yes Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG * From Matthew Dillon > > No hold on a minute. Some of us believe that adding those extra > braces and parenthesis makes the code a whole lot more readable. they > are NOT gratuitous in the least, certainly not from my point of view! > When you make the code more readable, you introduce further diffs, and you leave no reference against the original code of where the functional changes are. Either make the "base" code cleaned up by committing non-functional changes first, or commit against the "base" code your functional changes, and then clean it up. Otherwise, it's a pain in the ass to sort things out. And for what it's worth, I'm all for readability improvements in our code, I just also like to go and view diffs sometimes and try to figure out what's changed. But I think you're also right on the rule of thumb thing, if someone does not want to do this favor to everyone who might want to read diffs or annotate changes to the code and get something meaningful, that's fine. I've done it plenty of times myself. However, I tend to do that locally first, and then commit the harmless things as I go. If you ever look through one of my WIP repos (~jmallett/cvs on Freefall currently holds one, my sccs repo, as I did and do believe others should have easy access to it), I tend to piggyback a lot of stylistic nits, etc. as I go... But I don't think it's fair to do the merge like you're saying... But back to where this started, it's just a rule of thumb. Furthermore, it'd be nice if as people made stylistic changes for outside projects they committed them to the central repository, where applicable. Anyway, I'm out of paint. Good day. -- J. Mallett FreeBSD: The Power To Serve To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message