diff options
author | wm4 <wm4@nowhere> | 2018-01-24 03:20:15 +0100 |
---|---|---|
committer | Kevin Mitchell <kevmitch@gmail.com> | 2018-01-25 20:18:32 -0800 |
commit | 0d359879c9edfaefe8f4d500802781e32f2c8d9a (patch) | |
tree | b412fe69f8e0723ef11be2b9018e2e43768312f0 /player/command.c | |
parent | 9e64b9382265fb688fecda15b28ad6c41b587cd6 (diff) | |
download | mpv-0d359879c9edfaefe8f4d500802781e32f2c8d9a.tar.bz2 mpv-0d359879c9edfaefe8f4d500802781e32f2c8d9a.tar.xz |
command: add a change-list command
Requested. See manpage additions.
The main reason why this goes through the trouble to keep the
action/operation parameter separate is so that we don't expose some
option parser implementation details to the command (although that is a
relatively weak reason), and also to make it more different from the
"set" command, which can't support this type of option as it goes
through the property layer.
Fixes #5435.
Diffstat (limited to 'player/command.c')
-rw-r--r-- | player/command.c | 31 |
1 files changed, 31 insertions, 0 deletions
diff --git a/player/command.c b/player/command.c index 7b36dcd7a2..307a5535cb 100644 --- a/player/command.c +++ b/player/command.c @@ -4918,6 +4918,37 @@ int run_command(struct MPContext *mpctx, struct mp_cmd *cmd, struct mpv_node *re break; } + case MP_CMD_CHANGE_LIST: { + char *name = cmd->args[0].v.s; + char *op = cmd->args[1].v.s; + char *value = cmd->args[2].v.s; + struct m_config_option *co = m_config_get_co(mpctx->mconfig, bstr0(name)); + if (!co) { + set_osd_msg(mpctx, osdl, osd_duration, "Unknown option: '%s'", name); + return -1; + } + const struct m_option_type *type = co->opt->type; + bool found = false; + for (int i = 0; type->actions && type->actions[i].name; i++) { + const struct m_option_action *action = &type->actions[i]; + if (strcmp(action->name, op) == 0) + found = true; + } + if (!found) { + set_osd_msg(mpctx, osdl, osd_duration, "Unknown action: '%s'", op); + return -1; + } + char *optname = mp_tprintf(80, "%s-%s", name, op); // the dirty truth + int r = m_config_set_option_cli(mpctx->mconfig, bstr0(optname), + bstr0(value), M_SETOPT_RUNTIME); + if (r < 0) { + set_osd_msg(mpctx, osdl, osd_duration, + "Failed setting option: '%s'", name); + return -1; + } + break; + } + case MP_CMD_ADD: case MP_CMD_CYCLE: { |