Mirrors/mpv
1
0
Fork 0
mirror of https://github.com/mpv-player/mpv.git synced 2025-12-23 19:30:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

m_option: remove unneeded legacy syntax parsing

Browse Source 
This code got shuffled around over the years but fundamentally it was
first introduced in 2c22fcd350 to fix a
parsing issue with specifically the af and vf options. Syntax like af*
and vf* used to be valid and this was part of dealing with that case.
Later 50008adf4a completely changed this
and ripped out support for the wildcard behavior. But these few lines to
deal with the special case were never removed. The token splitting has
been unnecessary for years and is confusing, so clean this up and
finally remove it.
This commit is contained in:
Dudemanguy
2025-07-14 19:00:10 -05:00
parent 2763390c15
commit 5c4c27e6b1
1 changed files with 1 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
options/m_option.c
View File

@@ -123,12 +123,7 @@ int m_option_set_node_or_string(struct mp_log *log, const m_option_t *opt,
struct bstr name, void *dst, struct mpv_node *src)
{
if (src->format == MPV_FORMAT_STRING) {
// The af and vf option unfortunately require this, because the
// option name includes the "action".
bstr optname = name, a, b;
if (bstr_split_tok(optname, "/", &a, &b))
optname = b;
return m_option_parse(log, opt, optname, bstr0(src->u.string), dst);
return m_option_parse(log, opt, name, bstr0(src->u.string), dst);
} else {
return m_option_set_node(opt, dst, src);
}