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 /video/out/vo_gpu.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 'video/out/vo_gpu.c')
| -rw-r--r-- | video/out/vo_gpu.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/video/out/vo_gpu.c b/video/out/vo_gpu.c index 6abe20b0d7..584a86096b 100644 --- a/video/out/vo_gpu.c +++ b/video/out/vo_gpu.c @@ -186,9 +186,6 @@ static int control(struct vo *vo, uint32_t request, void *data) case VOCTRL_SET_PANSCAN: resize(vo); return VO_TRUE; - case VOCTRL_SET_EQUALIZER: - vo->want_redraw = true; - return VO_TRUE; case VOCTRL_SCREENSHOT: { struct vo_frame *frame = vo_get_current_vo_frame(vo); if (frame) |