[MPlayer-dev-eng] [PATCH] new C only matroska demuxer

Aurélien Jacobs aurel at gnuage.org
Fri Jan 16 22:14:41 CET 2004


On 1/16/2004, "Moritz Bunkus" <moritz at bunkus.org> wrote:

>> Could you point me out the problems you see ?
>
>Lemme test first, please :)

Sorry. Take your time ! Anyway I won't be connected this weekend.

>One thing I noticed right away, though, was
>that you don't supported linked seek heads (a seek head pointing to
>another seek head). That's trivial to fix, though.
>
>Another thing: Chapters aren't always at the start of the file, they
>might be at the end and only referenced via the seek head. That won't
>work with your current version either.
>
>In general you have to pay attention: several items are referenced in
>the seek head but might be found by 'normal' parsing as well. You'll
>have to make sure you don't parse anything more than once (chapters,
>cues, seek heads etc).

Here is a corrected version. I hope it solve all these issues.
I also followed Diego's suggestion.

Aurelien
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: new_demux_mkv_2.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040116/1d9d4829/attachment.txt>


More information about the MPlayer-dev-eng mailing list