DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:date:message-id:subject:from:to:content-type;
        b=sOkUX844PzUNGQjeNYGp1wGdn/DapkHtax0XH/USpKrV63jpPZzzxDgKU8ZHYQZA0c
         IrD8um0LRlr3H8uf6y22TiVS81eRm7r4TNHhfh3a1kZBqfhAh6oN+oTydabtcm+unn8i
         pZcQqBFmA6hNV7pwNxyv4KFi1RjP5IaS9bzy8=
MIME-Version: 1.0
Date: Wed, 6 Jan 2010 22:11:54 +0800
Message-ID: <b3657a101001060611k57407d6aj94f5931a2f0a5f33@mail.gmail.com>
Subject: [PATCH]btrfs: Remove a extra condition
From: liu weni <qingshenlwy@gmail.com>
To: linux-btrfs@vger.kernel.org, linux-kernel <linux-kernel@vger.kernel.org>,
       zheng.yan@oracle.com, chris.mason@oracle.com,
       Xiao Guangrong <xiaoguangrong@cn.fujitsu.com>,
       strongzgy <strongzgy@gmail.com>, Liu Hui <onlyflyer@gmail.com>
Content-Type: text/plain; charset=ISO-8859-1
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 918
Lines: 33

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-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/