summaryrefslogtreecommitdiffstats
path: root/video
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2013-10-13 23:40:23 +0200
committerwm4 <wm4@nowhere>2013-10-13 23:40:23 +0200
commit94e71b7cc0671782ff6f886425d87dd44cd695b4 (patch)
tree9a24619f3fdbea6b173aa5998bf8525af0b7433f /video
parenteeac26410f418ef45f15a7f1e9061d4eac25621b (diff)
downloadmpv-94e71b7cc0671782ff6f886425d87dd44cd695b4.tar.bz2
mpv-94e71b7cc0671782ff6f886425d87dd44cd695b4.tar.xz
mplayer, vo_image: simplify OSD redrawing, fix vo_image crash
vo_image didn't handle OSD redrawing correctly anymore after OSD redrawing behavior was changed in commit ed9295c (or maybe it has been a problem for a longer time, and only showed up now). Basically, flip_page was called unexpectedly and when no image was stored, which made it crash trying to access the image. This could happen when for example provoking OSD redrawing by pausing while using --vo=image, or by using this command line: mpv --vo=image '-vf=lavfi="select=not(mod(n\,3))"' Fix by removing the code that pretends vo_image can redraw OSD, and by removing the framestepping fallback, which could make bad things happen if the VO didn't support OSD redrawing. By now, there aren't any real VOs that can't redraw the OSD properly, so this code is not needed and just complicates things like vo_image. This change likely will also be useful for vo_lavc (encoding).
Diffstat (limited to 'video')
-rw-r--r--video/out/vo_image.c5
1 files changed, 0 insertions, 5 deletions
diff --git a/video/out/vo_image.c b/video/out/vo_image.c
index bb3264f597..411cdfd98b 100644
--- a/video/out/vo_image.c
+++ b/video/out/vo_image.c
@@ -143,11 +143,6 @@ static int preinit(struct vo *vo)
static int control(struct vo *vo, uint32_t request, void *data)
{
- switch (request) {
- // prevent random frame stepping by frontend
- case VOCTRL_REDRAW_FRAME:
- return true;
- }
return VO_NOTIMPL;
}