diff options
author | wm4 <wm4@nowhere> | 2019-11-30 00:50:31 +0100 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2019-11-30 00:50:31 +0100 |
commit | 56b06907a6a68de7ccf44cbfa9c9c6fe0edfc627 (patch) | |
tree | 6a8e4e01f9545166bdd1c345b4e150a567118d0f /player | |
parent | 90df6c79c9400d3f1c711ef5696324908a5126ce (diff) | |
download | mpv-56b06907a6a68de7ccf44cbfa9c9c6fe0edfc627.tar.bz2 mpv-56b06907a6a68de7ccf44cbfa9c9c6fe0edfc627.tar.xz |
command: remove another unneeded hack
Now that the option-to-property bridge is gone, this is not needed
anymore. It always took the "silent" path.
Also, at least as of before this commit, this didn't correctly print a
warning when accessing a deprecated option as property. This was because
m_config_get_co_raw() was used, which intentionally does not print any
warnings, so switch to the non-raw one. (Affects only options that have
.deprecation_message set.)
Diffstat (limited to 'player')
-rw-r--r-- | player/command.c | 14 |
1 files changed, 2 insertions, 12 deletions
diff --git a/player/command.c b/player/command.c index 9e42b8367d..91dd6f80e4 100644 --- a/player/command.c +++ b/player/command.c @@ -104,8 +104,6 @@ struct command_ctx { char *cur_ipc; char *cur_ipc_input; - int silence_option_deprecations; - struct mp_cmd_ctx *cache_dump_cmd; // in progress cache dumping }; @@ -379,14 +377,8 @@ static int mp_property_generic_option(void *ctx, struct m_property *prop, int action, void *arg) { MPContext *mpctx = ctx; - const char *optname = prop->name; - struct m_config_option *opt; - if (mpctx->command_ctx->silence_option_deprecations) { - // This case is specifically for making --reset-on-next-file=all silent. - opt = m_config_get_co_raw(mpctx->mconfig, bstr0(optname)); - } else { - opt = m_config_get_co(mpctx->mconfig, bstr0(optname)); - } + struct m_config_option *opt = + m_config_get_co(mpctx->mconfig, bstr0(prop->name)); if (!opt) return M_PROPERTY_UNKNOWN; @@ -3543,9 +3535,7 @@ static int mp_property_do_silent(const char *name, int action, void *val, struct MPContext *ctx) { struct command_ctx *cmd = ctx->command_ctx; - cmd->silence_option_deprecations += 1; int r = m_property_do(ctx->log, cmd->properties, name, action, val, ctx); - cmd->silence_option_deprecations -= 1; if (r == M_PROPERTY_OK && is_property_set(action, val)) mp_notify_property(ctx, (char *)name); return r; |