Second revision of the patch set, incorporating feedback from Nicolas George.
This also fixes the start_time option, which didn't work at all in the v1 patch set. I've added some tests based on the behaviour of the old version of the filter, and adapted the new version to pass them. The third patch just has some formatting fixes to make the config_props function easier to read. Calvin Walton (3): libavfilter/vf_fps: Add tests for start_time option libavfilter/vf_fps: Rewrite using activate callback libavfilter/vf_fps: Minor cleanups libavfilter/vf_fps.c | 391 ++++++++++++++++++----------------- tests/fate/filter-video.mak | 4 +- tests/ref/fate/filter-fps-start-drop | 11 + tests/ref/fate/filter-fps-start-fill | 11 + 4 files changed, 227 insertions(+), 190 deletions(-) create mode 100644 tests/ref/fate/filter-fps-start-drop create mode 100644 tests/ref/fate/filter-fps-start-fill -- 2.16.1 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel