Re: [PATCH 4/4] MultiAdmin module

From: Greg KH
Date: Mon May 01 2006 - 11:07:24 EST


On Mon, May 01, 2006 at 03:50:21PM +0200, Jan Engelhardt wrote:
>
> [PATCH 4/4] MultiAdmin module
>
> - Add the MultiAdmin to the mainline tree.
> I hope the rest is self-explanatory.
>
> Please do not mention CodingStyle for multiadm.c. I already know it. :)
> And I will get to it should it really be merged.

No one will review it if it isn't in the proper CodingStyle.

We have a coding style for a reason, it makes it a very simple thing for
anyone to review the code as the patterns are all the same. It turns
out that people's brains get trained to ignore the patterns and see the
details instead. Lots of research backs this up.

So switch to the common coding style, otherwise no one will look at your
code (or if they do, odds are they will miss a lot...)

thanks,

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