From owner-svn-src-all@FreeBSD.ORG Fri May 10 07:51:16 2013 Return-Path: Delivered-To: svn-src-all@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 86568A23 for ; Fri, 10 May 2013 07:51:16 +0000 (UTC) (envelope-from juli@clockworksquid.com) Received: from mail-la0-x234.google.com (mail-la0-x234.google.com [IPv6:2a00:1450:4010:c03::234]) by mx1.freebsd.org (Postfix) with ESMTP id 1062E7C4 for ; Fri, 10 May 2013 07:51:15 +0000 (UTC) Received: by mail-la0-f52.google.com with SMTP id fm20so3706180lab.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=aPdV5HdNKRPo+R5CubO1SNRV8SX7l2XPMv6ZR3DFnrctbp9vn7Ewyp8vGv/Ci0/++W O4TN0sknQ8hS+ApEITCr4l+EEjDT0cnhc+Xmawj68Vl0SXY+LyU45to0KGOSP9U9zn5a sPt70WeKZ6oLd21d++qjtljirKYGii0Fz410O24zKU0fOyPdljmiJgQF2j0l5in7DZdV CT9qV3trmM9GUQzALZTsxDwUM2NcEpjRgND4/Z98j6mEMNx0i4zAXPKnEc2KMP7g2cPu fMdlTEscuCh2HuWdQfJXkCzpIhREqVn7xoKuyYOO4zdHm7X8wU41CGWp7UrJmSV73sQn N3eQ== 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: ALoCoQlaGjl/gQHlbph2AsbyEWuu84pa12+t8Nfk/h7RTcbH5POaA6q7Awto52K5f+nxF09NDiPy Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 10 May 2013 07:51:16 -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"