Re: [patch 2/3] whitespace cleanups in fs/cifs/file.c

From: Jörn Engel
Date: Wed Dec 29 2004 - 07:11:48 EST


On Wed, 29 December 2004 10:51:30 +0100, Wichert Akkerman wrote:
> Previously Jörn Engel wrote:
> > On Wed, 29 December 2004 00:52:32 +0100, Jesper Juhl wrote:
> > > -static int cifs_relock_file(struct cifsFileInfo * cifsFile)
> > > +static int
> > > +cifs_relock_file(struct cifsFileInfo *cifsFile)
> >
> > Linus viciously prefers to keep return type and function name on a
> > single line. I cannot quite follow his reasoning, but would leave
> > that part out, unless explicitly requested by Steve.
>
> If they are on a single line grep will find the return type as well
> which is extremely convenient.

Ok, that reasoning I can follow. Thanks, Wichert!

Jörn

--
Anything that can go wrong, will.
-- Finagle's Law
-
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/