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

Reply via email to