diff options
| author | Zhao Lei <zhaolei@cn.fujitsu.com> | 2015-08-06 11:05:54 +0800 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2015-10-19 11:48:24 +0200 |
| commit | 7cf60f4660d4705094eb111e4c52114f8dc348dc (patch) | |
| tree | d2032c9662d14278e4c4ea5927b4e5857a3b0e35 /cmds-scrub.c | |
| parent | 9c3ce57e6bdb8f6af86e2c517bc8ea713e93a2c2 (diff) | |
| download | btrfs-progs-7cf60f4660d4705094eb111e4c52114f8dc348dc.tar.gz | |
btrfs-progs: use switch instead of a series of ifs for output error message
Signed-off-by: Zhao Lei <zhaolei@cn.fujitsu.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'cmds-scrub.c')
| -rw-r--r-- | cmds-scrub.c | 35 |
1 files changed, 20 insertions, 15 deletions
diff --git a/cmds-scrub.c b/cmds-scrub.c index 3ebda9d..1c3fc03 100644 --- a/cmds-scrub.c +++ b/cmds-scrub.c @@ -1469,21 +1469,26 @@ static int scrub_start(int argc, char **argv, int resume) ++err; continue; } - if (sp[i].ret && sp[i].ioctl_errno == ENODEV) { - if (do_print) - fprintf(stderr, "WARNING: device %lld not " - "present\n", devid); - continue; - } - if (sp[i].ret && sp[i].ioctl_errno == ECANCELED) { - ++err; - } else if (sp[i].ret) { - if (do_print) - fprintf(stderr, "ERROR: scrubbing %s failed " - "for device id %lld (%s)\n", path, - devid, strerror(sp[i].ioctl_errno)); - ++err; - continue; + if (sp[i].ret) { + switch (sp[i].ioctl_errno) { + case ENODEV: + if (do_print) + fprintf(stderr, + "WARNING: device %lld not present\n", + devid); + continue; + case ECANCELED: + ++err; + break; + default: + if (do_print) + fprintf(stderr, + "ERROR: scrubbing %s failed for device id %lld (%s)\n", + path, devid, + strerror(sp[i].ioctl_errno)); + ++err; + continue; + } } if (sp[i].scrub_args.progress.uncorrectable_errors > 0) e_uncorrectable++; |