This patch remove a extra condition. In this place, the outer is if (!cur || !uptodate) and then, there are 2 ways: !cur or ! uptodate. The "else if (!uptodate)" is not necessary. I think "else" is better. --- Signed-off-by: Liuwenyi <qingshenlwy@gmail.com> --- diff --git a/fs/btrfs/ctree.c b/fs/btrfs/ctree.c index c4bc570..654f702 100644 --- a/fs/btrfs/ctree.c +++ b/fs/btrfs/ctree.c @@ -679,7 +679,7 @@ int btrfs_realloc_node(struct btrfs_trans_handle *trans, if (!cur) { cur = read_tree_block(root, blocknr, blocksize, gen); - } else if (!uptodate) { + } else { btrfs_read_buffer(cur, gen); } } --- Best Regards, Liuwenyi 20100107 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html