Messages in this thread | | | Date | Sat, 2 Sep 2006 18:23:45 +0200 | From | Ingo Molnar <> | Subject | Re: [PATCH 03/16] GFS2: bmap and inode functions |
| |
* Jan Engelhardt <jengelh@linux01.gwdg.de> wrote:
> >> How about inverting the if() to: > >> > >> 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.
still it's part of the proper coding style :)
Ingo
-- VGER BF report: H 0 - 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/
| |