[FFmpeg-devel] [PATCH] matroska: Add incremental parsing of clusters.

[Michael Niedermayer]

On Mon, Apr 23, 2012 at 04:47:14PM -0700, Dale Curtis wrote:
> On Wed, Apr 18, 2012 at 3:21 PM, Dale Curtis <dalecurtis at chromium.org>wrote:
> 
> > On Wed, Apr 18, 2012 at 3:06 PM, Reimar Döffinger <
> > Reimar.Doeffinger at gmx.de> wrote:
> >
> >> On Tue, Apr 17, 2012 at 05:20:00PM -0700, dalecurtis at chromium.org wrote:
> >> > Currently incremental parsing is not supported for files with
> >> > SSA tracks since they require merging packets between clusters.
> >> > In this case the code falls back to non-incremental parsing.
> >>
> >> Without reading the code: is it absolutely 100% certain that only
> >> SSA needs that? Not e.g. also other subtitle formats or such?
> >>
> >> matroska_merge_packets is only called for CODEC_ID_SSA and it's the only
> > call site in the code base. So yes, as far as I can tell that's the only
> > situation which doesn't work with incremental parsing.
> >
> > - dale
> >
> 
> This has landed in libav. I haven't gotten any feedback on this list, so
> hopefully it'll show up in the next merge. Thanks!

Ive fixed the code and merged it.
Also to speed the process up in the future, please make sure that
patches are tested against latest ffmpeg.

Thanks, and sorry for the delay.

Also thanks to the libav developers for applying the patch

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- 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/20120424/4e1212da/attachment.asc>