diff options
author | wm4 <wm4@nowhere> | 2013-09-13 18:06:08 +0200 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2013-09-13 21:32:28 +0200 |
commit | 6cec60a454889907df46bf37adddfd874884b24d (patch) | |
tree | 662b3b7c6ce6924af8161566b3c9180ba086ffa8 /mpvcore/command.c | |
parent | ddd3ade02309259cda20de14bdeef334a006ad1f (diff) | |
download | mpv-6cec60a454889907df46bf37adddfd874884b24d.tar.bz2 mpv-6cec60a454889907df46bf37adddfd874884b24d.tar.xz |
core: add --deinterlace option, restore it with resume functionality
The --deinterlace option does on playback start what the "deinterlace"
property normally does at runtime. You could do this before by using the
--vf option or by messing with the vo_vdpau default options, but this
new option is supposed to be a "foolproof" way.
The main motivation for adding this is so that the deinterlace property
can be restored when using the video resume functionality
(quit_watch_later command).
Implementation-wise, this is a bit messy. The video chain is rebuilt in
mpcodecs_reconfig_vo(), where we don't have access to MPContext, so the
usual mechanism for enabling deinterlacing can't be used. Further,
mpcodecs_reconfig_vo() is called by the video decoder, which doesn't
have access to MPContext either. Moving this call to mplayer.c isn't
currently possible either (see below). So we just do this before frames
are filtered, which potentially means setting the deinterlacing every
frame. Fortunately, setting deinterlacing is stable and idempotent, so
this is hopefully not a problem. We also add a counter that is
incremented on each reconfig to reduce the amount of additional work per
frame to nearly zero.
The reason we can't move mpcodecs_reconfig_vo() to mplayer.c is because
of hardware decoding: we need to check whether the video chain works
before we decide that we can use hardware decoding. Changing it so that
this can be decided in advance without building a filter chain sounds
like a good idea and should be done, but we aren't there yet.
Diffstat (limited to 'mpvcore/command.c')
-rw-r--r-- | mpvcore/command.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/mpvcore/command.c b/mpvcore/command.c index f5421d48fe..b3a1b7a360 100644 --- a/mpvcore/command.c +++ b/mpvcore/command.c @@ -1155,10 +1155,13 @@ static void set_deinterlacing(struct MPContext *mpctx, bool enable) if (!enable) change_video_filters(mpctx, "del", VF_DEINTERLACE); } else { - int arg = enable; - if (vf->control(vf, VFCTRL_SET_DEINTERLACE, &arg) != CONTROL_OK) - change_video_filters(mpctx, "add", VF_DEINTERLACE); + if ((get_deinterlacing(mpctx) > 0) != enable) { + int arg = enable; + if (vf->control(vf, VFCTRL_SET_DEINTERLACE, &arg) != CONTROL_OK) + change_video_filters(mpctx, "add", VF_DEINTERLACE); + } } + mpctx->opts->deinterlace = get_deinterlacing(mpctx) > 0; } static int mp_property_deinterlace(m_option_t *prop, int action, |