condingstyle, was Re: utrace comments

[Christoph Hellwig <hch@infradead.org>]

I've separated this out under a new subject because some style issues
that so far aren't documented explicitly are in doubt here, and Roland
wants and Answer from Andrew.

We also should put clauses on this into CodingStyle.


On Sun, Apr 29, 2007 at 09:02:13PM -0700, Roland McGrath wrote:
> > 	The coding style here is wrong.  The else should be on the line
> > 	of the closing brace.  
> 
> I can ordinarily ignore syntax, but this is an abomination in the sight
> of the Lord and always will be.  Fortunately, it's far from being 100%
> consistently used in the kernel already.  People are welcome to change
> the code after I submit it, but I just can't make myself write it that
> way, sorry.

> > 	This doesn't follow kernel coding style at all, we always
> > 	put the && or || operators at the end of the closing line.
> 
> I could swear I've been "corrected" in the opposite direction on this one.
> It is not mentioned in Documentation/CodingStyle, and the existing kernel
> code is far from consistent on it.  I really don't care which way it is,
> but I'd like clear authoritative direction from Linus and Andrew before I
> bother with it.