diff options
| author | wm4 <wm4@nowhere> | 2015-02-02 18:06:52 +0100 |
|---|---|---|
| committer | Diogo Franco (Kovensky) <diogomfranco@gmail.com> | 2015-02-03 14:36:36 +0900 |
| commit | 32e84b7c9149e48f7250077e557ecfdba5f2a04f (patch) | |
| tree | 4c238ecfa0c49ae706628b4d5a03a9311a86ecba | |
| parent | bae322888d2728adf163d3bac05e7c325310fac2 (diff) | |
| download | mpv-32e84b7c9149e48f7250077e557ecfdba5f2a04f.tar.bz2 mpv-32e84b7c9149e48f7250077e557ecfdba5f2a04f.tar.xz | |
options: slightly improve help output for choice and flag options
| -rw-r--r-- | options/m_option.c | 21 |
1 files changed, 12 insertions, 9 deletions
diff --git a/options/m_option.c b/options/m_option.c index 32a38911e4..280821e4ca 100644 --- a/options/m_option.c +++ b/options/m_option.c @@ -135,8 +135,12 @@ static int parse_flag(struct mp_log *log, const m_option_t *opt, VAL(dst) = 0; return 1; } - mp_err(log, "Invalid parameter for %.*s flag: %.*s\n", - BSTR_P(name), BSTR_P(param)); + mp_fatal(log, "Invalid parameter for %.*s flag: %.*s\n", + BSTR_P(name), BSTR_P(param)); + mp_info(log, "Valid values are:\n"); + mp_info(log, " yes\n"); + mp_info(log, " no\n"); + mp_info(log, " (passing nothing)\n"); return M_OPT_INVALID; } else { if (dst) @@ -545,20 +549,19 @@ static int parse_choice(struct mp_log *log, const struct m_option *opt, return M_OPT_MISSING_PARAM; if ((opt->flags & M_OPT_MIN) && (opt->flags & M_OPT_MAX)) { long long val; - if (parse_longlong(log, opt, name, param, &val) == 1) { + if (parse_longlong(mp_null_log, opt, name, param, &val) == 1) { if (dst) *(int *)dst = val; return 1; } } - mp_err(log, "Invalid value for option %.*s: %.*s\n", - BSTR_P(name), BSTR_P(param)); - mp_err(log, "Valid values are:"); + mp_fatal(log, "Invalid value for option %.*s: %.*s\n", + BSTR_P(name), BSTR_P(param)); + mp_info(log, "Valid values are:\n"); for (alt = opt->priv; alt->name; alt++) - mp_err(log, " %s", alt->name); + mp_info(log, " %s\n", alt->name[0] ? alt->name : "(passing nothing)"); if ((opt->flags & M_OPT_MIN) && (opt->flags & M_OPT_MAX)) - mp_err(log, " %g-%g", opt->min, opt->max); - mp_err(log, "\n"); + mp_info(log, " %g-%g (integer range)\n", opt->min, opt->max); return M_OPT_INVALID; } if (dst) |