Re: [FFmpeg-devel] [PATCH 6/6] ffplay: auto-insert setfield, tinterlace filters for MXF files with separate fields layout

[Marton Balint]

On Wed, 22 Jul 2015, Matthieu Bouron wrote:
From: Matthieu Bouron <matthieu.bou...@stupeflix.com>

---
ffplay.c | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)

diff --git a/ffplay.c b/ffplay.c
index 58034c6..5038553 100644
--- a/ffplay.c
+++ b/ffplay.c
@@ -31,6 +31,7 @@
#include <stdint.h>

#include "libavutil/avstring.h"
+#include "libavutil/bprint.h"
#include "libavutil/colorspace.h"
#include "libavutil/eval.h"
#include "libavutil/mathematics.h"
@@ -1961,10 +1962,13 @@ static int configure_video_filters(AVFilterGraph 
*graph, VideoState *is, const c
    char sws_flags_str[128];
    char buffersrc_args[256];
    int ret;
+    AVBPrint args;
+    AVDictionaryEntry *entry;
    AVFilterContext *filt_src = NULL, *filt_out = NULL, *last_filter = NULL;
    AVCodecContext *codec = is->video_st->codec;
    AVRational fr = av_guess_frame_rate(is->ic, is->video_st, NULL);

+    av_bprint_init(&args, 0, 1);
    av_opt_get_int(sws_opts, "sws_flags", 0, &sws_flags);
    snprintf(sws_flags_str, sizeof(sws_flags_str), "flags=%"PRId64, sws_flags);
    graph->scale_sws_opts = av_strdup(sws_flags_str);
@@ -2033,6 +2037,23 @@ static int configure_video_filters(AVFilterGraph *graph, 
VideoState *is, const c
        }
    }

+    entry = av_dict_get(is->video_st->metadata, "mxf_frame_layout", NULL, 0);
+    if (entry && entry->value &&
+        (!strcmp(entry->value, "separate_fields") ||
+         !strcmp(entry->value, "segmented_frame"))) {
+
+        entry = av_dict_get(is->video_st->metadata, "mxf_field_dominance", 
NULL, 0);
+        if (entry && entry->value) {
+
+            av_bprintf(&args, "mode=%s", entry->value);
+
+            INSERT_FILT("setfield", args.str);
+            av_bprint_finalize(&args, NULL);
+        }
+
+        INSERT_FILT("tinterlace", "mode=merge");
+    }
+

A command line option would be good for this to override the default 
behaviour. Similar to how autorotate works.

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