On 12/14/18, Carl Eugen Hoyos <[email protected]> wrote: > 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?
OK, whatever. > > Thank you, Carl Eugen > _______________________________________________ > ffmpeg-devel mailing list > [email protected] > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
