[FFmpeg-devel] [PATCH 3/5] avcodec/bsf: Add list BSF API

Jan Sebechlebsky sebechlebskyjan at gmail.com
Fri Aug 5 14:41:44 EEST 2016



On 08/02/2016 05:26 PM, James Almer wrote:
> 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.
> _______________________________________________
I'll keep it this way then :)

Regards,
Jan



More information about the ffmpeg-devel mailing list