[FFmpeg-devel] [PATCH] ffmpeg: compute filtered frame pts after filling frame from buffer

[Michael Niedermayer]

On Mon, May 14, 2012 at 04:00:04PM +0200, Matthieu Bouron wrote:
> On Mon, May 14, 2012 at 03:32:03PM +0200, Matthieu Bouron wrote:
> > Hi,
> > 
> > This patch move the filter frame pts computation after filling frame from
> > buffer in poll_filters function.
> > This fixes some weird behaviour with overlay filter; got some constant
> > frame duplication in do_video_out since the frame pts are wrong:
> >   ffmpeg -i input -vf "movie=input[v1];[in][v2]overlay" output
> > 
> > Anyway no problem using the split filter with same input,
> >   ffmpeg -i input -vf "[in]split[v1][v2];[v1][v2]overlay" output
> > 
> 
> New patch attached, i didn't notice that a check was performed on
> filtered_frame pts before do_video_out is called.
> 
> For reference, the pts copy in avfilter_fill_frame_from_video_buffer_ref
> was introduced by a80217a5eec821fa9027eaf59ace95187843d320.
> 
> Regards,
> Matthieu

>  ffmpeg.c |    4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> f932f06b1b82ad6f9d3c3737b167a485bec2e9ad  0001-ffmpeg-affect-computed-filtered-frame-pts-after-fill.patch
> From 61b3556dbafae6df962cfec8865ae843a46b897d Mon Sep 17 00:00:00 2001
> From: Matthieu Bouron <matthieu.bouron at smartjog.com>
> Date: Mon, 14 May 2012 15:10:23 +0200
> Subject: [PATCH] ffmpeg: affect computed filtered frame pts after filling
>  frame from buffer

applied

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120514/d884f9bc/attachment.asc>