[FFmpeg-devel] [PATCH] avformat/mov: merge stts and ctts arrays into one

Alexander Strasser eclipse7 at gmx.net
Tue Jan 7 23:50:52 EET 2025


On 2025-01-07 10:55 -0300, James Almer wrote:
> Should reduce memory usage as well as remove code duplication.
>
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  libavformat/isom.h       |  15 +-
>  libavformat/mov.c        | 579 ++++++++++++++++-----------------------
>  tests/ref/fate/quickdraw |   2 +-
>  3 files changed, 251 insertions(+), 345 deletions(-)

[...]

> diff --git a/tests/ref/fate/quickdraw b/tests/ref/fate/quickdraw
> index 44610498c8..5746929502 100644
> --- a/tests/ref/fate/quickdraw
> +++ b/tests/ref/fate/quickdraw
> @@ -9,7 +9,7 @@
>  0,          3,          3,        1,   921600, 0xc0e68764
>  0,          4,          4,        1,   921600, 0xc0e68764
>  0,          5,          5,        1,   921600, 0xc0e68764
> -0,          7,          7,        2,   921600, 0x01a16629
> +0,          7,          7,        1,   921600, 0x01a16629

Why did the duration change?
Is it intended?
If yes I think it would be better to explain it in the
commit message.

>  0,          9,          9,        1,   921600, 0x01a16629
>  0,         10,         10,        1,   921600, 0x01a16629
>  0,         11,         11,        1,   921600, 0x01a16629
> --


  Alexander


More information about the ffmpeg-devel mailing list