[FFmpeg-devel] [PATCH] avutil/internal: add FF_ALLOC_ARRAY_OR_GOTO & FF_ALLOCZ_ARRAY_OR_GOTO

Nicolas George george at nsup.org
Mon Apr 21 12:21:14 CEST 2014


Le decadi 30 germinal, an CCXXII, Michael Niedermayer a écrit :
> I wanted to replace existing FF_ALLOC_OR_GOTO by this where
> appropriate, and for that it must use the same API as FF_ALLOC_OR_GOTO
> otherwise it wont mix well with unchanged surrounding FF_ALLOC_OR_GOTO
> code.

I had not realized there was already such a macro. And, I must say I am
surprised that most uses of it are both correct and natural.

(The exception is lsws/initFilter(), and it seems reasonably easy to get it
to return proper error codes.)

As I said, my comments were minor objections, and with this additional
information I really do not oppose at all.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140421/69a7b8fb/attachment.asc>


More information about the ffmpeg-devel mailing list