Re: [FFmpeg-devel] [PATCH v2] libavformat/mxfdec.c: support demuxing opatom audio without index

2015-01-09 Thread Mark Reid
On Thu, Jan 8, 2015 at 12:40 PM, Clément Bœsch wrote: > On Wed, Jan 07, 2015 at 10:31:41AM -0800, Mark Reid wrote: > [...] > > +segment = av_mallocz(sizeof(MXFIndexTableSegment)); > > sizeof(*segment)? cool, didn't know that worked! > > +if (!segment) { > > > +av_log(mxf->fc,

Re: [FFmpeg-devel] [PATCH v2] libavformat/mxfdec.c: support demuxing opatom audio without index

2015-01-08 Thread Clément Bœsch
On Wed, Jan 07, 2015 at 10:31:41AM -0800, Mark Reid wrote: [...] > +segment = av_mallocz(sizeof(MXFIndexTableSegment)); sizeof(*segment)? > +if (!segment) { > +av_log(mxf->fc, AV_LOG_ERROR, "failed to allocate > IndexTableSegment"); small note: missing \n You can probably drop

[FFmpeg-devel] [PATCH v2] libavformat/mxfdec.c: support demuxing opatom audio without index

2015-01-07 Thread Mark Reid
Hi, This approach is much simpler then my previous attempt. This version creates a IndexTableSegment and everything else can continue as normal. --- libavformat/mxfdec.c | 56 1 file changed, 56 insertions(+) diff --git a/libavformat/mxfdec.