From owner-svn-src-head@FreeBSD.ORG Fri May 10 07:51:23 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 0BEC0B28 for ; Fri, 10 May 2013 07:51:22 +0000 (UTC) (envelope-from juli@clockworksquid.com) Received: from mail-lb0-f180.google.com (mail-lb0-f180.google.com [209.85.217.180]) by mx1.freebsd.org (Postfix) with ESMTP id D577E7C7 for ; Fri, 10 May 2013 07:51:21 +0000 (UTC) Received: by mail-lb0-f180.google.com with SMTP id v1so3942444lbd.11 for ; Fri, 10 May 2013 00:51:15 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-received:mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type :x-gm-message-state; bh=6cON1t5I+4TLaoAoMw+V6/HhG9XxRhlrzw/b7Oxc1Qc=; b=gVHpjxt/jR1fKTcsVPa7jEopf4kiWS61D2n7tDlsBlFLH8l523ghXUP5ScLQiWbR0O JqMNXzz3sReggwWzA4k9/iwdx3PwszpHQ/6hdxwaEVjPLmuIvXmQ/XeC3tv/tLj1TuwK YZfCS4UoZ9aTCSYBARIKArEBa1QUkR40iGmUV5CJhm6FjZlTxGOL2O3YeG5H7Xr/ms5A MeMK8IO84JAaQj+0VnJ16KcP9iAIxEi8LW7nU5TB8BFnCyHzOFUxoBI/X0Of+i/T8X4R huU7CMCZ77NFuVA9If3Hk09P58rXaxi8Y4If2YLUKbxk/5gzRCilwnp8sUUVNXS9oNQX k6rA== X-Received: by 10.112.72.233 with SMTP id g9mr6986525lbv.131.1368172274917; Fri, 10 May 2013 00:51:14 -0700 (PDT) MIME-Version: 1.0 Sender: juli@clockworksquid.com Received: by 10.152.128.67 with HTTP; Fri, 10 May 2013 00:50:54 -0700 (PDT) In-Reply-To: <201305100349.r4A3n5qh081766@svn.freebsd.org> References: <201305100349.r4A3n5qh081766@svn.freebsd.org> From: Juli Mallett Date: Fri, 10 May 2013 00:50:54 -0700 X-Google-Sender-Auth: pTpyFMROmDahDcGrLM_HEFW5fVg Message-ID: Subject: Re: svn commit: r250431 - head/usr.bin/xargs To: Eitan Adler Content-Type: text/plain; charset=UTF-8 X-Gm-Message-State: ALoCoQmFXUyNET7meP3NdDEQGc1W7TWHhpfVxeisrHZM2p/2Qqdbu15kBDK6qGiy3oV2reuwjt0/ Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org 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: Fri, 10 May 2013 07:51:23 -0000 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. Curious what the motivation was here; was xargs the only utility in usr.bin which didn't conform to style(9) in this regard? (No.) Was this part of a tree-wide sweep to finally bring conformity to these rules? (I hope not.) What's the point?* Juli. *: Not that I've never made changes equally-gratuitous; I certainly have, and so would more strongly discourage people from making them given the regret and embarrassment I feel about having done so with the aid of years of hindsight, and a greater awareness of how easy it is to make minor functional improvements, or at least do refactoring with some purpose in mind, if one just wants to leave one's thumbprint. (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.) On Thu, May 9, 2013 at 8:49 PM, Eitan Adler wrote: > Author: eadler > Date: Fri May 10 03:49:05 2013 > New Revision: 250431 > URL: http://svnweb.freebsd.org/changeset/base/250431 > > Log: > Style(9) changes fo xargs.c > > Modified: > head/usr.bin/xargs/xargs.c > > Modified: head/usr.bin/xargs/xargs.c > ============================================================================== > --- head/usr.bin/xargs/xargs.c Fri May 10 03:42:48 2013 (r250430) > +++ head/usr.bin/xargs/xargs.c Fri May 10 03:49:05 2013 (r250431) > @@ -670,12 +670,14 @@ pids_init(void) > static int > pids_empty(void) > { > + > return (curprocs == 0); > } > > static int > pids_full(void) > { > + > return (curprocs >= maxprocs); > } > > @@ -709,7 +711,6 @@ findfreeslot(void) > > if ((slot = findslot(NOPID)) < 0) > errx(1, "internal error: no free pid slot"); > - > return (slot); > } > > @@ -721,13 +722,13 @@ findslot(pid_t pid) > for (slot = 0; slot < maxprocs; slot++) > if (childpids[slot] == pid) > return (slot); > - > return (-1); > } > > static void > clearslot(int slot) > { > + > childpids[slot] = NOPID; > } > > @@ -762,6 +763,7 @@ prompt(void) > static void > usage(void) > { > + > fprintf(stderr, > "usage: xargs [-0opt] [-E eofstr] [-I replstr [-R replacements] [-S replsize]]\n" > " [-J replstr] [-L number] [-n number [-x]] [-P maxprocs]\n"