Re: [PATCH 03/16] GFS2: bmap and inode functions

From: Pavel Machek
Date: Tue Sep 05 2006 - 16:16:19 EST


Hi!

> >> if(ip == NULL)
> >> return;
> >> if(test_bit(GLF_DIRTY, &gl->gl_flags))
> >> gfs_inode_attr_in(ip);
> >> gfs2_meta_cache_flush(ip);
> >
> >btw., it should be "if (", not "if(".
>
> There is no such rule in CodingStyle.

But there should be... so that if does not look like function call.

Pavel

--
Thanks for all the (sleeping) penguins.
-
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/