diff options
| author | Guido Cella <guido@guidocella.xyz> | 2021-08-03 12:11:10 +0200 |
|---|---|---|
| committer | Dudemanguy <random342@airmail.cc> | 2021-08-03 13:17:12 +0000 |
| commit | dd7a49eb71a65c488770458cbac3bc48cfee6b80 (patch) | |
| tree | 536752d48fdb6fe22e3051999fe5fafc84f23187 | |
| parent | 416668d3c8bb5d09ebb3d5e3dbe715856165898b (diff) | |
| download | mpv-dd7a49eb71a65c488770458cbac3bc48cfee6b80.tar.bz2 mpv-dd7a49eb71a65c488770458cbac3bc48cfee6b80.tar.xz | |
options: don't always set set-locally to true
Since 1d1d1fbff9 option-info/<name>/set-locally was being set to true
for every option. This broke setting start from ytdl-hook, which doesn't
overwrite start if it was set-locally. Fix this so that only adding an
option to reset-on-next-file or setting file-local-options/<name> make
set-locally true like before.
However, it's arguable that just adding an option to
reset-on-next-file without ever changing it should not make set-locally
true, so that e.g.
--reset-on-next-file=start 'https://youtube.com/watch?v=...&t=30'
will start at 30, though it currently doesn't.
Fixes #9081.
| -rw-r--r-- | options/m_config_frontend.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/options/m_config_frontend.c b/options/m_config_frontend.c index f8dbbcf5b5..fdb03e0898 100644 --- a/options/m_config_frontend.c +++ b/options/m_config_frontend.c @@ -283,8 +283,7 @@ void m_config_backup_all_opts(struct m_config *config) void m_config_backup_watch_later_opts(struct m_config *config) { for (int n = 0; n < config->num_opts; n++) - ensure_backup(&config->watch_later_backup_opts, BACKUP_LOCAL, - &config->opts[n]); + ensure_backup(&config->watch_later_backup_opts, 0, &config->opts[n]); } struct m_config_option *m_config_get_co_raw(const struct m_config *config, |