[FFmpeg-devel] [PATCH 6/9] lavc/mpeg4videodec: do not invent a framerate from guessed numbers
Andreas Rheinhardt
andreas.rheinhardt at outlook.com
Sun Oct 1 16:14:49 EEST 2023
Anton Khirnov:
> Improves timestamps for fate-m4v*
What makes the new cfr timestamps better?
> ---
> libavcodec/mpeg4videodec.c | 3 --
> tests/ref/fate/m4v | 84 +++++++++++++++++++-------------------
> tests/ref/fate/m4v-cfr | 1 +
> 3 files changed, 43 insertions(+), 45 deletions(-)
>
> diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c
> index a8dd57bf6b..b24fe3db20 100644
> --- a/libavcodec/mpeg4videodec.c
> +++ b/libavcodec/mpeg4videodec.c
> @@ -3141,9 +3141,6 @@ static int decode_vop_header(Mpeg4DecContext *ctx, GetBitContext *gb,
>
> av_log(s->avctx, AV_LOG_WARNING,
> "time_increment_bits set to %d bits, based on bitstream analysis\n", ctx->time_increment_bits);
> - if (s->avctx->framerate.num && 4*s->avctx->framerate.num < 1<<ctx->time_increment_bits) {
> - s->avctx->framerate.num = 1<<ctx->time_increment_bits;
> - }
> }
>
> if (IS_3IV1)
> diff --git a/tests/ref/fate/m4v b/tests/ref/fate/m4v
> index cebe443adc..4eee84d01b 100644
> --- a/tests/ref/fate/m4v
> +++ b/tests/ref/fate/m4v
> @@ -1,47 +1,47 @@
> -#tb 0: 1/60
> +#tb 0: 1/5
> #media_type 0: video
> #codec_id 0: rawvideo
> #dimensions 0: 320x240
> #sar 0: 1/1
> 0, 0, 0, 1, 115200, 0x7262156b
> -0, 11, 11, 1, 115200, 0xa6aebc2f
> -0, 22, 22, 1, 115200, 0x9c7beba1
> -0, 33, 33, 1, 115200, 0x556037a2
> -0, 44, 44, 1, 115200, 0x532e3a23
> -0, 60, 60, 1, 115200, 0x110511f4
> -0, 71, 71, 1, 115200, 0xb7f929a4
> -0, 82, 82, 1, 115200, 0xdab9f3c2
> -0, 93, 93, 1, 115200, 0x441413dd
> -0, 104, 104, 1, 115200, 0x01163f33
> -0, 120, 120, 1, 115200, 0x47720c78
> -0, 131, 131, 1, 115200, 0x07c21356
> -0, 142, 142, 1, 115200, 0x085d4291
> -0, 153, 153, 1, 115200, 0xf6db074c
> -0, 164, 164, 1, 115200, 0x95093e75
> -0, 180, 180, 1, 115200, 0x5f8118ef
> -0, 191, 191, 1, 115200, 0x2b4de34a
> -0, 202, 202, 1, 115200, 0x0a731857
> -0, 213, 213, 1, 115200, 0xd75ef3be
> -0, 224, 224, 1, 115200, 0x321cf5a9
> -0, 240, 240, 1, 115200, 0x3eb222fd
> -0, 251, 251, 1, 115200, 0xc7d92f3e
> -0, 262, 262, 1, 115200, 0x6fda0366
> -0, 273, 273, 1, 115200, 0x6bb61b03
> -0, 284, 284, 1, 115200, 0x5f367ef8
> -0, 300, 300, 1, 115200, 0x237d0c77
> -0, 311, 311, 1, 115200, 0xafa813ef
> -0, 322, 322, 1, 115200, 0x39263ef4
> -0, 333, 333, 1, 115200, 0x47c70441
> -0, 344, 344, 1, 115200, 0xd43fffb0
> -0, 360, 360, 1, 115200, 0x75696afb
> -0, 371, 371, 1, 115200, 0xb0f117a3
> -0, 382, 382, 1, 115200, 0x002f42f0
> -0, 393, 393, 1, 115200, 0xa192487e
> -0, 404, 404, 1, 115200, 0x19a7072e
> -0, 420, 420, 1, 115200, 0xc500669c
> -0, 431, 431, 1, 115200, 0xe4636ba9
> -0, 442, 442, 1, 115200, 0xdf3d5d86
> -0, 453, 453, 1, 115200, 0xf30825d5
> -0, 464, 464, 1, 115200, 0xe3c944a1
> -0, 480, 480, 1, 115200, 0x8fec4420
> -0, 491, 491, 1, 115200, 0x9381fdab
> +0, 1, 1, 1, 115200, 0xa6aebc2f
> +0, 2, 2, 1, 115200, 0x9c7beba1
> +0, 3, 3, 1, 115200, 0x556037a2
> +0, 4, 4, 1, 115200, 0x532e3a23
> +0, 5, 5, 1, 115200, 0x110511f4
> +0, 6, 6, 1, 115200, 0xb7f929a4
> +0, 7, 7, 1, 115200, 0xdab9f3c2
> +0, 8, 8, 1, 115200, 0x441413dd
> +0, 9, 9, 1, 115200, 0x01163f33
> +0, 10, 10, 1, 115200, 0x47720c78
> +0, 11, 11, 1, 115200, 0x07c21356
> +0, 12, 12, 1, 115200, 0x085d4291
> +0, 13, 13, 1, 115200, 0xf6db074c
> +0, 14, 14, 1, 115200, 0x95093e75
> +0, 15, 15, 1, 115200, 0x5f8118ef
> +0, 16, 16, 1, 115200, 0x2b4de34a
> +0, 17, 17, 1, 115200, 0x0a731857
> +0, 18, 18, 1, 115200, 0xd75ef3be
> +0, 19, 19, 1, 115200, 0x321cf5a9
> +0, 20, 20, 1, 115200, 0x3eb222fd
> +0, 21, 21, 1, 115200, 0xc7d92f3e
> +0, 22, 22, 1, 115200, 0x6fda0366
> +0, 23, 23, 1, 115200, 0x6bb61b03
> +0, 24, 24, 1, 115200, 0x5f367ef8
> +0, 25, 25, 1, 115200, 0x237d0c77
> +0, 26, 26, 1, 115200, 0xafa813ef
> +0, 27, 27, 1, 115200, 0x39263ef4
> +0, 28, 28, 1, 115200, 0x47c70441
> +0, 29, 29, 1, 115200, 0xd43fffb0
> +0, 30, 30, 1, 115200, 0x75696afb
> +0, 31, 31, 1, 115200, 0xb0f117a3
> +0, 32, 32, 1, 115200, 0x002f42f0
> +0, 33, 33, 1, 115200, 0xa192487e
> +0, 34, 34, 1, 115200, 0x19a7072e
> +0, 35, 35, 1, 115200, 0xc500669c
> +0, 36, 36, 1, 115200, 0xe4636ba9
> +0, 37, 37, 1, 115200, 0xdf3d5d86
> +0, 38, 38, 1, 115200, 0xf30825d5
> +0, 39, 39, 1, 115200, 0xe3c944a1
> +0, 40, 40, 1, 115200, 0x8fec4420
> +0, 41, 41, 1, 115200, 0x9381fdab
> diff --git a/tests/ref/fate/m4v-cfr b/tests/ref/fate/m4v-cfr
> index e2d02032fe..4eee84d01b 100644
> --- a/tests/ref/fate/m4v-cfr
> +++ b/tests/ref/fate/m4v-cfr
> @@ -44,3 +44,4 @@
> 0, 38, 38, 1, 115200, 0xf30825d5
> 0, 39, 39, 1, 115200, 0xe3c944a1
> 0, 40, 40, 1, 115200, 0x8fec4420
> +0, 41, 41, 1, 115200, 0x9381fdab
More information about the ffmpeg-devel
mailing list