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

[Aurelien Jacobs]

On Mon, 12 Mar 2007 15:23:13 +0200
Uoti Urpala <uoti.urpala at pp1.inet.fi> wrote:

> On Mon, 2007-03-12 at 14:01 +0100, Aurelien Jacobs wrote:
> > On Mon, 12 Mar 2007 09:56:22 +0200
> > Uoti Urpala <uoti.urpala at pp1.inet.fi> wrote:
> > > What's the reason for doing this kind of reordering?
> > 
> > Because without it, the affected videos are playing all jerky.
> 
> Playing all jerky in what player using what options?

ffplay, mplayer... no special option.

> Do you have a sample which shows this behavior?

http://samples.mplayerhq.hu/Matroska/V_MPEG_ISO/

> > > Doesn't this code
> > > necessarily produce pts values in the wrong order for some B-frame
> > > sequences such as H264 B pyramid?
> > 
> > I'm not sure about it. I've no samples handy.
> > I should try to build one...
> 
> This code will sort the pts of consecutive B frame packets in ascending
> order. There is more than one possible display order for a sequence
> (longer than 1) of B frames but the code will give the same sorted pts
> sequence for all of them, which cannot be correct for more than one of
> the cases.

Unless the matroska muxer has already reordered the frames in ascending
pts order (without reordering the pts themselves) ?

Aurel