[FFmpeg-devel] [PATCH] avformat/mxfdec: fix pixel format extraction for cinema j2k

[Tomas Härdin]

mån 2020-09-14 klockan 10:48 +0100 skrev Rémi Achard:
> > Why this added hunk? Appending strong refs like this is likely to have
> > all sorts of unintended effects. The same MaterialPackages will appear
> > multiple times for example.
> 
> This is my attempt to fix an issue in the fate tests.

FATE works fine for me, at least with a default configuration. Can you
provide what options you're using?

> If you look at mxf_read_generic_descriptor, there is two places where we
> call mxf_read_strong_ref_array, if this happens on the same descriptor the
> second call will override the first one. This happen in the test suite for
> an XAVC file CDCI descriptor, see the previous post I made for the breaking
> test example.

You mean there's both a subdescriptor using local tag 0x3F01 and one
using mxf_subdescriptor_array_smpte or mxf_subdescriptor_array_interop?
In the same CDCIDescriptor?

/Tomas