On 15/06/15 13:13, Michael Niedermayer wrote:
> This should make no difference as the byte is ignored
>
> Found-by: tim nicholson <nicho...@yahoo.com>
> Signed-off-by: Michael Niedermayer <michae...@gmx.at>
> ---
> libavformat/mxfdec.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> index 7389555..b3c25b7 100644
> --- a/libavformat/mxfdec.c
> +++ b/libavformat/mxfdec.c
> @@ -1156,7 +1156,7 @@ static const MXFCodecUL
> mxf_data_essence_container_uls[] = {
> };
>
> static const MXFCodecUL mxf_codec_uls[] = {
> - { {
> 0x06,0x0E,0x2B,0x34,0x04,0x01,0x01,0x09,0x04,0x01,0x02,0x02,0x03,0x01,0x01,0x00
> }, 14, AV_CODEC_ID_JPEG2000 },
> + { {
> 0x06,0x0E,0x2B,0x34,0x04,0x01,0x01,0x07,0x04,0x01,0x02,0x02,0x03,0x01,0x01,0x00
> }, 14, AV_CODEC_ID_JPEG2000 },
> { {
> 0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00
> }, 0, AV_CODEC_ID_NONE },
> };
>
>
LGTM ;)
Also just noticed that we now have both "MXFCodecUL ff_mxf_codec_uls[]"
and "MXFCodecUL mxf_codec_uls[]" in the mxf codebase. similar names
similar functionality and one a subset of the other. Yet more UL
confusion in mxf land to eventually untangle.
--
Tim.
Key Fingerprint 38CF DB09 3ED0 F607 8B67 6CED 0C0B FC44 8B0B FC83
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
