[Mplayer-cvslog] CVS: main/libmpdemux demux_mkv_old.cpp, 1.54, 1.55

[Moritz Bunkus]

Hi,

> If I am not wrong this file is used by external matroska.

'extern' vs 'internal' is just a matter of how Matroska files are read
(low level file reading). 'external' uses libebml/libmatroska while
'internal' uses the stuff in libmpdemux/ebml.c.

> It would be good to have it, at least for regresion test.

demux_mkv.c has been better than demux_mkv_old.cpp for some time now
with more features etc. Anyway, who does regression tests?

> Or maybe in case matroska developers deside to extend something?

If we (yes, I am a Matroska dev) change something then I update
demux_mkv.c. And to be honest, I don't want to update two demuxers.

I'll remote it soon.

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-cvslog/attachments/20040715/636c481f/attachment.pgp>