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

[emcodem at ffastrans.com]

Am 2021-07-04 20:28, schrieb emcodem at ffastrans.com:
> Am 2021-07-04 17:28, schrieb Marton Balint:
>> On Sat, 3 Jul 2021, Tomas Härdin wrote:
>> 
>>> lör 2021-07-03 klockan 15:13 +0200 skrev emcodem at ffastrans.com:
>>>> 
>>>> Unfortunately the wetransfer link for the fate samples expired, so i 
>>>> thought it might be a good idea to resend it as link to gdrive:
>>>> https://drive.google.com/file/d/1yXTdS9RfOsduzg49vBLEshdmIzdaVQfd/view?usp=sharing
>>>> 
>>>> Also attached the 2 patches: 1 from cus for mxfdec.c and one from 
>>>> myself for the corresponding fate samples.
>>>> After applying the mxfdec.c patch, fate will pass with the currently 
>>>> existing tests but the files in the zip must be uploaded to the fate 
>>>> suite before applying my corresponding patch of course (otherwise 
>>>> the files don't exist).
>>>> 
>>>> It would be cool if someone found the time and wants to apply this.
>>> 
>>> The patch works (except for the samples not being in FATE)
>> 
>> Actually I found one file where the packetization behaviour changes,
>> because after the patch a fake index is generated based of the now
>> recognized duration:
>> 
>> samples/MXF/freemxf/freeMXF-mxf-dv-1.mxf
>> 
>> but I guess the file is wrong because clip wrapped UL is used when the
>> file seems frame wrapped instead?
> 
> Nice finding! I can confirm this, it is actually clip wrapped looking
> at the mxf dump from bmx. These samples are pretty outdated anyway :D
> Unfortunately i don't have enough insight on the internals yet on the
> topic about adding/freeing/deleting metadata sets :-(

FRAME Wrapped it is, not clip wrapped :rolleyes: