[FFmpeg-devel] [PATCH] add (e)ac3float decoders

Justin Ruggles justin.ruggles
Thu Mar 10 16:09:51 CET 2011


On 03/10/2011 09:53 AM, Kostya wrote:

> On Thu, Mar 10, 2011 at 03:43:47PM +0100, Michael Niedermayer wrote:
>> On Thu, Mar 10, 2011 at 08:29:14PM +0800, FB2000 wrote:
>>> On Tue, Mar 8, 2011 at 5:27 AM, Michael Niedermayer <michaelni at gmx.at>
>>> wrote:
>>>
>>>> On Mon, Mar 07, 2011 at 03:01:46PM -0500, Justin Ruggles wrote:
>>>>>
>>>>> 1) There is no need for separate int16 and float decoders. The AC3
>>>>> decoder is a floating-point decoder. It just happens to output int16 at
>>>>> the end.
>>>>
>>>> can you send a patch that does this?
>>>>
>>>>
>>> Well, if these are what wanted. See attached patches for AC3 and DCA.
>>
>> Kostya, ben, justin you are maintainers of that code
>> ok if i apply this or do you want to wait until we support planar audio?
> 
> I'd wait for that, I suspect it would be large performance drop on some
> platforms like ARM.

I prefer to either wait or to make it optional for some decoders until
we have planar audio support.

-Justin



More information about the ffmpeg-devel mailing list