[MPlayer-cvslog] CVS: main/libmpdemux demux_mkv.c,1.51,1.52

[Rich Felker]

On Sun, Nov 06, 2005 at 08:27:23PM -0800, Loren Merritt wrote:
> On Sun, 6 Nov 2005, Rich Felker wrote:
> >On Sat, Nov 05, 2005 at 05:56:25AM +0100, Loren Merritt CVS wrote:
> >>CVS change done by Loren Merritt CVS
> >>
> >>Update of /cvsroot/mplayer/main/libmpdemux
> >>In directory mail:/var2/tmp/cvs-serv2517
> >>
> >>Modified Files:
> >>	demux_mkv.c
> >>Log Message:
> >>sort timestamps instead of assuming conventional B-frame order. (fixes 
> >>x264 B-pyramid)
> >
> >Why? My understanding is that even though mplayer calls the variable
> >pts, it really wants dts. If this is the case, just use the standard
> >pts->dts algorithm and all cases should work automatically... Or is
> >there a reason this isn't possible?
> 
> The general pts->dts algorithm requires knowing what the delay is.
> Should demux_mkv parse the SPS header to find it?

Hmm... yes mkv is ugly because it does not include delay in the file
headers. I dunno what to do. Is it even possible to demux mkv
correctly without reading all the way to EOF to ensure that there are
no earlier frames than the next one?

Rich