From 36b7cff5826f2d9352831bce9a6a84a638acb44b Mon Sep 17 00:00:00 2001 From: Guido Cella Date: Wed, 21 Jul 2021 18:43:52 +0200 Subject: manpage: fix watch-later-options examples --watch-later-options-remove doesn't accept multiple options, so split the example. Also suggest the more correct -clr to empty the list, and remove the workaround to not print an error with --watch-later-options= --- DOCS/man/options.rst | 5 +++-- options/m_config_frontend.c | 4 +--- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/DOCS/man/options.rst b/DOCS/man/options.rst index 0745eba0e9..a91c4e670e 100644 --- a/DOCS/man/options.rst +++ b/DOCS/man/options.rst @@ -867,9 +867,10 @@ Program Behavior - ``--watch-later-options-remove=fullscreen`` Resuming a file won't restore the fullscreen state. - - ``--watch-later-options-remove=volume,mute`` + - ``--watch-later-options-remove=volume`` + ``--watch-later-options-remove=mute`` Resuming a file won't restore the volume or mute state. - - ``--watch-later-options=`` + - ``--watch-later-options-clr`` Resuming a file won't restore any option except the starting position. diff --git a/options/m_config_frontend.c b/options/m_config_frontend.c index 4a8c583f91..f8dbbcf5b5 100644 --- a/options/m_config_frontend.c +++ b/options/m_config_frontend.c @@ -249,9 +249,7 @@ bool m_config_watch_later_backup_opt_changed(struct m_config *config, { struct m_config_option *co = m_config_get_co(config, bstr0(opt_name)); if (!co) { - // --watch-later-options= makes the first list item an empty string. - if (strcmp(opt_name, "") != 0) - MP_ERR(config, "Option %s not found.\n", opt_name); + MP_ERR(config, "Option %s not found.\n", opt_name); return false; } -- cgit v1.2.3