Re: [Patch 02/18] include/linux/logfs.h

From: JÃrn Engel
Date: Thu Aug 09 2007 - 16:07:32 EST


On Thu, 9 August 2007 00:56:29 +0200, Arnd Bergmann wrote:
> On Wednesday 08 August 2007, JÃrn Engel wrote:
> > +++ linux-2.6.21logfs/include/linux/logfs.hÂÂÂÂÂ2007-08-08 02:57:37.000000000 +0200
> > @@ -0,0 +1,500 @@
> > +/*
> > + * fs/logfs/logfs.h
> > + *
>
> The comment does not match the file name. Better remove the file names
> entirely from introduction comments.

Maybe. This file needs to get moved again anyway.

JÃrn

--
Debugging is twice as hard as writing the code in the first place.
Therefore, if you write the code as cleverly as possible, you are,
by definition, not smart enough to debug it.
-- Brian W. Kernighan
-
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/