diff options
| author | Chris Ball <cjb@laptop.org> | 2011-03-07 10:05:21 -0500 |
|---|---|---|
| committer | Chris Mason <chris.mason@oracle.com> | 2011-10-25 09:18:32 -0400 |
| commit | fcdc0929c6ea051dad59818210df53fd03eaf4b1 (patch) | |
| tree | d2adab24e3a8f10db07ee82b1bdd3db4fc77ca67 /extent-cache.c | |
| parent | 29e6fc2aa637956ae57c5bf70208dea12e5b78cb (diff) | |
Fix unused-but-set errors in gcc-4.6
gcc-4.6 (as shipped in Fedora) turns on -Wunused-but-set-variable by
default, which breaks the build when combined with -Wall, e.g.:
debug-tree.c: In function ‘print_extent_leaf’:
debug-tree.c:45:13: error: variable ‘last_len’ set but not used [-Werror=unused-but-set-variable]
debug-tree.c:44:13: error: variable ‘last’ set but not used [-Werror=unused-but-set-variable]
debug-tree.c:41:21: error: variable ‘item’ set but not used [-Werror=unused-but-set-variable]
cc1: all warnings being treated as errors
This patch fixes the errors by removing the unused variables.
Signed-off-by: Chris Ball <cjb@laptop.org>
Signed-off-by: Hugo Mills <hugo@carfax.org.uk>
Diffstat (limited to 'extent-cache.c')
| -rw-r--r-- | extent-cache.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/extent-cache.c b/extent-cache.c index b871e18f..3dd6434e 100644 --- a/extent-cache.c +++ b/extent-cache.c @@ -96,13 +96,11 @@ int insert_existing_cache_extent(struct cache_tree *tree, struct cache_extent *pe) { struct rb_node *found; - struct cache_extent *entry; found = tree_insert(&tree->root, pe->start, pe->size, &pe->rb_node); - if (found) { - entry = rb_entry(found, struct cache_extent, rb_node); + if (found) return -EEXIST; - } + return 0; } |