player: change semantics of --no-correct-pts

Before this commit, this mode estimated the frame time by subtracting
successive packet PTS values. This is complete non-sense for video
codecs which use reordering. The code compensated frame times for these
non-sense using the FPS value, but confused the rest of the player with
non-sense jumping around timestamps. So, all in all this mode is not
very useful.

Repurpose this mode for fixed frame rate playback. This gives almost the
same behavior as the old mode with forced framerate (--fps option). The
result is simpler and often more robust.

1 files changed, 0 insertions, 1 deletions

diff --git a/video/decode/dec_video.h b/video/decode/dec_video.h
index 9802e4c300..3856d12043 100644
--- a/video/decode/dec_video.h
+++ b/video/decode/dec_video.h
@@ -59,7 +59,6 @@ struct dec_video {
     float initial_decoder_aspect;
 
     // State used only by player/video.c
-    float next_frame_time;
     double last_pts;
 };