On Sun, Oct 19, 2014 at 02:34:40PM -0700, Kevin Mitchell wrote:
> Doh. Not having a good day. Use consistent metadata tags.
> 
> On Sun, Oct 19, 2014 at 1:37 PM, Kevin Mitchell <kevmi...@gmail.com> wrote:
> > Thanks for the quick merge. Unfortunately, i realized i only added the
> > metadata for single frame detection. This patch adds both single and
> > multiple frame detection.

> From 36235c82dee67b568aacee919f5b357102c0e2f6 Mon Sep 17 00:00:00 2001
> From: Kevin Mitchell <kevmi...@gmail.com>
> Date: Sun, 19 Oct 2014 13:32:56 -0700
> Subject: [PATCH] avfilter/vf_idet: add both multiple and single frame
>  detection metadata
> 
> ---
>  libavfilter/vf_idet.c | 13 +++++++++----
>  1 file changed, 9 insertions(+), 4 deletions(-)
> 
> diff --git a/libavfilter/vf_idet.c b/libavfilter/vf_idet.c
> index ebbc7d2..339f4a6 100644
> --- a/libavfilter/vf_idet.c
> +++ b/libavfilter/vf_idet.c
> @@ -178,10 +178,15 @@ static int filter_frame(AVFilterLink *link, AVFrame 
> *picref)
>  
>      filter(ctx);
>  
> -    av_dict_set_int(metadata, "lavfi.idet.tff", idet->prestat[TFF], 0);
> -    av_dict_set_int(metadata, "lavfi.idet.bff", idet->prestat[BFF], 0);
> -    av_dict_set_int(metadata, "lavfi.idet.progressive", 
> idet->prestat[PROGRESSIVE], 0);
> -    av_dict_set_int(metadata, "lavfi.idet.undetermined", 
> idet->prestat[UNDETERMINED], 0);
> +    av_dict_set_int(metadata, "lavfi.idet.single.tff", idet->prestat[TFF], 
> 0);
> +    av_dict_set_int(metadata, "lavfi.idet.single.bff", idet->prestat[BFF], 
> 0);
> +    av_dict_set_int(metadata, "lavfi.idet.single.progressive", 
> idet->prestat[PROGRESSIVE], 0);
> +    av_dict_set_int(metadata, "lavfi.idet.single.undetermined", 
> idet->prestat[UNDETERMINED], 0);
> +
> +    av_dict_set_int(metadata, "lavfi.idet.multiple.tff", 
> idet->poststat[TFF], 0);
> +    av_dict_set_int(metadata, "lavfi.idet.multiple.bff", 
> idet->poststat[BFF], 0);
> +    av_dict_set_int(metadata, "lavfi.idet.multiple.progressive", 
> idet->poststat[PROGRESSIVE], 0);
> +    av_dict_set_int(metadata, "lavfi.idet.multiple.undetermined", 
> idet->poststat[UNDETERMINED], 0);
>  

Can you bump minor avfilter version this time?

And complete documentation, please.

[...]

-- 
Clément B.

Attachment: pgpbNBFUTX_OZ.pgp
Description: PGP signature

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

Reply via email to