From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S936591AbYEUTnn (ORCPT ); Wed, 21 May 2008 15:43:43 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1761128AbYEUTnE (ORCPT ); Wed, 21 May 2008 15:43:04 -0400 Received: from saeurebad.de ([85.214.36.134]:46460 "EHLO saeurebad.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759998AbYEUTm7 (ORCPT ); Wed, 21 May 2008 15:42:59 -0400 From: Johannes Weiner To: Cyrill Gorcunov Cc: Andrew Morton , Theodore Tso , Christoph Hellwig , Al Viro , linux-kernel@vger.kernel.org, davem@davemloft.net Subject: Re: CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard indentation of arguments) References: <12113495282137-git-send-email-kongjianjun@gmail.com> <20080521083413.GM28946@ZenIV.linux.org.uk> <20080521015037.add0b78e.akpm@linux-foundation.org> <20080521103206.GA24545@infradead.org> <20080521120939.GF8581@mit.edu> <20080521104644.5b6f4e63.akpm@linux-foundation.org> <20080521185725.GD6932@cvg> Date: Wed, 21 May 2008 21:42:32 +0200 In-Reply-To: <20080521185725.GD6932@cvg> (Cyrill Gorcunov's message of "Wed, 21 May 2008 22:57:25 +0400") Message-ID: <87ej7v4f6f.fsf@saeurebad.de> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.0.60 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.1.3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Cyrill Gorcunov writes: > [Andrew Morton - Wed, May 21, 2008 at 10:46:44AM -0700] > | On Wed, 21 May 2008 08:09:39 -0400 Theodore Tso wrote: > | > | > On Wed, May 21, 2008 at 06:32:06AM -0400, Christoph Hellwig wrote: > | > > On Wed, May 21, 2008 at 01:50:37AM -0700, Andrew Morton wrote: > | > > > Oh, what a marvellous way to encourage new contributors that was. Thank > | > > > you so much. > | > > > > | > > > For the record: Al speaks only for himself and a lack of expressed > | > > > disagrement from others should not be taken as agreement. > | > > > | > > But I'd like to second the opinion. This is getting a little too far. > | > > We should rather try to at least enforce very basic standards a lot of > | > > the crap shoved in doesn't follow instead of wanking around about exact > | > > placement of whitespaces. > | > > | > The real question is whether people who are wanking about whitespace > | > and spelling fixes in comments will graduate to writing real, useful > | > patches. If they won't, there's no point to encouraging them. > | > > | > | Guys, get a clue. It doesn't matter what that person did. It is the > | effect upon *all* other potential developers which is so damaging here. > | Not upon this individual. > | > > Btw, we have CodingStyle, SubmittingPatches and other, but why don't > we have something like KernelNewbieGuide? Don't get me wrong, but > there could be written all rules about - what is good to do, what is bad. > So a newbiew who wants to be usefull for kernel could read it and decide > what should be done. /Don't beat me ;) / And of course I know about > kernelnewbie.org but this (even quite short) document could help I > think. How about the following? --- From: Johannes Weiner Subject: [PATCH] CodingStyle: no more trivial coding style fixes, please Add a note to CodingStyle that style cleanups should only be done if the code is really offending and violating the kernel conventions heavily. But no more oneliners fixing indentation and the like. Signed-off-by: Johannes Weiner --- Documentation/CodingStyle | 12 ++++++++++++ 1 file changed, 12 insertions(+) --- a/Documentation/CodingStyle +++ b/Documentation/CodingStyle @@ -783,6 +783,18 @@ own custom mode, or may have some other work correctly. + Chapter 19: No coding style patches, please + +While all these conventions should be honored when writing new code +please do not send patches that fix minimal coding style issues only. + +If a whole file or several logically connected ones are in a really +bad shape (i.e. violating several points named here), a patch cleaning +them up in one go is okay. + +But do not send patches that fix indentation of two lines. It is not +worth the effort. + Appendix I: References