2018-12-14 15:52 GMT+01:00, Paul B Mahol <[email protected]>: > On 12/14/18, Carl Eugen Hoyos <[email protected]> wrote: >> 2018-12-12 23:44 GMT+01:00, Paul B Mahol <[email protected]>: >>> On 12/12/18, Carl Eugen Hoyos <[email protected]> wrote: >>>> 2018-12-12 23:30 GMT+01:00, Paul B Mahol <[email protected]>: >>>>> On 12/12/18, Carl Eugen Hoyos <[email protected]> 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; >> >> New patch attached. >> >> Please comment, Carl Eugen > > No need to alloc channel context, just use fixed size: [2];
I assumed nearly all G.729 streams are mono and given that this improvement was about avoiding an unneeded allocation this version would be cleaner. You don't agree? Thank you, Carl Eugen _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
