On 8/2/2016 12:14 PM, Nicolas George wrote: >> +AVBSFList *av_bsf_list_alloc(void); > This is personal, but for new APIs, I like "int foo_alloc(Foo **rfoo)" > better than "Foo *foo_alloc(void)": that way, the caller can forward the > error code instead of guessing it is ENOMEM.
There is no other error this could generate. It's literally an av_mallocz wrapper. Every other similar alloc() function in avcodec.h (with no extra parameters that could generate EINVAL errors and such) also have this signature, so lets keep try to keep it consistent. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
