diff options
author | Colin Ian King <colin.king@canonical.com> | 2018-07-17 10:59:28 -0400 |
---|---|---|
committer | Andreas Gruenbacher <agruenba@redhat.com> | 2018-07-24 18:08:59 -0400 |
commit | d1b0cb933c8e638947ea72f3ab4e3dad4325bb96 (patch) | |
tree | c51be8a31fc80c01fa8fcab68bda83240f78e652 /fs/gfs2/dir.c | |
parent | f95cbb44abf9d6545769147d5abec4770c89872d (diff) |
gfs2: remove redundant variable 'moved'
Variable 'moved' s being assigned but is never used hence it is
redundant and can be removed. This has been the case ever since commit
c752666c.
Cleans up clang warning:
warning: variable 'moved' set but not used [-Wunused-but-set-variable]
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
Diffstat (limited to 'fs/gfs2/dir.c')
-rw-r--r-- | fs/gfs2/dir.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/gfs2/dir.c b/fs/gfs2/dir.c index d97ad89955d1..e37002560c11 100644 --- a/fs/gfs2/dir.c +++ b/fs/gfs2/dir.c | |||
@@ -1011,7 +1011,7 @@ static int dir_split_leaf(struct inode *inode, const struct qstr *name) | |||
1011 | u64 bn, leaf_no; | 1011 | u64 bn, leaf_no; |
1012 | __be64 *lp; | 1012 | __be64 *lp; |
1013 | u32 index; | 1013 | u32 index; |
1014 | int x, moved = 0; | 1014 | int x; |
1015 | int error; | 1015 | int error; |
1016 | 1016 | ||
1017 | index = name->hash >> (32 - dip->i_depth); | 1017 | index = name->hash >> (32 - dip->i_depth); |
@@ -1113,8 +1113,6 @@ static int dir_split_leaf(struct inode *inode, const struct qstr *name) | |||
1113 | 1113 | ||
1114 | if (!prev) | 1114 | if (!prev) |
1115 | prev = dent; | 1115 | prev = dent; |
1116 | |||
1117 | moved = 1; | ||
1118 | } else { | 1116 | } else { |
1119 | prev = dent; | 1117 | prev = dent; |
1120 | } | 1118 | } |