diff options
| author | Norton.Zhu <norton.zhu@huawei.com> | 2016-01-15 02:17:09 +0300 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-01-15 03:00:49 +0300 |
| commit | 3eb5bdf0f47726e3ef0a875e0cce3cca462ee70d (patch) | |
| tree | 7a6fcc72a9ad8f328901a30291ce7a4d224d56b1 /fs/ocfs2/file.c | |
| parent | 2886357b24662fcf7dc138c9d5e3166e99a55750 (diff) | |
| download | linux-3eb5bdf0f47726e3ef0a875e0cce3cca462ee70d.tar.xz | |
ocfs2: optimize bad declarations and redundant assignment
In ocfs2_parse_options,
a) it's better to declare variables(small size) outside of while loop;
b) 'option' will be set by match_int, 'option = 0;' makes no sense, if
match_int failed, it just goto bail and return.
Signed-off-by: Norton.Zhu <norton.zhu@huawei.com>
Reviewed-by: Joseph Qi <joseph.qi@huawei.com>
Cc: Gang He <ghe@suse.com>
Cc: Mark Fasheh <mfasheh@suse.de>
Acked-by: Joel Becker <jlbec@evilplan.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/ocfs2/file.c')
0 files changed, 0 insertions, 0 deletions