[FFmpeg-devel] [PATCH 1/5] ffplay: move frame step pause from the video thread to video_refresh
Clément Bœsch
ubitux at gmail.com
Wed Dec 26 01:07:33 CET 2012
On Wed, Dec 26, 2012 at 12:46:20AM +0100, Marton Balint wrote:
> This way we pause the video right after we displayed a new frame. This should
> fix ticket #2053.
>
> Signed-off-by: Marton Balint <cus at passwd.hu>
> ---
> ffplay.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/ffplay.c b/ffplay.c
> index 38673b5..070fbd7 100644
> --- a/ffplay.c
> +++ b/ffplay.c
> @@ -1396,6 +1396,9 @@ display:
> video_display(is);
>
> pictq_next_picture(is);
> +
> + if (is->step && !is->paused)
> + stream_toggle_pause(is);
> }
> } else if (is->audio_st) {
> /* draw the next audio frame */
> @@ -1923,9 +1926,6 @@ static int video_thread(void *arg)
>
> if (ret < 0)
> goto the_end;
> -
> - if (is->step)
> - stream_toggle_pause(is);
> }
> the_end:
> avcodec_flush_buffers(is->video_st->codec);
Indeed this is better, but I still have the issue sometimes. Strangely I'm
somehow able to trigger it more often with this:
http://samples.ffmpeg.org/MPEG-VOB/interlaced/bug1966ex1.vob
I have no idea why though…
--
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121226/453d6b3b/attachment.asc>
More information about the ffmpeg-devel
mailing list