Messages in this thread Patch in this message | | | Date | Sat, 20 Mar 2010 14:37:31 +0300 | From | Dan Carpenter <> | Subject | [patch 2/3] btrfs cleanup: remove more dead code |
| |
rb_node cannot be an ERR_PTR() here so I removed the misleading check. Also I removed some more dead code at the bottom.
It looks like the code from lookup_extent_mapping() may have been copy and pasted into search_extent_mapping()?
Signed-off-by: Dan Carpenter <error27@gmail.com> --- This doesn't change anything. Could you put it into linux-next?
diff --git a/fs/btrfs/extent_map.c b/fs/btrfs/extent_map.c index 94668b1..010f299 100644 --- a/fs/btrfs/extent_map.c +++ b/fs/btrfs/extent_map.c @@ -380,16 +380,7 @@ struct extent_map *search_extent_mapping(struct extent_map_tree *tree, em = NULL; goto out; } - if (IS_ERR(rb_node)) { - em = ERR_PTR(PTR_ERR(rb_node)); - goto out; - } em = rb_entry(rb_node, struct extent_map, rb_node); - goto found; - - em = NULL; - goto out; - found: atomic_inc(&em->refs); out:
| |