[FFmpeg-cvslog] ffmpeg: remove unused and errorneous AVFrame timestamp check

Hendrik Leppkes git at videolan.org
Mon Oct 17 06:02:56 EEST 2016


ffmpeg | branch: release/3.0 | Hendrik Leppkes <h.leppkes at gmail.com> | Sat Oct  1 16:15:45 2016 +0200| [27f1fb1251437038caebbd64e9f791c614028ae2] | committer: Michael Niedermayer

ffmpeg: remove unused and errorneous AVFrame timestamp check

Decoders have previously not used AVFrame.pts, and with the upcoming
deprecation of pkt_pts (in favor of pts), this would lead to an errorneous
interpration of timestamps.

(cherry picked from commit 04a3577263782cd6d70722d4ae18d75fee03dbc4)
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=27f1fb1251437038caebbd64e9f791c614028ae2
---

 ffmpeg.c | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/ffmpeg.c b/ffmpeg.c
index 435ad57..4d1a972 100644
--- a/ffmpeg.c
+++ b/ffmpeg.c
@@ -2014,12 +2014,7 @@ static int decode_audio(InputStream *ist, AVPacket *pkt, int *got_output)
             }
     }
 
-    /* if the decoder provides a pts, use it instead of the last packet pts.
-       the decoder could be delaying output by a packet or more. */
-    if (decoded_frame->pts != AV_NOPTS_VALUE) {
-        ist->dts = ist->next_dts = ist->pts = ist->next_pts = av_rescale_q(decoded_frame->pts, avctx->time_base, AV_TIME_BASE_Q);
-        decoded_frame_tb   = avctx->time_base;
-    } else if (decoded_frame->pkt_pts != AV_NOPTS_VALUE) {
+    if (decoded_frame->pkt_pts != AV_NOPTS_VALUE) {
         decoded_frame->pts = decoded_frame->pkt_pts;
         decoded_frame_tb   = ist->st->time_base;
     } else if (pkt->pts != AV_NOPTS_VALUE) {



More information about the ffmpeg-cvslog mailing list