diff options
author | pavelxdd <pavel.otchertsov@gmail.com> | 2017-12-16 01:29:05 +0300 |
---|---|---|
committer | pavelxdd <pavel.otchertsov@gmail.com> | 2017-12-16 01:29:05 +0300 |
commit | 7fc9ff54d37aa7def8bf3cf5eb7b86b29722c129 (patch) | |
tree | 109b75a876b24abed651df0a0915df3df8cf92ee | |
parent | 0e311fc0e8658a2c1f60bd57366c854e7fc129bc (diff) | |
download | mpv-7fc9ff54d37aa7def8bf3cf5eb7b86b29722c129.tar.bz2 mpv-7fc9ff54d37aa7def8bf3cf5eb7b86b29722c129.tar.xz |
options: don't report errors on help value for OPT_SIZE_BOX
The same idea as in 3723e61 but for OPT_SIZE_BOX now.
Affects options `autofit`, `autofit-larger` and `autofit-smaller`.
-rw-r--r-- | options/m_option.c | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/options/m_option.c b/options/m_option.c index e08cfa882b..b03ad014b8 100644 --- a/options/m_option.c +++ b/options/m_option.c @@ -1996,6 +1996,10 @@ const m_option_type_t m_option_type_geometry = { static int parse_size_box(struct mp_log *log, const m_option_t *opt, struct bstr name, struct bstr param, void *dst) { + bool is_help = bstr_equals0(param, "help"); + if (is_help) + goto error; + struct m_geometry gm; if (!parse_geometry_str(&gm, param)) goto error; @@ -2009,10 +2013,12 @@ static int parse_size_box(struct mp_log *log, const m_option_t *opt, return 1; error: - mp_err(log, "Option %.*s: invalid size: '%.*s'\n", - BSTR_P(name), BSTR_P(param)); - mp_err(log, "Valid format: W[%%][xH[%%]] or empty string\n"); - return M_OPT_INVALID; + if (!is_help) { + mp_err(log, "Option %.*s: invalid size: '%.*s'\n", + BSTR_P(name), BSTR_P(param)); + } + mp_info(log, "Valid format: W[%%][xH[%%]] or empty string\n"); + return is_help ? M_OPT_EXIT : M_OPT_INVALID; } const m_option_type_t m_option_type_size_box = { |