[FFmpeg-devel] [PATCH]lavc/g729dec: Support stereo streams

[Paul B Mahol]

Hi,

On 12/13/18, Paul B Mahol <onemda at gmail.com> wrote:
> On 12/12/18, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>> 2018-12-12 23:44 GMT+01:00, Paul B Mahol <onemda at gmail.com>:
>>> On 12/12/18, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>>>> 2018-12-12 23:30 GMT+01:00, Paul B Mahol <onemda at gmail.com>:
>>>>> On 12/12/18, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>>>>>> Hi!
>>>>>>
>>>>>> Attached patch fixes ticket #4553 for me.
>>>>>>
>>>>>> Please comment, Carl Eugen
>>>>>
>>>>> I do not like what you did to G729Context.
>>>>
>>>>> Please rewrite patch.
>>>>> Move relevant parts into G729ChannelContext chan[2];
>>>>
>>>> Which part of the context does not need to be duplicated?
>>>
>>> AudioDSPContext adsp;
>>
>> Looks like 24 bytes. (Or twelve?)
>> I would really prefer not to make this patch more
>> complicated to save 24 bytes on the heap that may get lost
>> on the additional allocation.
>
> Then I will do it, if you can not.
>

Feel free to apply patch as is, I will then rewrite code.