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

[Marton Balint]

On Sun, 27 Jun 2021, emcodem at ffastrans.com wrote:

> 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.

See the attached patch, that is what I had in mind. Or is it overkill? Can 
you test it with your dataset and see if it makes any difference?

Thanks,
Marton
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-avformat-mxfdec-prefer-footer-and-complete-partition.patch
Type: text/x-patch
Size: 6404 bytes
Desc: 
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20210628/53e8bbd7/attachment.bin>