mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Cyrill Gorcunov" <gorcunov@gmail.com>
To: "Jesper Juhl" <jesper.juhl@gmail.com>
Cc: "Jonathan Corbet" <corbet@lwn.net>,
	rdunlap@xenotime.net, tytso@mit.edu, hch@infradead.org,
	viro@zeniv.linux.org.uk, linux-kernel@vger.kernel.org,
	davem@davemloft.net, "Andrew Morton" <akpm@linux-foundation.org>
Subject: Re: CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard indentation of arguments)
Date: Thu, 22 May 2008 08:51:44 +0400	[thread overview]
Message-ID: <aa79d98a0805212151y22190e09kddfde7514c184c26@mail.gmail.com> (raw)
In-Reply-To: <9a8748490805211646s5ef93f8ey43ebbc7746fb1a3b@mail.gmail.com>

On Thu, May 22, 2008 at 3:46 AM, Jesper Juhl <jesper.juhl@gmail.com> wrote:
[...]
>
> Try to stay away from doing pointless work, like just fixing up the
> coding style in a file, reformatting comments etc - it's not worth the
> effort and your patch is likely to be rejected...
[...]

Quite good document, I must say. Jesper, I think that part of
text (what to stay away from) *must* be marked special somehow.
I'm not sure if every newbie would read this document before sending
patch but we will be able to say "Hey, we warned you not the once
about useless work! Don't do that again!" (maybe it should be
even CAPITALIZED)

  parent reply	other threads:[~2008-05-22  4:51 UTC|newest]

Thread overview: 56+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-21  5:58 [PATCH] Standard indentation of arguments Jianjun Kong
2008-05-21  7:19 ` Jiri Slaby
2008-05-21 11:35   ` David Newall
2008-05-21  8:34 ` CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard indentation of arguments) Al Viro
2008-05-21  8:50   ` Andrew Morton
2008-05-21  9:19     ` CFD: linux-wanking@vger.kernel.org David Miller
2008-05-21  9:41     ` CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard indentation of arguments) Al Viro
2008-05-21 17:44       ` Andrew Morton
2008-05-21 19:45         ` Theodore Tso
2008-05-21 20:38           ` Rene Herman
2008-05-21 20:50             ` Andrew Morton
2008-05-21 21:08               ` Rene Herman
2008-05-21 21:12                 ` Andrew Morton
2008-05-21 21:32                   ` Rene Herman
2008-05-22 19:40           ` Jaya Kumar
2008-05-21 10:32     ` Christoph Hellwig
2008-05-21 12:09       ` Theodore Tso
2008-05-21 17:46         ` Andrew Morton
2008-05-21 18:57           ` Cyrill Gorcunov
2008-05-21 19:31             ` Andrew Morton
2008-05-21 20:10               ` CFD: linux-wanking@vger.kernel.org David Miller
2008-05-21 21:25                 ` Willy Tarreau
2008-05-21 21:44                   ` Stefan Richter
2008-05-21 21:56                     ` Willy Tarreau
2008-05-21 21:08               ` CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard indentation of arguments) Valdis.Kletnieks
2008-05-21 23:01               ` Adrian Bunk
2008-05-21 23:17                 ` Rene Herman
2008-05-21 23:55                   ` Adrian Bunk
2008-05-22  0:10                     ` Rene Herman
2008-05-22 14:31                       ` Adrian Bunk
2008-05-21 23:35                 ` Al Viro
2008-05-22  0:38                   ` Adrian Bunk
2008-05-22  1:05                     ` Alan Cox
2008-05-22 16:08                 ` Stefan Richter
2008-05-21 19:33             ` Randy Dunlap
2008-05-21 19:45               ` Cyrill Gorcunov
2008-05-21 19:54                 ` Andrew Morton
2008-05-21 20:06                   ` Jesper Juhl
2008-05-21 20:27                     ` CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard indentation of arguments), " Jonathan Corbet
2008-05-21 20:37                       ` Jesper Juhl
2008-05-21 23:46                         ` Jesper Juhl
2008-05-22  0:12                           ` Adrian Bunk
2008-05-22  0:32                             ` Jesper Juhl
2008-05-22  1:03                             ` Alan Cox
2008-05-22 14:56                             ` Pavel Machek
2008-05-22 15:29                             ` Arjan van de Ven
2008-05-22  0:14                           ` Al Viro
2008-05-22  0:34                             ` Jesper Juhl
2008-05-22  1:07                           ` Johannes Weiner
2008-05-22  4:51                           ` Cyrill Gorcunov [this message]
2008-05-22  5:34                           ` david
2008-05-31 16:23                           ` Nick Andrew
2008-05-21 19:42             ` Johannes Weiner
2008-05-21 19:47               ` Cyrill Gorcunov
2008-05-21 19:49           ` CFD: linux-wanking@vger.kernel.org David Miller
2008-05-21 22:03             ` Al Viro

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aa79d98a0805212151y22190e09kddfde7514c184c26@mail.gmail.com \
    --to=gorcunov@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=corbet@lwn.net \
    --cc=davem@davemloft.net \
    --cc=hch@infradead.org \
    --cc=jesper.juhl@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rdunlap@xenotime.net \
    --cc=tytso@mit.edu \
    --cc=viro@zeniv.linux.org.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®