[FFmpeg-devel] [PATCH 2/2] image2: Add "start_number" private option to the demuxer

[Michael Niedermayer]

On Sat, Jun 23, 2012 at 10:11:50AM +0600, Mashiat Sarker Shakkhar wrote:
> Currently if a pattern is given we search for up to the fifth file name in
> that sequence. This option sets that limit to an arbitrary number.
[...]
> @@ -375,6 +376,7 @@ static const AVOption options[] = {
>      { "video_size",   "", OFFSET(video_size),   AV_OPT_TYPE_STRING, {.str = NULL}, 0, 0, DEC },
>      { "framerate",    "", OFFSET(framerate),    AV_OPT_TYPE_STRING, {.str = "25"}, 0, 0, DEC },
>      { "loop",         "", OFFSET(loop),         AV_OPT_TYPE_INT,    {.dbl = 0},    0, 1, DEC },
> +    { "start_number", "first number in the sequence", OFFSET(start_number), AV_OPT_TYPE_INT, {.dbl = 1}, 1, INT_MAX, DEC },
>      { NULL },

I think this should be called max_start_number

otherwise the patch LGTM

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120623/582de4d6/attachment.asc>