From eb8bc57240d5d3e4680ff1df18a0a7792e96bd0c Mon Sep 17 00:00:00 2001 From: Stefano Sabatini Date: Tue, 24 May 2011 18:14:42 +0200 Subject: cmdutils: remove OPT_FUNC2 Make ff* tools only accept opt_* functions taking two arguments. The distinction between functions with one and two arguments is quite pointless. Simplify parse_options() code. --- cmdutils.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) (limited to 'cmdutils.c') diff --git a/cmdutils.c b/cmdutils.c index 8c9542accb..2bc6b7417f 100644 --- a/cmdutils.c +++ b/cmdutils.c @@ -273,15 +273,13 @@ unknown_opt: *po->u.int64_arg = parse_number_or_die(opt, arg, OPT_INT64, INT64_MIN, INT64_MAX); } else if (po->flags & OPT_FLOAT) { *po->u.float_arg = parse_number_or_die(opt, arg, OPT_FLOAT, -INFINITY, INFINITY); - } else if (po->flags & OPT_FUNC2) { - if (po->u.func2_arg(opt, arg) < 0) { - fprintf(stderr, "%s: failed to set value '%s' for option '%s'\n", argv[0], arg, opt); - exit(1); - } } else if (po->flags & OPT_DUMMY) { /* Do nothing for this option */ } else { - po->u.func_arg(arg); + if (po->u.func_arg(opt, arg) < 0) { + fprintf(stderr, "%s: failed to set value '%s' for option '%s'\n", argv[0], arg, opt); + exit(1); + } } if(po->flags & OPT_EXIT) exit(0); -- cgit v1.2.3