[FFmpeg-devel] [PATCH] mxfdec.c: prefer metadata from Footer

emcodem at ffastrans.com emcodem at ffastrans.com
Mon Jun 28 00:40:17 EEST 2021


Am 2021-06-27 20:12, schrieb Marton Balint:
> Why? I though it is enough if you store the partition number in the
> metadata set, that way you should be able to compare if the existing
> metadata set is better than the current one when adding a new metadata
> set. Or am I missing something?

OK, i just had a try on this but honestly i don't know how this could 
work without a very deep change of the whole mxfdec.c.
The problem is that i cannot just add a field to the struct 
MXFMetadataSet as this points to raw data which has been read from the 
mxf file. I could add a field but if i initialize the value, i will 
automatically destroy the original raw data which was read from the mxf 
file.

In case i did not miss anything, i guess the 1-line change i sent is 
kind of best effort: it should not destroy any existing funtionality but 
in certain cases, it delivers more accurate metadata.

Lemme know your thoughts!


More information about the ffmpeg-devel mailing list