[FFmpeg-devel] [PATCH 3/5] avformat/mxfdec: guess wrapping of tracks by other tracks with the same body sid

[Baptiste Coudurier]

Hi Tomas, I hope you are doing well

> On Apr 14, 2019, at 8:58 AM, Tomas Härdin <tjoppen at acc.umu.se> wrote:
> 
> […]
> 
>> @@ -2553,6 +2563,12 @@ static int mxf_parse_structural_metadata(MXFContext *mxf)
>>          }
>>      }
>> 
>> +    for (int i = 0; i < mxf->fc->nb_streams; i++) {
>> +        MXFTrack *track = mxf->fc->streams[i]->priv_data;
>> +        if (track && track->body_sid && track->wrapping == UnknownWrapped)
>> +            track->wrapping = mxf_get_wrapping_by_body_sid(mxf->fc, track->body_sid);
>> +    }
> 
> Or maybe put a warning here instead. Vendors should fix their dang MXF
> muxers..


Yeah…

Since MXF is a container exclusively used in a professional environment, I believe we could be
way more strict than other formats. I also feel we don’t have to support older files that were
often broken.

—
Baptiste


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: Message signed with OpenPGP
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190414/085de15a/attachment.sig>