On 4/7/2017 1:48 PM, Vittorio Giovara wrote: > On Fri, Apr 7, 2017 at 2:27 PM, Steve Lhomme <rob...@gmail.com> wrote: >> + * @note The struct should be allocated with >> av_mastering_display_metadata_alloc() >> + * and its size is not a part of the public ABI. >> + */ >> +typedef struct AVMasteringDisplayMetadata { >> +} AVMasteringDisplayMetadata; >> + >> +/** >> + * Allocate an AVMasteringDisplayMetadata structure and set its fields to >> + * default values. The resulting struct can be freed using av_freep(). >> + * >> + * @return An AVMasteringDisplayMetadata filled with default values or NULL >> + * on failure. >> + */ >> +AVMasteringDisplayMetadata *av_mastering_display_metadata_alloc(void); > > This signature might be problematic, it should host a size_t* which > should be filled with the size of the struct, like it's done for other > side data (except stereo3d for historical reasons).
That will mean different signature between projects. And for that matter, why were you against me trying to add a replacement alloc function with this parameter, then? _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel