mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Richard Gooch <rgooch@ras.ucalgary.ca>
To: Jeff Garzik <jgarzik@mandrakesoft.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: GPLONLY kernel symbols???
Date: Mon, 12 Nov 2001 21:37:02 -0700	[thread overview]
Message-ID: <200111130437.fAD4b2j17329@vindaloo.ras.ucalgary.ca> (raw)
In-Reply-To: <3BF09E44.58D138A6@mandrakesoft.com>
In-Reply-To: <200111130324.fAD3OE916102@vindaloo.ras.ucalgary.ca> <Pine.GSO.4.21.0111122249160.22925-100000@weyl.math.psu.edu> <200111130358.fAD3wgb16617@vindaloo.ras.ucalgary.ca> <3BF09E44.58D138A6@mandrakesoft.com>

Jeff Garzik writes:
> Richard Gooch wrote:
> > Alexander Viro writes:
> > > On Mon, 12 Nov 2001, Richard Gooch wrote:
> > > > Dave Jones writes:
> > > > > How about running mtrr.c & devfs code through scripts/Lindent
> > > > > sometime btw?
> 
> Go ahead and Lindent mtrr.c, it hasn't been touched by rgooch in a
> while...

Hey!

> > > > That would be a step backwards: I wouldn't be able to read my own code
> > > > then.
> > >
> > > You mean that you are unable to read any of the core kernel source?
> > > That would explain a lot...
> > 
> > Were you born rude, or did you have to practice it?
> 
> I would argue both ;-)  He is a Usenet denizen after all.
> (takes one to know one... I'm one as well :))

I gave up Usenet years ago, about when the signal to noise ratio fell
under 0.1% and I got tired of wasting time wading through the flames.

Now, on to your points about coding style. I'll say this once only. I
don't want to waste time with an argument about this.

> He and davej still have a point.  Your code formatting is
> non-standard, and is difficult to read.  A document exists
> CodingStyle which explains a good style, and further -why- it is a
> good style.

And one of the very first things that document says that coding style
is very personal and he doesn't want to force his views on others.
So, as a symbolic gesture, I printed out and burned that document.

> Among other reasons, because of long term maintenance.
> 
> How do you expect others in the Linux kernel community to review
> your code, if it is widely considered difficult to read?  How do you
> expect people to maintain your code when are no longer around?  The
> Linux kernel will be around long after you and I and others leave
> kernel development.  Others need to read and maintain this code.

If and when I step down as maintainer (if I do so, I'll publically
pass the baton to the new maintainer), the new maintainer can indent
to their preference. Until that time, *I'm* the maintainer, and *I*
need to be able to read the code efficiently. It's the part of the
kernel I spend the most time in, after all.

And the coding style used elsewhere in the kernel is revolting to
me. More importantly, it's harder for me to parse than my own style.
I shouldn't have to constantly stumble over an appalling coding style
in my own code!

"He who writes the code gets to choose".

				Regards,

					Richard....
Permanent: rgooch@atnf.csiro.au
Current:   rgooch@ras.ucalgary.ca

  parent reply	other threads:[~2001-11-13  4:38 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-10-16 16:27 Christoph Lameter
2001-10-16 17:07 ` John Levon
2001-10-16 17:22   ` Alexander Viro
2001-10-17  2:17 ` Keith Owens
2001-10-17  4:59   ` Christoph Lameter
2001-10-17  5:54     ` Keith Owens
2001-10-17  6:15       ` Christoph Lameter
2001-10-17 18:28         ` Gerhard Mack
2001-10-17 19:34           ` Wayne Whitney
2001-10-17 21:31             ` pierre
2001-10-17 20:32               ` Arjan van de Ven
2001-10-17 19:14                 ` David Lang
2001-10-17 20:37                   ` Arjan van de Ven
2001-10-17 19:20                     ` David Lang
2001-10-17 22:02                 ` pierre
2001-10-18 13:45                   ` M. R. Brown
2001-10-21 16:05                   ` Alan Cox
2001-10-18 13:32               ` M. R. Brown
2001-10-17 19:34           ` Ben Greear
2001-10-17 20:18           ` pierre
2001-10-17 14:15   ` John Levon
2001-11-13  2:57   ` Richard Gooch
2001-11-13  3:21     ` Dave Jones
2001-11-13  3:24     ` Richard Gooch
2001-11-13  3:53       ` Alexander Viro
2001-11-13  3:58       ` Richard Gooch
2001-11-13  4:10         ` Alexander Viro
2001-11-13  4:15         ` Jeff Garzik
2001-11-13  9:57           ` [OT] Coding Style (was: Re: GPLONLY kernel symbols???) Henning P. Schmiedehausen
2001-11-28 22:35           ` GPLONLY kernel symbols??? Dr. Kelsey Hudson
2001-11-13  4:21         ` Richard Gooch
2001-11-13  4:36           ` Alexander Viro
2001-11-13  4:49           ` Richard Gooch
2001-11-13 10:46           ` Martin Dalecki
2001-11-13  4:37         ` Richard Gooch [this message]
2001-11-13  4:49           ` Alexander Viro
2001-11-13  4:54           ` Jeff Garzik
2001-11-13  4:58           ` Davide Libenzi
2001-11-13  5:07             ` John Alvord
2001-11-28 22:41               ` Dr. Kelsey Hudson
2001-11-29  0:35                 ` Michael H. Warfield
2001-11-29  5:04                   ` Dr. Kelsey Hudson
2001-11-13  5:00           ` Richard Gooch
2001-11-13  5:14             ` Jeff Garzik
2001-11-13  5:29               ` Robert Love
2001-11-13  5:23             ` Richard Gooch
2001-11-13  5:44               ` Erik Andersen
2001-11-13  5:56               ` Richard Gooch
2001-11-13  5:03           ` Richard Gooch
2001-11-13  5:27             ` Alexander Viro
2001-11-13  5:35             ` Richard Gooch
2001-11-13  5:44               ` Jeff Garzik
2001-11-13 10:01           ` Alan Cox
2001-11-13 10:29             ` Jeff Garzik
2001-11-13 13:47             ` Dave Jones
2001-11-13 16:54               ` Alan Cox
2001-11-13 17:17                 ` Dave Jones
2001-11-15 10:03               ` [PATCH] mtrr (was Re: GPLONLY kernel symbols???) Paul Gortmaker
2001-11-13 10:48           ` GPLONLY kernel symbols??? Martin Dalecki
2001-11-13  9:52         ` Roman Zippel
2001-11-13  4:05     ` Jeff Garzik

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=200111130437.fAD4b2j17329@vindaloo.ras.ucalgary.ca \
    --to=rgooch@ras.ucalgary.ca \
    --cc=jgarzik@mandrakesoft.com \
    --cc=linux-kernel@vger.kernel.org \
    /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®