[Ffmpeg-cvslog] r8334 - trunk/libavformat/matroska.c

Uoti Urpala uoti.urpala
Wed Mar 14 08:47:57 CET 2007

On Wed, 2007-03-14 at 01:27 +0100, Aurelien Jacobs wrote:
> I wonder why I didn't thought about setting st->need_parsing myself...
> See quick and dirty attached patch. It disable my custom pts reordering
> code and set st->need_parsing. And this indeed gives a smooth playback
> for soraya.mkv.
> Wait... That's not so simple. With the attached patch, planet.earth.mkv
> don't play at all ! It gives only sound, no video. Probably another
> small stupid mistake, but I'm too tired right now.

st->need_parsing won't fix the problem with the dts calculation code I
mentioned earlier (and which is not a problem in the demuxer).

>  I will look at
> this tomorrow, and hopefully, we can get ride of my ugly pts reordering
> code.

The pts reordering code should be removed anyway. The non-reordered pts
values are correct and the reordered version is _wrong_ even if it
happens to make ffplay work better because of bugs elsewhere.

More information about the ffmpeg-cvslog mailing list