mån 2021-01-04 klockan 01:28 +0100 skrev Andreas Rheinhardt: > Signed-off-by: Andreas Rheinhardt <[email protected]> > --- > There is also another key that is duplicated: header_open_partition_key > and mxf_jp2k_rsiz. But they seem to mean something different which makes > me wonder if one of these keys is incorrect; I have therefore refrained > from deduplicating them.
Looks like I didn't notice this when I reviewed the patch. This indicates that the MXF muxer that wrote the file that 2af260e3 adds support for is broken. S422m-2006 says that the UL designator for Rsiz should be 04.01.06.03.01.00.00.00. S400m-2004 defines the prefix to this designator, which is the usual 06.0E.2B.34.04.01.01.VV where VV is the registry version. Did we have a sample for this? If you decide to deduplicate rsiz then you should have kind of comment to this effect, and/or a link to this thread and http://ffmpeg.org/pipermail/ffmpeg-devel/2015-March/169591.html The patch itself looks fine. Might as well deduplicate all keys. There's little sense in having separate sets of keys defined for the muxer and demuxer. /Tomas _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
