To post an update, Kevin's got an open code review covering this functionality. Alexander, would you (and others interested) mind having a look at it on gerrit? Especially since you have worked on some related code.
https://gerrit.asterisk.org/#/c/5260/ Thanks, Matthew Fredrickson On Mon, Mar 20, 2017 at 11:33 AM, Kevin Harwell <kharw...@digium.com> wrote: > On Thu, Mar 16, 2017 at 4:25 PM, James Cloos <cl...@jhcloos.com> wrote: >> >> Given Alexander's note, clearly something along those lines must be >> done. >> >> But instead of hardcoding the current mapping in the dynamic option's c >> code, I'd put it in a config file. >> >> Ie, the config file would specify static mappings for otherwise dynamic >> codecs. It should provide commented out examples which match the static >> numbers used by pre-dynamic versions of asterisk to show how to set >> them. >> >> This would allow one to only force static mappings when one has a peer >> which requires them. >> > > Hi James, > > This idea was discussed a bit in #asterisk-dev the other day too. I think > adding this as a configurable option would be beneficial for the reason you > laid out. However, due to the nature of the changes required to make that > work we're going to move forward with just using the current payload values > assigned by Asterisk as defaults. This should cover the majority of user > cases with less complexity. > > -- > > Kevin Harwell > Digium, Inc. | Software Developer > 445 Jan Davis Drive NW - Huntsville, AL 35806 - USA > Check us out at: http://digium.com & http://asterisk.org > > > -- > _____________________________________________________________________ > -- Bandwidth and Colocation Provided by http://www.api-digital.com -- > > asterisk-dev mailing list > To UNSUBSCRIBE or update options visit: > http://lists.digium.com/mailman/listinfo/asterisk-dev -- Matthew Fredrickson Digium, Inc. | Engineering Manager 445 Jan Davis Drive NW - Huntsville, AL 35806 - USA -- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
