diff mbox series

[FFmpeg-devel] Fix wrong time_base bug

Message ID 20210315120609.469718-1-donoughliu@gmail.com
State New
Headers show
Series [FFmpeg-devel] Fix wrong time_base bug | expand

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Donough Liu March 15, 2021, 12:06 p.m. UTC
Set time_base to inv of frame rate is only valid when input video have fixed frame rate.

Signed-off-by: Donough Liu <donoughliu@gmail.com>
---
 doc/examples/transcoding.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/doc/examples/transcoding.c b/doc/examples/transcoding.c
index 5aff08c135..49ae32c719 100644
--- a/doc/examples/transcoding.c
+++ b/doc/examples/transcoding.c
@@ -170,7 +170,7 @@  static int open_output_file(const char *filename)
                 else
                     enc_ctx->pix_fmt = dec_ctx->pix_fmt;
                 /* video time_base can be set to whatever is handy and supported by encoder */
-                enc_ctx->time_base = av_inv_q(dec_ctx->framerate);
+                enc_ctx->time_base = av_inv_q(av_mul_q(dec_ctx->framerate, (AVRational){dec_ctx->ticks_per_frame, 1}));
             } else {
                 enc_ctx->sample_rate = dec_ctx->sample_rate;
                 enc_ctx->channel_layout = dec_ctx->channel_layout;