[FFmpeg-devel] [PATCH] lavf/mxfenc: Bump EDIT_UNITS_PER_BODY

Marton Balint cus at passwd.hu
Mon Feb 13 21:33:07 EET 2023



On Mon, 13 Feb 2023, Tomas Härdin wrote:

> Passes FATE because we don't have any tests that mux files with a
> whopping 250 frames. Tested with a jpeg2000 Tears of Steel sample.

Have you checked what BMX does? Because as far as I remember for 
RDD9 (XDCAM HD) this should not exceed 10 seconds.

I suggest adding an option for this with -1 default, and unless the user 
overrides it make it 238 in case of MPEG2 essence, and infinite (0) 
otherwise.

There is also code which allocates the index entries in 
EDIT_UNITS_PER_BODY increments, that probably should be replaced with 
av_dynarray2_add...

Regards,
Marton


More information about the ffmpeg-devel mailing list