diff options
author | wm4 <wm4@nowhere> | 2016-09-18 16:06:12 +0200 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2016-09-18 16:08:21 +0200 |
commit | 2415b695724c15def54a039527ee9eb574ffb65f (patch) | |
tree | 97ed9e4c272bf3ac29764557db73b24d99503945 /options/m_config.c | |
parent | 3ecc6d0a7934c42395d863b74e47f903fe864760 (diff) | |
download | mpv-2415b695724c15def54a039527ee9eb574ffb65f.tar.bz2 mpv-2415b695724c15def54a039527ee9eb574ffb65f.tar.xz |
player: more option/property consistency fixes
Some properties had a different type from their equivalent options (such
as mute, volume, deinterlace, edition). This wasn't really sane, as raw
option values should be always within their bounds. On the other hand,
these properties use a different type to reflect runtime limits (such as
range of available editions), or simply to improve the "UI" (you don't
want to cycle throuhg the completely useless "auto" value when cycling
the "mute" property).
Handle this by making them always return the option type, but also
allowing them to provide a "constricted" type, which is used for UI
purposes. All M_PROPERTY_GET_CONSTRICTED_TYPE changes are related to
this.
One consequence is that you can set the volume property to arbitrary
high values just like with the --volume option, but using the "add"
command it still restricts it to the --volume-max range.
Also deprecate --chapter, as it is grossly incompatible to the chapter
property. We pondered renaming it to --chapters, or introducing a more
powerful --range option, but concluded that --start --end is actually
enough.
These changes appear to take care of the last gross property/option
incompatibilities, although there might still be a few lurking.
Diffstat (limited to 'options/m_config.c')
-rw-r--r-- | options/m_config.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/options/m_config.c b/options/m_config.c index 7cc9a46792..6dd72380d9 100644 --- a/options/m_config.c +++ b/options/m_config.c @@ -87,9 +87,6 @@ struct m_opt_backup { void *backup; }; -static struct m_config_option *m_config_get_co_raw(const struct m_config *config, - struct bstr name); - static int parse_include(struct m_config *config, struct bstr param, bool set, int flags) { @@ -556,8 +553,8 @@ static void m_config_add_option(struct m_config *config, MP_TARRAY_APPEND(config, config->opts, config->num_opts, co); } -static struct m_config_option *m_config_get_co_raw(const struct m_config *config, - struct bstr name) +struct m_config_option *m_config_get_co_raw(const struct m_config *config, + struct bstr name) { if (!name.len) return NULL; |