On Mon, 29 Apr 2013 15:27:39 -0400, Derek Buitenhuis <[email protected]> wrote: > On 2013-04-29 2:58 PM, Anton Khirnov wrote: > > 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. > > That's a bit misleading, because it makes it sound like they must have one > AVBufferRef per pointer, even though the pointers may point to different > parts of the same contiguous buffer. > > How about: > > - buf[] must contain one or more pointers to AVBufferRef structures. Each of > the frame's data and extended_data pointers must be contained in these. That > is, one pointer for each allocated chunk of memory, ^^^^^^^ I'd s/pointer/AVBufferRef/, pointer is too vague
> not one AVBufferRef per data[] entry. Well it can be one AVBufferRef per data entry, this sentence kinda implies otherwise. maybe rephrase it like "not necessarily one AVBufferRef per ..." or "..., but several data pointers may share one AVBufferRef" -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
