On Mon, Mar 14, 2016 at 01:44:24PM +0100, Benjamin Steffes wrote:
> Signed-off-by: Benjamin Steffes <benjaminst...@gmail.com>
> ---
>  libavfilter/vf_detelecine.c | 40 +++++++++++++++++++---------------------
>  1 file changed, 19 insertions(+), 21 deletions(-)

i think this is correct

independant of this patch, could you add a fate test for the various
cases, this seems a bit convoluted so something that has fields with
numbers displayed on them and telecined in various patterns and then
detelecined would ensure that this stays working

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to