Messages in this thread | | | Date | Wed, 29 Dec 2004 13:10:33 +0100 | From | Jörn Engel <> | Subject | Re: [patch 2/3] whitespace cleanups in fs/cifs/file.c |
| |
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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |