From owner-freebsd-current Wed May 29 13:40:39 2002 Delivered-To: freebsd-current@freebsd.org Received: from rwcrmhc53.attbi.com (rwcrmhc53.attbi.com [204.127.198.39]) by hub.freebsd.org (Postfix) with ESMTP id 307CC37B401; Wed, 29 May 2002 13:40:17 -0700 (PDT) Received: from InterJet.elischer.org ([12.232.206.8]) by rwcrmhc53.attbi.com (InterMail vM.4.01.03.27 201-229-121-127-20010626) with ESMTP id <20020529204016.QWKS11659.rwcrmhc53.attbi.com@InterJet.elischer.org>; Wed, 29 May 2002 20:40:16 +0000 Received: from localhost (localhost.elischer.org [127.0.0.1]) by InterJet.elischer.org (8.9.1a/8.9.1) with ESMTP id NAA18943; Wed, 29 May 2002 13:33:42 -0700 (PDT) Date: Wed, 29 May 2002 13:33:41 -0700 (PDT) From: Julian Elischer To: Poul-Henning Kamp Cc: Matthew Dillon , "David O'Brien" , John Baldwin , FreeBSD current users Subject: Re: Seeking OK to commit KSE MIII In-Reply-To: <13192.1022704058@critter.freebsd.dk> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII 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 On Wed, 29 May 2002, Poul-Henning Kamp wrote: > In message <200205292007.g4TK71YD062671@apollo.backplane.com>, Matthew Dillon w > rites: > > > I agree that as a general rule of thumb it makes sense to commit > > whitespace/paren/brace changes separately, but that is ALL it is. > > A rule of thumb. It should not be followed blindly, on principle, > > if it has an adverse effect on the developer tring to do the work, > > and it certainly should not be made a prerequisit for having good > > work comitted. I see no benefit to the project or the developer > > community. > > For complex patches, which should get good and thorough reviews before > they are committed, avoiding unsubstantial style/whitespace changes from > cluttering the patch should be mandatory. > > Julians patch clearly belongs in this category. > > Consequently, it is in Julians own interest to get the whitespace/style > changes into the tree first (they don't need a review, if he does > them right) since that makes the patch people have to read through > correspondingly smaller. Generally I agree with this statement.. in the case of the braces in question, they are part of an if statement that got rewritten so I'm slightly surprised that they are considered "unrelated" to the patch. I'm happy to commit it separatly right nowe though, as it's another 6 line less of diff :-) > > -- > Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 > phk@FreeBSD.ORG | TCP/IP since RFC 956 > FreeBSD committer | BSD since 4.3-tahoe > Never attribute to malice what can adequately be explained by incompetence. > To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message