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

Paul B Mahol onemda at gmail.com
Thu Dec 13 10:57:20 EET 2018


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.


More information about the ffmpeg-devel mailing list