diff options
| author | Dudemanguy <random342@airmail.cc> | 2024-02-14 11:26:05 -0600 |
|---|---|---|
| committer | Dudemanguy <random342@airmail.cc> | 2024-02-17 18:56:20 +0000 |
| commit | 073858fcddddc757dcfb0e10640d19ac8d6d925d (patch) | |
| tree | 80702da260eece33f7df6660a35d82834d14849c /player/command.c | |
| parent | fb33bb7fc4aec809f05235a08ab41b918f3f160e (diff) | |
| download | mpv-073858fcddddc757dcfb0e10640d19ac8d6d925d.tar.bz2 mpv-073858fcddddc757dcfb0e10640d19ac8d6d925d.tar.xz | |
vo: remove VOCTRL_SET_EQUALIZER and simplify into UPDATE_VIDEO
Since 03cf150ff3516789d581214177f291d46310aaf4, the only purpose of this
VOCTRL was to signal a redraw to the vo. It actualy could have been
removed in 531868fe0d2a35fbbff78d9a9ff8f96df73e69fd, but this was
missed. The UPDATE_VIDEO flag is better anyway because it allows us to
handle a wide variety of options scattered around that require the VO to
update itself and redraw. We can remove both of the custom callbacks in
vo.c and only leave the VOCTRL_VO_OPTS_CHANGED one. Additionally, that
commit also introduced vo_set_want_redraw, but this is redundant and not
needed. The VOs that use VOCTRL_UPDATE_RENDER_OPTS already set
vo->want_redraw, and those are the only VOs where these options are
relevant in the first place. So we can remove this as well and just let
the big callback in player/command do everything.
Diffstat (limited to 'player/command.c')
| -rw-r--r-- | player/command.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/player/command.c b/player/command.c index 08257a0bbb..3273684227 100644 --- a/player/command.c +++ b/player/command.c @@ -7122,7 +7122,7 @@ void mp_option_change_callback(void *ctx, struct m_config_option *co, int flags, if (flags & UPDATE_VIDEO) { if (mpctx->video_out) { - vo_set_want_redraw(mpctx->video_out); + vo_control(mpctx->video_out, VOCTRL_UPDATE_RENDER_OPTS, NULL); mp_wakeup_core(mpctx); } } |