From owner-p4-projects@FreeBSD.ORG Tue Oct 16 04:56:10 2007 Return-Path: Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 699E316A468; Tue, 16 Oct 2007 04:56:10 +0000 (UTC) Delivered-To: perforce@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 729E916A41B; Tue, 16 Oct 2007 04:56:09 +0000 (UTC) (envelope-from bright@elvis.mu.org) Received: from elvis.mu.org (elvis.mu.org [192.203.228.196]) by mx1.freebsd.org (Postfix) with ESMTP id 67B4F13C459; Tue, 16 Oct 2007 04:56:09 +0000 (UTC) (envelope-from bright@elvis.mu.org) Received: by elvis.mu.org (Postfix, from userid 1192) id 824691A4D80; Mon, 15 Oct 2007 21:31:33 -0700 (PDT) Date: Mon, 15 Oct 2007 21:31:33 -0700 From: Alfred Perlstein To: Hans Petter Selasky Message-ID: <20071016043133.GW31826@elvis.mu.org> References: <200709231625.l8NGPhaR097038@repoman.freebsd.org> <46F6A3C3.6010408@FreeBSD.org> <200709232113.34718.hselasky@freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200709232113.34718.hselasky@freebsd.org> User-Agent: Mutt/1.4.2.3i Cc: Perforce Change Reviews , "Constantine A. Murenin" Subject: Re: PERFORCE change 126745 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: Tue, 16 Oct 2007 04:56:10 -0000 * Hans Petter Selasky [070923 13:13] wrote: > Hi Constantine, > > Thanks for your input and I _know_ that my code is not 100% style compliant. > Most of the style misbehaves are probably there due to the way my "xemacs" > autoformatting behaves. And I love curly brackets and parenthesis, by the > way :-) > > My plan is to clean up all the style stuff by a small C-program in the end, > because doing it by hand is waste of my time. Probably it will take less time > to write that program than the actual manual edit when we are talking about > doing alot of edits. It wonders me if such a tool already exists, because the > code is technically OK. > > Anyone that wants to be a little more constructive and point towards where the > FreeBSD style transformer program is? I assume it would be extremely useful > to everyone that is forced to use multiple different styles depending on what > project they are contributing to, like me. Then before commit I will run that > script and verify the differences. And that's it. > > --HPS Hans, one of the issues with doing such changes at the end is that it effectively obliterates the ability to "cvs annotate" your code. I'm sure perforce has an 'annotate' command as well. I would suggest that you find/fix the style now and apply it now rather than later as the longer you wait, the more history you obliterate. -- - Alfred Perlstein