diff options
author | Mark Reid <mindmark@gmail.com> | 2021-01-16 17:48:45 -0800 |
---|---|---|
committer | Anton Khirnov <anton@khirnov.net> | 2021-02-22 11:12:10 +0100 |
commit | a82a05276997d3fa5da5a4b7e09cd71bbe283c50 (patch) | |
tree | 08e879aaa4fae07a9345e32a2d52a29438d1b02c /libavformat/mov.c | |
parent | 1c36e7c1a524cd944679b3be8a52dde85d442ada (diff) |
avformat/mov: fix timecode with counter mode flag set
The current behaviour ends up squaring the avg_frame_rate if the conter mode flag is set.
This messes up the timecode calculation, and looks to me as a regression that
seems to have been introduced 428b4aac.
Upon further testing is seems that no special case is need for having the counter flag set.
av_timecode_init appears to handles the timecode correctly, at least in the sample files
I have.
Here is a sample mov file with the counter flag set
https://www.dropbox.com/s/5l4fucb9lhq523s/timecode_counter_mode.mov
before the patch ffmpeg will report the timecode as:
00:37:11:97 and warns that the timecode framerate is 576000000/1002001
after patch:
14:50:55:02
Signed-off-by: Anton Khirnov <anton@khirnov.net>
Diffstat (limited to 'libavformat/mov.c')
-rw-r--r-- | libavformat/mov.c | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/libavformat/mov.c b/libavformat/mov.c index 2a701e9f70..f5e0c69ef3 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -2352,17 +2352,6 @@ FF_DISABLE_DEPRECATION_WARNINGS st->codec->time_base = av_inv_q(st->avg_frame_rate); FF_ENABLE_DEPRECATION_WARNINGS #endif - /* adjust for per frame dur in counter mode */ - if (tmcd_ctx->tmcd_flags & 0x0008) { - int timescale = AV_RB32(st->codecpar->extradata + 8); - int framedur = AV_RB32(st->codecpar->extradata + 12); - st->avg_frame_rate = av_mul_q(st->avg_frame_rate, (AVRational){timescale, framedur}); -#if FF_API_LAVF_AVCTX -FF_DISABLE_DEPRECATION_WARNINGS - st->codec->time_base = av_mul_q(st->codec->time_base , (AVRational){framedur, timescale}); -FF_ENABLE_DEPRECATION_WARNINGS -#endif - } if (size > 30) { uint32_t len = AV_RB32(st->codecpar->extradata + 18); /* name atom length */ uint32_t format = AV_RB32(st->codecpar->extradata + 22); |