[FFmpeg-devel] [PATCH 1/6 v2] avutil/frame: add a flag to not create duplicate entries in a side data array

Anton Khirnov anton at khirnov.net
Wed Mar 27 14:25:56 EET 2024


Quoting James Almer (2024-03-27 12:49:49)
> >      * T does have a MULTI prop, then user decides whether to replace,
> >        add, or do nothing
> > 
> > I think the default behaviour for MULTI types should be adding, and the
> > other two cases should be covered by the user explicitly.
> > 
> > Then we only need a single flag, which only applies to non-MULTI types,
> > and chooses whether to replace or not.
> 
> I don't follow. How does a single flag let you choose between all these 
> scenarios?

I doesn't, my point is that for MULTI types the functions should always
add a new instance, because very few scenarios need anything else.

-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list