Juan De León (12019-08-23):
> I changed it to an inline function, returns SIZE_MAX if it fails to make
> av_malloc() fail and return NULL.

You neglected to check for SIZE_MAX afterwards. I suspect there are
architectures where such a malloc could succeed.

If you insist on a function rather than a macro, I personally consider
bad style to rely on a magic value for error: return a boolean for
success and pass the size through a pointer.

But in this case, a macro would probably be the simplest.

Regards,

-- 
  Nicolas George

Attachment: signature.asc
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to