From owner-p4-projects@FreeBSD.ORG Sun Nov 26 11:32:36 2006 Return-Path: X-Original-To: p4-projects@freebsd.org Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id CF31116A415; Sun, 26 Nov 2006 11:32:35 +0000 (UTC) X-Original-To: perforce@freebsd.org Delivered-To: perforce@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id AAE8A16A403 for ; Sun, 26 Nov 2006 11:32:35 +0000 (UTC) (envelope-from netchild@freebsd.org) Received: from redbull.bpaserver.net (redbullneu.bpaserver.net [213.198.78.217]) by mx1.FreeBSD.org (Postfix) with ESMTP id 93CFF43D58 for ; Sun, 26 Nov 2006 11:31:41 +0000 (GMT) (envelope-from netchild@freebsd.org) Received: from outgoing.leidinger.net (p54A5CF6B.dip.t-dialin.net [84.165.207.107]) by redbull.bpaserver.net (Postfix) with ESMTP id 1419F2E1A5; Sun, 26 Nov 2006 12:32:31 +0100 (CET) Received: from Magellan.Leidinger.net (Magellan.Leidinger.net [192.168.1.1]) by outgoing.leidinger.net (Postfix) with ESMTP id C29B75B54EB; Sun, 26 Nov 2006 12:32:08 +0100 (CET) Date: Sun, 26 Nov 2006 12:32:51 +0100 From: Alexander Leidinger To: Divacky Roman Message-ID: <20061126123251.7ce6cfa6@Magellan.Leidinger.net> In-Reply-To: <20061126094753.GA37342@stud.fit.vutbr.cz> References: <200611251806.kAPI6sSZ037775@repoman.freebsd.org> <20061126094753.GA37342@stud.fit.vutbr.cz> Organization: FreeBSD X-Mailer: Sylpheed-Claws 2.6.0 (GTK+ 2.10.6; i386-portbld-freebsd7.0) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-BPAnet-MailScanner-Information: Please contact the ISP for more information X-BPAnet-MailScanner: Found to be clean X-BPAnet-MailScanner-SpamCheck: not spam, SpamAssassin (not cached, score=-14.764, required 6, BAYES_00 -15.00, DK_POLICY_SIGNSOME 0.00, FORGED_RCVD_HELO 0.14, SARE_FROM_SPAM_WORD3 0.10) X-BPAnet-MailScanner-From: netchild@freebsd.org X-Spam-Status: No Cc: Perforce Change Reviews Subject: Re: PERFORCE change 110523 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 26 Nov 2006 11:32:36 -0000 Quoting Divacky Roman (Sun, 26 Nov 2006 10:47:53 +0100): > On Sat, Nov 25, 2006 at 06:06:54PM +0000, Alexander Leidinger wrote: > > http://perforce.freebsd.org/chv.cgi?CH=110523 > > > > Change 110523 by netchild@netchild_magellan on 2006/11/25 18:06:38 > > > > semi-automatic style(9) > > > > Affected files ... > > > > .. //depot/projects/linuxolator/src/sys/compat/linux/linux_futex.c#7 edit > > .. //depot/projects/linuxolator/src/sys/compat/linux/linux_futex.h#2 edit > > I dont think this is wise.. this file is taken from netbsd and we want to > be as much compatible with them as possible. pls back-out I did send the patch to manu@netbsd. I'm sure they are interested in the space -> tabs and space at the end of the line fixes too. When you look at the file you will notice that I fixed several lines which diverged from the rest of the file. And when you look at the diff between the NetBSD version and our version, you will notice that several of the style(9) fixes are in the code you changed. Apart from that, if you want to compare files coming from outside our tree which contain whitespace fixes you can use "-bB" as options to diff. I did it (15KB diff) and it shows a lot of lines which changed, but none of them because of pure whitespace changes. Some statistics: The "diff -ubB netbsd freebsd" contains 313 lines starting with a + (340 without the "-bB"), 108 with a - (135 without the "-bB"), the NetBSD file has a total of 330 lines (8KB), our file has a total of 535 lines (12KB). I would say our file already diverged a lot and and a lot of the whitespace changes are done where we already diverged. OK? Additionally AFAIK kib@ wants to work a little bit more on the futexes. I don't know when and how much, but it sounded like our version of the file will/may change. Bye, Alexander. -- ...and that is how we know the Earth to be banana-shaped. http://www.Leidinger.net Alexander @ Leidinger.net: PGP ID = B0063FE7 http://www.FreeBSD.org netchild @ FreeBSD.org : PGP ID = 72077137