[MPlayer-dev-eng] [PATCH] mkv first frame duration

[Moritz Bunkus]

Hi,

> I don't know what was the purpose of the test that I removed, and I'm not
> sure that I didn't break anything.

As other demuxers also use this code you should test with such files as
well or create a new case just for Matroska.

It would be even better if the demux_packet_t struct would be changed to
contain a 'duration' field that would be used instead of this difference
if it is != 0. Hmm... That should be easy enough to add. I'll try that
tomorrow.

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