[FFmpeg-devel] [PATCH] remove MAX_STREAMS check from mms.c

Aurelien Jacobs aurel
Mon Oct 4 15:02:20 CEST 2010


On Sun, Oct 03, 2010 at 05:09:10PM -0400, Ronald S. Bultje wrote:
> Hi,
> 
> On Sun, Oct 3, 2010 at 5:01 PM, Aurelien Jacobs <aurel at gnuage.org> wrote:
> > The MAX_STREAMS check in mms.c looks useless with the new API. Attached
> > patch just drop it.
> > mms.c is the last user of MAX_STREAMS in lavf, and I would like to get
> > ride of it soon.
> 
> http://archives.free.net.ph/message/20100812.225254.fc8cfbcf.ca.html

Ooops... I thought there was already a discussion about this but I
couldn't find it anymore...
Well, OK. Fine with me. The demuxer can have its own internal safty
check. But this have to stay inside the demuxer itself, so that the
value can be tuned depending on the characteristics of the format or the
demuxer itself without impacting any other code.
Patch attached.

Aurel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mms_max_streams.diff
Type: text/x-diff
Size: 1208 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101004/3492fe3b/attachment.diff>



More information about the ffmpeg-devel mailing list