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

[John Donaghy]

> The patch way the patch works now seems mostly OK, though I did not read
> it carefully. However you should generally use doubles, not floats, for
> pts values.

This version uses a double for the pts

Re my previous email..."However there is an issue with the (only) HDTV
sample I have in that the average frametime recorded in the file seems
to be wrong"

I assuming now that it isnt possible to reliably get the average frame
time directly from the container because the value isnt always right
for HD content. However it is always possible to calculate it from the
pts values in the video stream. While these pts values arent accurate
for displaying an individual frame they are always pretty close and by
the time you've read a couple of seconds of video you can use them to
accurately determine the average frame time pretty accurately.

So in the attached version of the patch I keep recalculating the
average frame time until the end of playback unless the user performs
a seek.

I could possibly stop recalculating the average frame time after a few
seconds and it wouldnt make much difference to the adjusted pts
values. (I'm also assuming that the frame count wont overflow here.)

I'm not sure it is possible to recalculate the average frame time
after a seek because I dont know a way to get the frame number at that
point.

So, in this version there shouldn't be a problem as long as the user
doesnt attempt a seek in the first second or so of playback.

John
-------------- next part --------------
A non-text attachment was scrubbed...
Name: asfpts2.patch
Type: application/octet-stream
Size: 2386 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20060919/f0931693/attachment.obj>