[FFmpeg-devel] [PATCH 2/7] decode: add a method for attaching lavc-internal data to frames

wm4 nfxjfg at googlemail.com
Thu Oct 5 23:56:46 EEST 2017


On Thu, 5 Oct 2017 22:01:56 +0200
Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:

> 2017-10-05 21:22 GMT+02:00 wm4 <nfxjfg at googlemail.com>:
> 
> >> And this is just one example ...  
> >
> > Your one example was invalid, next one please.  
> 
> One example is more than enough, we can all see that this
> is not clean new code.

But I showed that his example isn't actually a problem.

> What is wrong about adding a new field if you really need it?

Because AVFrame shouldn't have internal libavcodec fields?


More information about the ffmpeg-devel mailing list