[FFmpeg-devel] [PATCH] avformat: Add max_streams option

Michael Niedermayer michael at niedermayer.cc
Sat Nov 19 02:36:08 EET 2016


On Fri, Nov 18, 2016 at 10:51:33PM +0100, Andreas Cadhalpun wrote:
> On 18.11.2016 17:03, Michael Niedermayer wrote:
> > --- a/libavformat/options_table.h
> > +++ b/libavformat/options_table.h
> > @@ -105,6 +105,7 @@ static const AVOption avformat_options[] = {
> >  {"format_whitelist", "List of demuxers that are allowed to be used", OFFSET(format_whitelist), AV_OPT_TYPE_STRING, { .str = NULL },  CHAR_MIN, CHAR_MAX, D },
> >  {"protocol_whitelist", "List of protocols that are allowed to be used", OFFSET(protocol_whitelist), AV_OPT_TYPE_STRING, { .str = NULL },  CHAR_MIN, CHAR_MAX, D },
> >  {"protocol_blacklist", "List of protocols that are not allowed to be used", OFFSET(protocol_blacklist), AV_OPT_TYPE_STRING, { .str = NULL },  CHAR_MIN, CHAR_MAX, D },
> > +{"max_streams", "maximum number of streams", OFFSET(max_streams), AV_OPT_TYPE_INT, { .i64 = INT_MAX }, 0, INT_MAX, D },
> >  {NULL},
> >  };
> 
> Wouldn't it make more sense to default to a saner value like 100 or 1000 or so?

was thinking that too but that belongs in a seperate patch


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

When the tyrant has disposed of foreign enemies by conquest or treaty, and
there is nothing more to fear from them, then he is always stirring up
some war or other, in order that the people may require a leader. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161119/440dfd88/attachment.sig>


More information about the ffmpeg-devel mailing list