[FFmpeg-devel] [PATCH] Don't fail on Matroska with too many streams

Michael Niedermayer michaelni
Wed Jan 27 18:57:19 CET 2010


On Wed, Jan 27, 2010 at 11:00:01PM +0530, Jai Menon wrote:
> On Wed, Jan 27, 2010 at 10:53 PM, Joakim Plate <elupus at ecce.se> wrote:
> >> >
> >> > iam not matroska maintainer but i take the liberty to reject this
> >> > please work on dynamic allocation of things so that the MAX_STREAMS
> >> > limit disappears or advocate bumping the value up with the next
> >> > major ver bump.
> >>
> >> just curious : should the removal of MAX_FILES also be considered? i'm
> >> not suggesting i have a use case for this though, just asking...
> >>
> >
> > If so, please bump to atleast 41, since that is the maximum allowed for dvds
> > and what we have it patched to in our code.
> 
> well, i can submit a patch increasing the current limits right now
> (with the required #ifs) but the long term plan is to dynamically
> allocate per-stream/file contexts as needed. is just increasing both
> these values okay for now, michael?

yes, wit the needed ifdefs for MAX_STREAMS
MAX_FILES is just in ffmpeg.c and shouldnt need ifdefs

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100127/3445763f/attachment.pgp>



More information about the ffmpeg-devel mailing list