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

Michael Niedermayer michaelni at gmx.at
Tue Apr 22 02:24:55 CEST 2014


On Mon, Apr 21, 2014 at 12:21:14PM +0200, Nicolas George wrote:
> 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.

ok, applied

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140422/8ad78b20/attachment.asc>


More information about the ffmpeg-devel mailing list