[FFmpeg-devel] [PATCH 4/4] lavd/oss: implement surround playback.

Paul B Mahol onemda at gmail.com
Mon Feb 4 12:47:51 CET 2013


On 2/3/13, Nicolas George <nicolas.george at normalesup.org> wrote:
> Le quintidi 15 pluviose, an CCXXI, Paul B Mahol a ecrit :
>> I see. +2 if this is documented so better solution, if ever is found,
>> can be fixed.
>
> I can add the following comment:
>
> /*
>  * libswresample can do channels reordering, but does not currently offer
>  * guarantees that the samples are handled as opaque binary data, which is
>  * necessary to handle non-native binary representations supported by the
>  * device.
>  */
>
>> This should be fine without alsa in name of file that does reordering.
>> (Isn't it generic after all ...)
>
> I agree on the principle, but as far as I know, ALSA is actually the origin
> of this particular channel order. The channel order is documented,
> indirectly, in various places of ALSA's headers and source code.

I already wrote incomplete code for OSS that does use swr.
You ignored my work completely.
I kindly ask you to not touch OSS code with alsa header because after
carefully thinking all night I see no reason for blocking swr into OSS
code.


More information about the ffmpeg-devel mailing list