Messages in this thread |  | | Date | Tue, 5 Sep 2006 20:18:46 +0000 | From | Pavel Machek <> | Subject | Re: [PATCH 03/16] GFS2: bmap and inode functions |
| |
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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |