Re: [FFmpeg-devel] [PATCH]lavf/mxfdec: Support canopus codecs

2016-03-10 Thread Carl Eugen Hoyos
Tomas Härdin codemill.se> writes: > > Attached patch fixes ticket #5316 here. > > I unfortunately have no idea what the hunk in mxf_read_header()  > > does, decoding works fine without it. > > Strange. Why was mxf_canopus_essence_element_key added then? > Pushing without it is fine if it fixes

Re: [FFmpeg-devel] [PATCH]lavf/mxfdec: Support canopus codecs

2016-03-09 Thread Tomas Härdin
On Wed, 2016-03-09 at 15:21 +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #5316 here. > I unfortunately have no idea what the hunk in mxf_read_header()  > does, decoding works fine without it. Strange. Why was mxf_canopus_essence_element_key added then? Pushing without it i

[FFmpeg-devel] [PATCH]lavf/mxfdec: Support canopus codecs

2016-03-09 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #5316 here. I unfortunately have no idea what the hunk in mxf_read_header() does, decoding works fine without it. Please comment, Carl Eugen diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index f4222fc..3c43968 100644 --- a/libavformat/mxfdec.c +++ b/lib