[MPlayer-dev-eng] [PATCH] reorder pts when needed

[Dan Oscarsson]

sön 2011-11-27 klockan 18:20 +0100 skrev Reimar Döffinger:

> The point of -nocorrect-pts mode is to be based on FPS first of all,
> the fact that single incorrect pts matter so much seems to indicate
> that for some reason it no longer properly fulfills that point.

I have additional code that aligns video frame display with vsyncs. This
allows me to play 24Hz movies on my 24HZ lcd tv without frame drops or
judder. If the pts values are not in order, there will be missed vsyncs
or fram drops to compensate giving bad viewing on tv.

I would expect decode_video to return correct pts for each frame. If the
intention of -nocorrect-pts mode is for decode_video to return wrong pts
for a given frame - then that is not what I would have expected. To
return a somewhat incorrect value - yes, but to return the pts of
another frame - no.

I assumed -nocorrect-pts mode is the old way to do it. But now when we
mix old demuxers and new decoders mplayer have to change so that works
too. I think demuxer lavf plus h264 works (gives pts in order), but
demux_ts + h264 does not as decode_video do not do the reordering it
should. I would not be surprised if there are other cases where
reordering is not done - but needed to get the rights pts values for
each frame.

  Dan