diff options
author | Rakesh Pandit <rakesh@tuxera.com> | 2014-03-20 11:43:35 +0200 |
---|---|---|
committer | David Sterba <dsterba@suse.cz> | 2014-04-11 19:05:40 +0200 |
commit | f84db1ca7039a714a7f7c559916f3e84ac1515ef (patch) | |
tree | 5ff77406fc0a05d00c949460c27e48e48b80cd50 /btrfs.c | |
parent | 1c1c7caf5545f25965cf94cd840f53a58ea67648 (diff) |
Btrfs-progs: btrfs: remove dead code in handle_options
Just cleanup: remove useless return type, while loop and dead code.
Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
Signed-off-by: David Sterba <dsterba@suse.cz>
Diffstat (limited to 'btrfs.c')
-rw-r--r-- | btrfs.c | 33 |
1 files changed, 11 insertions, 22 deletions
@@ -212,31 +212,20 @@ static int cmd_version(int argc, char **argv) return 0; } -static int handle_options(int *argc, char ***argv) +static void handle_options(int *argc, char ***argv) { - char **orig_argv = *argv; - - while (*argc > 0) { + if (*argc > 0) { const char *arg = (*argv)[0]; - if (arg[0] != '-') - break; - - if (!strcmp(arg, "--help")) { - break; - } else if (!strcmp(arg, "--version")) { - break; - } else { - fprintf(stderr, "Unknown option: %s\n", arg); - fprintf(stderr, "usage: %s\n", - btrfs_cmd_group.usagestr[0]); - exit(129); - } - - (*argv)++; - (*argc)--; + if (arg[0] != '-' || + !strcmp(arg, "--help") || + !strcmp(arg, "--version")) + return; + fprintf(stderr, "Unknown option: %s\n", arg); + fprintf(stderr, "usage: %s\n", + btrfs_cmd_group.usagestr[0]); + exit(129); } - - return (*argv) - orig_argv; + return; } static const struct cmd_group btrfs_cmd_group = { |