[FFmpeg-devel] [PATCH] vf_fps: Don't flush a cached frame if it should have been dropped

[Derek Buitenhuis]

On 10/4/2016 10:21 PM, Michael Niedermayer wrote:
>> This "break" may actually be more correct output, but I am not familiar enough
>> with lavfi or vf_fps to say. Removing the last frame in case where it should have
>> been removed was the entire point of this patch.
> 
> the change to fate is wrong
> the input has a timebase of 1/3000, all timestamps are multiplies of
> 100 its thus basically 30 fps (with skiped frames) and the used
> -vf fps=30 should restore skip frames to make it cfr, loosing the last
> frame entirely is wrong in this case

I think you are correct. However, I'm not exactly sure of the proper way to fix
this. Not without more thought...

- Derek