On Sun, 28 Apr 2013 22:31:57 -0400, Derek Buitenhuis <[email protected]> wrote: > It wasn't very bvious from docs alone what exactly must go in ^^
> there. > > Signed-off-by: Derek Buitenhuis <[email protected]> > --- > libavcodec/avcodec.h | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > index d8e5d5e..124ec9a 100644 > --- a/libavcodec/avcodec.h > +++ b/libavcodec/avcodec.h > @@ -1920,8 +1920,9 @@ typedef struct AVCodecContext { > * extended_data must be allocated with av_malloc() and will be > freed in > * av_frame_unref(). > * * otherwise exended_data must point to data > - * - buf[] must contain references to the buffers that contain the frame > - * data. > + * - buf[] must contain one or more pointers to AVBufferRef strutures, > which > + * contain the frame's data references. See: av_buffer_create(), > + * av_buffer_alloc(), and av_buffer_ref(). I'm not quite convinced this new wording is very clear. E.g. what are the "frame's data references"? Maybe something like ...to AVBufferRef structures (also note the typo). Each of the frame's data / extended_data pointers must be contained in one of the AVBufferRefs. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
