[FFmpeg-devel] [PATCH 1/4] avcodec/avpacket: add av_packet_copy_side_data()

wm4 nfxjfg at googlemail.com
Mon Sep 25 19:43:50 EEST 2017


On Mon, 25 Sep 2017 10:58:31 -0300
James Almer <jamrial at gmail.com> wrote:

> > Using av_packet_copy_props() instead of introducing yet another weird
> > function would be better.  
> 
> It can't be used in some cases. Look at the last two patches in the set.
> copy_props can't be used there without some extra pointless work.

Well, you need to copy the props first, before you write any fields
that are touched by the function. But that's true for the various side
data types too. What if you want to copy a specific set of side data
values, but not others? You could add arbitrary permutations of this,
one for every use case you could think of.


More information about the ffmpeg-devel mailing list