From owner-svn-src-head@FreeBSD.ORG Sun May 12 03:02:58 2013 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id 977DD632; Sun, 12 May 2013 03:02:58 +0000 (UTC) (envelope-from julian@freebsd.org) Received: from vps1.elischer.org (vps1.elischer.org [204.109.63.16]) by mx1.freebsd.org (Postfix) with ESMTP id 53BCC833; Sun, 12 May 2013 03:02:58 +0000 (UTC) Received: from Julian-MBP3.local ([12.157.112.67]) (authenticated bits=0) by vps1.elischer.org (8.14.5/8.14.5) with ESMTP id r4C32uEA077975 (version=TLSv1/SSLv3 cipher=DHE-RSA-CAMELLIA256-SHA bits=256 verify=NO); Sat, 11 May 2013 20:02:57 -0700 (PDT) (envelope-from julian@freebsd.org) Message-ID: <518F065B.5010404@freebsd.org> Date: Sat, 11 May 2013 20:02:51 -0700 From: Julian Elischer User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:17.0) Gecko/20130328 Thunderbird/17.0.5 MIME-Version: 1.0 To: smckay@internode.on.net Subject: Re: svn commit: r250431 - head/usr.bin/xargs References: <65172.1368181303@internode.on.net> In-Reply-To: <65172.1368181303@internode.on.net> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Juli Mallett , svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Eitan Adler X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 12 May 2013 03:02:58 -0000 On 5/10/13 3:21 AM, smckay@internode.on.net wrote: > > > On Fri 10/05/13 17:50 , Juli Mallett jmallett@FreeBSD.org sent: >> Eitan, >> >> Are other changes coming to xargs for which these style changes make >> sense? If not, I'd say that style changes which only adjust vertical >> whitespace, especially those which mostly add vertical whitespace at >> the start of functions with no local variables (which is one of the >> most archaic and unusual things style(9) requires), are probably the >> most gratuitous and silly style changes to make. That doesn't even >> rise to being a style change, it's just a whitespace change. And I >> really do believe that vertical whitespace changes are much, much >> sillier than horizontal whitespace changes. > > Juli, it seems we're bothered by the same thing here: that at least > some of the vertical white space requirements of style(9) are > silly and (rightly) widely ignored. So I think we should change > the guide to fix this rather than damaging a lot of perfectly > readable code. > > I'd like to emphasise that I'm not even slightly cranky at Eitan for > his commit since we committers are presumed to have agreed to follow > style(9) and he's just changing code to match the official style. My > complaint is that the official style isn't useful in this small regard > and should be changed. > >> (Watch for standalone commits from people who used to be >> very active developers who have recently hit the threshold for getting >> commit bit reaper E-Mails. They're fairly instructive in the art of >> making occasional insignificant changes for the sake of making >> changes.) > > Well, I used to be a low activity developer and have transitioned to > being a very low activity developer, so I know the reaper and the > temptation of a quick fix to get it off your back. However it's better > to let the reaper pester you than to make a low quality commit. After > all, if you haven't anything new and interesting to add, there is always > that convenient list of bugs that could be fixed... enters bikeshed. with green brush.. Most of the rules there are for a reason.. sometimes the reason is out dated. The single blank line at the top (of a function, not a block) when there are no locals has annoyed me sometimes but most of the other rules I've got to like and I'm loath to change that one just for the sake of change. > > Cheers, > > Stephen. > > >