[FFmpeg-devel] [PATCH 01/10] lavfi: add avfilter_get_audio_buffer_ref_from_arrays_channels.

[Michael Niedermayer]

On Mon, Dec 31, 2012 at 08:03:16PM -0500, Derek Buitenhuis wrote:
> On 31/12/2012 7:47 PM, Nicolas George wrote:
> > Sorry, I thought that part was rhetoric:
> > 
> >      * For planar audio, each channel has a separate data pointer, and
> >      * linesize[0] contains the size of each channel buffer.
> >      * For packed audio, there is just one data pointer, and linesize[0]
> >      * contains the total size of the buffer for all channels.
> > 
> > I am about to push the patch with Stefano's comments.
> 
> So what it really is, is the size of one audio channel's buffer


> ("planes", which
> is also a hideous abuse of the word.).

there are packed and planar formats, for video and audio.
        Packed      Planar
Audio   LRLRLR...   LLLL... RRRR...
Video   YUVYUV...   YYYY... UUUU... VVVV...

And yep, thats hideously consistent, planar formats have planes not
just for video but also for audio


> 
> FWIW I think audio "planes" and "linesize" are abominations which have sprung
> up from reusing AVFrame for audio, using video-specific fields. I have no idea
> how anyone outside of FFmpeg/Libav could understand this...

A patch that uses a differently named field than linesize for audio is
welcome


> 
> Is the above at least documented somewhere public?

Ive seen doxy about it so it should be, dunno which file though


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Avoid a single point of failure, be that a person or equipment.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130101/ca0cadcf/attachment.asc>