On 12/12/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; > > 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. _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
