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

[wm4]

On Mon, 16 Oct 2017 19:28:27 +0800
Xiaolei Yu <dreifachstein at gmail.com> wrote:

> On 10/03/2017 09:15 PM, wm4 wrote:
> > From: Anton Khirnov <anton at khirnov.net>
> > 
> > Use the AVFrame.opaque_ref field. The original user's opaque_ref is
> > wrapped in the lavc struct and then unwrapped before the frame is
> > returned to the caller.
> > 
> > This new struct will be useful in the following commits.
> >   
> 
> How about using AVFrame::buf[]?

My local crash-fix hack for videotoolbox actually uses buf[] for this
purpose, but it won't work in general, because video data might need
it. Also that would have exactly the same problems as opaque_ref.
Actually, all approaches will have exactly the same problems as
opaque_ref, but michael doesn't get it.