diff options
author | Harvey Harrison <harvey.harrison@gmail.com> | 2008-04-28 05:16:21 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-04-28 11:58:46 -0400 |
commit | 8acc570fabc022d41baedc0c9bf976a4b9d16d6f (patch) | |
tree | 8edd409aa140bff7674c5734daef4db9412c2942 | |
parent | e13601bc6aa427a6483e7f66fde55ac3b5fed335 (diff) |
reiserfs: fix more sparse warnings in do_balan.c
fs/reiserfs/do_balan.c:1467:10: warning: symbol 'ret_val' shadows an earlier one
fs/reiserfs/do_balan.c:275:6: originally declared here
fs/reiserfs/do_balan.c:1471:23: warning: symbol 'ih' shadows an earlier one
fs/reiserfs/do_balan.c:249:67: originally declared here
Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com>
Cc: Chris Mason <chris.mason@oracle.com>
Cc: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | fs/reiserfs/do_balan.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/fs/reiserfs/do_balan.c b/fs/reiserfs/do_balan.c index 7ee4208793b6..2f87f5b14630 100644 --- a/fs/reiserfs/do_balan.c +++ b/fs/reiserfs/do_balan.c | |||
@@ -1464,29 +1464,29 @@ static int balance_leaf(struct tree_balance *tb, struct item_head *ih, /* item h | |||
1464 | } else | 1464 | } else |
1465 | /* item falls wholly into S_new[i] */ | 1465 | /* item falls wholly into S_new[i] */ |
1466 | { | 1466 | { |
1467 | int ret_val; | 1467 | int leaf_mi; |
1468 | struct item_head *pasted; | 1468 | struct item_head *pasted; |
1469 | 1469 | ||
1470 | #ifdef CONFIG_REISERFS_CHECK | 1470 | #ifdef CONFIG_REISERFS_CHECK |
1471 | struct item_head *ih = | 1471 | struct item_head *ih_check = |
1472 | B_N_PITEM_HEAD(tbS0, item_pos); | 1472 | B_N_PITEM_HEAD(tbS0, item_pos); |
1473 | 1473 | ||
1474 | if (!is_direntry_le_ih(ih) | 1474 | if (!is_direntry_le_ih(ih_check) |
1475 | && (pos_in_item != ih_item_len(ih) | 1475 | && (pos_in_item != ih_item_len(ih_check) |
1476 | || tb->insert_size[0] <= 0)) | 1476 | || tb->insert_size[0] <= 0)) |
1477 | reiserfs_panic(tb->tb_sb, | 1477 | reiserfs_panic(tb->tb_sb, |
1478 | "PAP-12235: balance_leaf: pos_in_item must be equal to ih_item_len"); | 1478 | "PAP-12235: balance_leaf: pos_in_item must be equal to ih_item_len"); |
1479 | #endif /* CONFIG_REISERFS_CHECK */ | 1479 | #endif /* CONFIG_REISERFS_CHECK */ |
1480 | 1480 | ||
1481 | ret_val = | 1481 | leaf_mi = |
1482 | leaf_move_items(LEAF_FROM_S_TO_SNEW, | 1482 | leaf_move_items(LEAF_FROM_S_TO_SNEW, |
1483 | tb, snum[i], | 1483 | tb, snum[i], |
1484 | sbytes[i], | 1484 | sbytes[i], |
1485 | S_new[i]); | 1485 | S_new[i]); |
1486 | 1486 | ||
1487 | RFALSE(ret_val, | 1487 | RFALSE(leaf_mi, |
1488 | "PAP-12240: unexpected value returned by leaf_move_items (%d)", | 1488 | "PAP-12240: unexpected value returned by leaf_move_items (%d)", |
1489 | ret_val); | 1489 | leaf_mi); |
1490 | 1490 | ||
1491 | /* paste into item */ | 1491 | /* paste into item */ |
1492 | bi.tb = tb; | 1492 | bi.tb = tb; |