Re: GPLONLY kernel symbols???

Jeff Garzik (jgarzik@mandrakesoft.com)
Mon, 12 Nov 2001 23:15:00 -0500


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...

> > > 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 :))

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.

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.

Jeff

-- 
Jeff Garzik      | Only so many songs can be sung
Building 1024    | with two lips, two lungs, and one tongue.
MandrakeSoft     |         - nomeansno

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/