On Thu, Jan 8, 2015 at 12:40 PM, Clément Bœsch <u...@pkh.me> 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, AV_LOG_ERROR, "failed to allocate > IndexTableSegment"); > > small note: missing \n > > You can probably drop the log altogether thought, this is unlikely to > happen anyway. > > yeah I'll just drop the log line, and send a new patch. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel