On date Tuesday 2014-10-21 11:08:47 +0100, Derek Buitenhuis encoded:
> On 10/21/2014 10:47 AM, Stefano Sabatini wrote:
> > - AV_OPT_TYPE_INT, { .i64 = 2 }, 2, INT_MAX, V|A|F},
> > + AV_OPT_TYPE_INT, { .i64 = 1 }, 1, INT_MAX, V|A|F},
>
> I'm not sure I follow how this 'allows' n=1? Doesn't this just
> change the default?
I noticed that after I sent the patch. Patch tested with n=1.
--
FFmpeg = Fierce Forgiving Mastering Peaceless Elitarian Geisha
>From c3396eab5ef727d2b09c14e4bc47248c12d1c343 Mon Sep 17 00:00:00 2001
From: Stefano Sabatini <stefa...@gmail.com>
Date: Tue, 21 Oct 2014 11:45:30 +0200
Subject: [PATCH] lavfi/concat: accept a single segment
Simplify scripting operations, so that n=1 is not to be considered as a
special case.
---
libavfilter/avf_concat.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavfilter/avf_concat.c b/libavfilter/avf_concat.c
index 18f373c..266bb36 100644
--- a/libavfilter/avf_concat.c
+++ b/libavfilter/avf_concat.c
@@ -59,7 +59,7 @@ typedef struct {
static const AVOption concat_options[] = {
{ "n", "specify the number of segments", OFFSET(nb_segments),
- AV_OPT_TYPE_INT, { .i64 = 2 }, 2, INT_MAX, V|A|F},
+ AV_OPT_TYPE_INT, { .i64 = 2 }, 1, INT_MAX, V|A|F},
{ "v", "specify the number of video streams",
OFFSET(nb_streams[AVMEDIA_TYPE_VIDEO]),
AV_OPT_TYPE_INT, { .i64 = 1 }, 0, INT_MAX, V|F },
--
1.8.3.2
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
