[FFmpeg-devel] [PATCH 1/6] avutil/mastering_display_metadata: add av_mastering_display_metadata_alloc2()
michael at niedermayer.cc
Mon Dec 12 18:31:51 EET 2016
On Mon, Dec 12, 2016 at 10:06:31AM +0100, wm4 wrote:
> On Sun, 11 Dec 2016 22:20:39 +0100
> Michael Niedermayer <michael at niedermayer.cc> wrote:
> > 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
> > things
> Unfortunately Libav doesn't seem to have much interest, since it would
> break the API (or at least ABI). Also they don't have many side data
> types that require allocators.
While it makes sense for us to listen and consider what Libav does,
and we may very well choose to do the same. We could choose to
do something different, if people prefer that ...
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Avoid a single point of failure, be that a person or equipment.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: Digital signature
More information about the ffmpeg-devel