From 1a38741dce3e32218521cc2b00819dac1ccebc3a Mon Sep 17 00:00:00 2001 From: wm4 Date: Fri, 6 Feb 2015 15:42:12 +0100 Subject: options: allow ":" as separator in key-value lists Before this, unquoted occurrences of ":" lead to parsing errors. There's no reason to reject it, especially since the traditional MPlayer syntax uses ":" as separator. (Which is also the reason why ":" was rejected before: the parser shares this code for handling splitting/quoting, and we merely checked explicitly whether the option was split on ",".) --- options/m_option.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'options/m_option.c') diff --git a/options/m_option.c b/options/m_option.c index de65587884..74d6509111 100644 --- a/options/m_option.c +++ b/options/m_option.c @@ -1409,7 +1409,7 @@ static int parse_keyvalue_list(struct mp_log *log, const m_option_t *opt, MP_TARRAY_APPEND(NULL, lst, num, bstrto0(NULL, val)); } - if (!bstr_eatstart0(¶m, ",")) + if (!bstr_eatstart0(¶m, ",") && !bstr_eatstart0(¶m, ":")) break; } if (dst) -- cgit v1.2.3