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

[Uoti Urpala]

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? Do you have a
sample which shows this behavior?

> > 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.