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

[Moritz Bunkus]

Heya,

> Sure, I understant well that you can't maintain the 2. But I will try
> to maintain my code as I can.

I'll do my main work with your demuxer then (like adding new stream
types and stuff).

> Could you point me out the problems you see ?

Lemme test first, please :) 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).

Mosu

-- 
If Darl McBride was in charge, he'd probably make marriage
unconstitutional too, since clearly it de-emphasizes the commercial
nature of normal human interaction, and probably is a major impediment
to the commercial growth of prostitution. - Linus Torvalds
-------------- 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/mplayer-dev-eng/attachments/20040116/153ae22e/attachment.pgp>