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

[Dale Curtis]

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!

- dale