[FFmpeg-devel] [PATCH 1/6] avutil/mastering_display_metadata: add av_mastering_display_metadata_alloc2()

Michael Niedermayer michael at niedermayer.cc
Sun Dec 11 23:20:39 EET 2016

On Sun, Dec 11, 2016 at 02:00:04PM +0100, wm4 wrote:
> On Sun, 11 Dec 2016 00:33:03 -0300
> Here are some brainstormed alternative ideas to adding those ...2()
> functions:
> - add functions to add side data by type to AVFrames etc.
> - provide a generic function that allocates side data by passing the
>   side data ID to it (would need separate ones for packet and stream
>   side data)

> - unify the side data enums into one (i.e. merge packet and frame side
>   data),

I was wondering previously already why there are multiple enums
I think a single enum list would be simpler independant of other

>   and provide a central function to allocate or retrieve size by
>   side data ID
> - provide functions that return the struct size, and let the user
>   use av_mallocz() (instead of both allocating and returning the size)
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161211/ae3cebb4/attachment.sig>

More information about the ffmpeg-devel mailing list