From owner-svn-src-all@freebsd.org Thu Sep 3 02:42:26 2020 Return-Path: Delivered-To: svn-src-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id B26253D3E3B; Thu, 3 Sep 2020 02:42:26 +0000 (UTC) (envelope-from danfe@freebsd.org) Received: from freefall.freebsd.org (freefall.freebsd.org [96.47.72.132]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "freefall.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4BhlTG4JHlz4dsc; Thu, 3 Sep 2020 02:42:26 +0000 (UTC) (envelope-from danfe@freebsd.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1599100946; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=YYhD0YejjfAOu/V5XvjhlD2tjsTepFVzBQpgNa8zT5U=; b=bQuH2kLFJ6wKqMDiy452j30fdHI0wqwq2WniA3mU3W/Tj1UVoSZdyryWs88j+cMiKJ17uR 7h1SOFliofeNDzFdVbak0CHqQL/xjdzYmo7l8OgPUd3uj0pUefUf3k/Z5wBkfqp7angeU1 +Ub444lUt8GwKhkJGmBH7x+ffTsHwddX/a5B/iZWkrtBI1IaQKZNI+eCVMFhg+PIv0hz3r vglgP2Q/uTJ/lk+7JY6KnPzMeFFutmR0J2UVeBQkJHm/AwtxD0nKfWdYiuj4U+a0BuHp8k Ov8HMj2g72XXYTV3L7YeEwFK8eGbnQfrw7Gr0dp7Tm1D2qAvfHjgk9yddWZ80w== Received: by freefall.freebsd.org (Postfix, from userid 1033) id 8C91516608; Thu, 3 Sep 2020 02:42:26 +0000 (UTC) Date: Thu, 3 Sep 2020 02:42:26 +0000 From: Alexey Dokuchaev To: Michael Tuexen Cc: Pedro Giffuni , Mateusz Guzik , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r365071 - in head/sys: net net/altq net/route net80211 netgraph netgraph/atm netgraph/atm/ccatm netgraph/atm/sscfu netgraph/atm/sscop netgraph/atm/uni netgraph/bluetooth/common netgraph... Message-ID: <20200903024226.GA54078@FreeBSD.org> References: <202009012119.081LJERb018106@repo.freebsd.org> <95844C00-D10A-456D-AD29-DF572043074F@fh-muenster.de> <20200902020507.GA38274@FreeBSD.org> <20200902180626.GA88595@FreeBSD.org> <6124a908-25a5-e023-16da-7963ba229b7f@FreeBSD.org> <08636D5E-AA07-4AE7-B5AC-656B08CF564B@fh-muenster.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <08636D5E-AA07-4AE7-B5AC-656B08CF564B@fh-muenster.de> ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1599100946; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=YYhD0YejjfAOu/V5XvjhlD2tjsTepFVzBQpgNa8zT5U=; b=AgmMuAXhGnqWJWVm0zNH5fOsVgSpB0ILrolmB46m90/hatVvQVO1ERbY/YG12KW5cphQwS sk/+J2/UE6PV18tAurRC9/w9F8vj5slIVLDg4fv4dWRxZSuK26ajJNHB1zgNy0uPggFfpd 6XL/ACC+1j0hKdAztWj3KIEocAkLr7wAX0p9fIcE+bXIBs4XcIlwcgiRLJIcbCSl5U2zWo t6RYOpQyJ4oW+ivtfUhG5yDISJK4dwr2zzC8ctIPNQpaDzj0lywA8WYomhLsv+RzYYsxsD GNEGGG5/bpvIWnE0NLI4BnStZG36qYobSWAXo+kLZvvlQjUvE+/NLpQaihl6Dg== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1599100946; a=rsa-sha256; cv=none; b=XfpFInODpgEHevFWXZ6CjE8uqQaRNfF3pZ9A/l7j47IP/JnPR4Aqm9Pr9Z/y1i1XJAGQ4J +HihKni6mlj7oAGkGyFjvI+eSSwt3OdHxRtSvKsWa4FlM8e921PI+Ej5jEqQ/SUHAYHFvB ZRJtvMn5nfTTI35qt7iIyScwih2x/gWt/bmp2kNInW6yE7ragBachW1OOZlJPsjNKt8KdU 0pQsp6wS/jEuw0dZc8tUoCdVwtBYbgGsvwefORim1S5gATFj0sUNA5NYtDlOY9Yzd6hwAP T69reonNDaMXZoHYXWJKcklX20TgmZUoVl+PZJIb54GEpkRl5Sm/mSjL8UY45w== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.33 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: Thu, 03 Sep 2020 02:42:26 -0000 On Wed, Sep 02, 2020 at 11:39:51PM +0200, Michael Tuexen wrote: > > On 2. Sep 2020, at 20:28, Pedro Giffuni wrote: > > On 02/09/2020 13:06, Alexey Dokuchaev wrote: > >> On Wed, Sep 02, 2020 at 10:18:15AM -0500, Pedro Giffuni wrote: > >>> On 01/09/2020 21:05, Alexey Dokuchaev wrote: > >>>> ... > >>>> This is common sense. I can't count how often I wanted to hack on > >>>> something in the base/kernel and was turned away by this atrocious > >>>> excessive whitespace mess. > >>>> > >>>> Thank you Mateusz for cleaning this up. > >>> I honestly don't care much, but spaces do no harm and can make the code > >>> more readable. Sort of a silent comment, or what you do in written > >>> language when you start a new paragraph. > >> Right, but that's the example of appropriate usage of whitespace. I was > >> talking about *excessive* whitespace, that is, more than two \n's in a > >> row if we speak of newlines (subject of these commits). > > > > But how much space is rather subjective so Michael is right in asking > > what rule has been violated. > > > > No one is asking for the change to be reverted: the damage, if any, > > is already done. > > Just to be clear: I have NOT asked for reverting, I did not mentioned it. Wow wow, come down people. Indeed, no one was asking for reverting and I don't understand why we're talking about it. :-) > The why was explained: Some developers don't work on files which > violate whitespace rules. Usually they still will, but would often prefer to fix the style issues first so they don't get in the way. This is a bit frustrating because, per style(9), stylistic changes are hard on the source repository (e.g. they pessimize "svn blame") and should be avoided without good reason, but we cannot just seat and watch how style bugs accumulate and make the code harder and harder to work with. > I want to understand which rules have to be followed (and why). In general, FreeBSD code we write should follow style(9); it specifically mentions "do not add whitespace at the end of a line" and "... followed by one blank line" but doesn't go as far as explicitly forbidding multiple consecutive newlines. To me it's pretty obvious, and while others might have different sens esthe'tique, usually it is lack thereof (no offense) or mere ignorance. ./danfe P.S. Old-school tools like indent(1) or `uncrustify' were never widely popular, I guess, because they did not possess enough knowledge of the language to always produce correct results. Perhaps new era tools, like clang-format, could bring this to a whole new level.