On Fri, 05 Feb 2010 23:15:26 +0900, Jiro SEKIBA <jir@xxxxxxxxx> wrote: > Hi, > > This is a trivial patch to delete unnecessary condition in nilfs_dat_translate. > > nilfs_dat_translate() will asign translated address to *blocknrp if blocknrp > is not NULL. However the condition is unneeded, because all callers of > nilfs_dat_translate() pass blocknrp properly. > > Signed-off-by: Jiro SEKIBA <jir@xxxxxxxxx> Applied. Ryusuke Konishi > --- > fs/nilfs2/dat.c | 3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) > > diff --git a/fs/nilfs2/dat.c b/fs/nilfs2/dat.c > index 187dd07..9d1e5de 100644 > --- a/fs/nilfs2/dat.c > +++ b/fs/nilfs2/dat.c > @@ -388,8 +388,7 @@ int nilfs_dat_translate(struct inode *dat, __u64 vblocknr, sector_t *blocknrp) > ret = -ENOENT; > goto out; > } > - if (blocknrp != NULL) > - *blocknrp = blocknr; > + *blocknrp = blocknr; > > out: > kunmap_atomic(kaddr, KM_USER0); > -- > 1.5.6.5 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html