From dc48893630d11829121a45137460afb45e24ef2d Mon Sep 17 00:00:00 2001 From: wm4 Date: Sat, 17 Sep 2016 18:07:40 +0200 Subject: options: simplify M_OPT_EXIT There were multiple values under M_OPT_EXIT (M_OPT_EXIT-n for n>=0). Somehow M_OPT_EXIT-n either meant error code n (with n==0 no error?), or the number of option valus consumed (0 or 1). The latter is MPlayer legacy, which left it to the option type parsers to determine whether an option took a value or not. All of this was changed in mpv, by requiring the user to use explicit syntax ("--opt=val" instead of "-opt val"). In any case, the n value wasn't even used (anymore), so rip this all out. Now M_OPT_EXIT-1 doesn't mean anything, and could be used by a new error code. --- options/m_option.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'options/m_option.c') diff --git a/options/m_option.c b/options/m_option.c index fa9a9578ab..4ef5481ceb 100644 --- a/options/m_option.c +++ b/options/m_option.c @@ -2104,7 +2104,7 @@ static int parse_imgfmt(struct mp_log *log, const m_option_t *opt, mp_info(log, " %s", list[i]); mp_info(log, "\n"); talloc_free(list); - return M_OPT_EXIT - 1; + return M_OPT_EXIT; } unsigned int fmt = mp_imgfmt_from_name(param, true); @@ -2175,7 +2175,7 @@ static int parse_afmt(struct mp_log *log, const m_option_t *opt, for (int i = 1; i < AF_FORMAT_COUNT; i++) mp_info(log, " %s", af_fmt_to_str(i)); mp_info(log, "\n"); - return M_OPT_EXIT - 1; + return M_OPT_EXIT; } int fmt = 0; @@ -2219,7 +2219,7 @@ static int parse_channels(struct mp_log *log, const m_option_t *opt, mp_info(log, "\nOther values:\n" " auto-safe\n"); } - return M_OPT_EXIT - 1; + return M_OPT_EXIT; } bool auto_safe = bstr_equals0(param, "auto-safe"); @@ -2670,7 +2670,7 @@ static int get_obj_param(struct mp_log *log, bstr opt_name, bstr obj_name, BSTR_P(opt_name), BSTR_P(obj_name), BSTR_P(name)); return M_OPT_UNKNOWN; } - if (r > M_OPT_EXIT) + if (r != M_OPT_EXIT) mp_err(log, "Option %.*s: " "Error while parsing %.*s parameter %.*s (%.*s)\n", BSTR_P(opt_name), BSTR_P(obj_name), BSTR_P(name), @@ -2696,7 +2696,7 @@ static int get_obj_param(struct mp_log *log, bstr opt_name, bstr obj_name, } r = m_config_set_option_ext(config, bstr0(opt), val, flags); if (r < 0) { - if (r > M_OPT_EXIT) + if (r != M_OPT_EXIT) mp_err(log, "Option %.*s: " "Error while parsing %.*s parameter %s (%.*s)\n", BSTR_P(opt_name), BSTR_P(obj_name), opt, BSTR_P(val)); @@ -2776,7 +2776,7 @@ print_help: ; mp_warn(log, "Option %.*s: item %.*s doesn't exist.\n", BSTR_P(opt_name), BSTR_P(name)); } - r = M_OPT_EXIT - 1; + r = M_OPT_EXIT; exit: free_str_list(&args); @@ -2973,7 +2973,7 @@ static int parse_obj_settings_list(struct mp_log *log, const m_option_t *opt, } } mp_info(log, "\n"); - return M_OPT_EXIT - 1; + return M_OPT_EXIT; } if (op == OP_CLR) { -- cgit v1.2.3