[FFmpeg-cvslog] fftools/ffmpeg: when framerate is set, prefer its inverse as output timebase

Anton Khirnov git at videolan.org
Thu Apr 8 12:01:40 EEST 2021


ffmpeg | branch: master | Anton Khirnov <anton at khirnov.net> | Mon Apr  5 10:44:36 2021 +0200| [72184cc4cba21abc42f3efff94bc0e36f8b270a9] | committer: Anton Khirnov

fftools/ffmpeg: when framerate is set, prefer its inverse as output timebase

Codec timebase is not well-defined for streamcopy, so it should only be
used as the last resort.

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

 fftools/ffmpeg.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
index 46bb014de8..8e6206647f 100644
--- a/fftools/ffmpeg.c
+++ b/fftools/ffmpeg.c
@@ -3151,8 +3151,12 @@ static int init_output_stream_streamcopy(OutputStream *ost)
         return ret;
 
     // copy timebase while removing common factors
-    if (ost->st->time_base.num <= 0 || ost->st->time_base.den <= 0)
-        ost->st->time_base = av_add_q(av_stream_get_codec_timebase(ost->st), (AVRational){0, 1});
+    if (ost->st->time_base.num <= 0 || ost->st->time_base.den <= 0) {
+        if (ost->frame_rate.num)
+            ost->st->time_base = av_inv_q(ost->frame_rate);
+        else
+            ost->st->time_base = av_add_q(av_stream_get_codec_timebase(ost->st), (AVRational){0, 1});
+    }
 
     // copy estimated duration as a hint to the muxer
     if (ost->st->duration <= 0 && ist->st->duration > 0)



More information about the ffmpeg-cvslog mailing list