[MPlayer-dev-eng] [PATCH] further dvr-ms playback improvements

Uoti Urpala uoti.urpala at pp1.inet.fi
Sat Aug 26 00:41:40 CEST 2006


On Sat, 2006-08-26 at 00:19 +0200, Nico Sabbi wrote:
> in my last patch I removed the ASF case in the switch() in video.c
> in order to make mplayer use the timestamp deltas.

Yes you removed the ASF case, and since that was the only place using
the timestamps (without -correct-pts) now they won't be used even if
fps==1000...

And if you do something sane instead so that the timestamps are used,
then what is the point of adding code to set fps in the decoders? The
fps value will matter little if the timestamps are what determines
playback timing.

I'm not sure what you're trying to do with the video.c changes really.
The current patch seems like total nonsense (just removing the timestamp
handling as mentioned above). There already is the -correct-pts mode. If
you're not trying to create another implementation of that then what?
How the video.c code handles asf when the demuxer sets fps=1000 doesn't
seem to be any more broken than the other VFR cases.




More information about the MPlayer-dev-eng mailing list