Hi!
Attached patch fixes decoding the sample from ticket #4820 for me.
Please comment, Carl Eugen
diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
index 5734976..82b5ef3 100644
--- a/libavformat/mxfdec.c
+++ b/libavformat/mxfdec.c
@@ -1156,11 +1156,6 @@ static const MXFCodecUL mxf_data_es
On Tue, Sep 1, 2015 at 2:53 PM, Carl Eugen Hoyos wrote:
> Hi!
>
> Attached patch fixes decoding the sample from ticket #4820 for me.
>
> Please comment, Carl Eugen
Probably ok; seems like in d0061e77 the existence of ff_mxf_codec_uls
was missed,
and the introduced mxf_codec_uls was actually redun
On Tue, 2015-09-01 at 23:53 +0200, Carl Eugen Hoyos wrote:
> Hi!
>
> Attached patch fixes decoding the sample from ticket #4820 for me.
>
> Please comment, Carl Eugen
> -static const MXFCodecUL mxf_codec_uls[] = {
> -{ {
> 0x06,0x0E,0x2B,0x34,0x04,0x01,0x01,0x07,0x04,0x01,0x02,0x02,0x03,0x0
Tomas Härdin codemill.se> writes:
> > Attached patch fixes decoding the sample from
> ticket #4820 for me.
> The JPEG2000 tag is in ff_mxf_codec_uls[] anyway, so
> mxf_codec_uls[] should be safe to nuke. I presume you
> rate make fate on this?
I did not (specifically) ran make fate for this c
Tomas Härdin codemill.se> writes:
[...]
Please also look at the patch attached in ticket #4820,
I will add the additional uls if you don't object.
Carl Eugen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffm
Carl Eugen Hoyos ag.or.at> writes:
> Please also look at the patch attached in ticket #4820,
> I will add the additional uls if you don't object.
I pushed these additional codec_uls since they affect
decoding only together with my patch posted here.
Hope that's ok, Carl Eugen
___
