----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3800/#review12668 -----------------------------------------------------------
/team/group/media_formats-reviewed-trunk/channels/chan_iax2.c <https://reviewboard.asterisk.org/r/3800/#comment22929> Ah, I should probably explain why find_best was dropped from this function. Basically, it didn't have any use in the current implementation and when I looked what it did in 12 and prior, it was relying on a fairly arbitrary list for choosing the best codec based on audio quality which I don't believe we have anymore, so I dropped the option. If we need to reimplement this for some reason, I can look into that. - Jonathan Rose On July 15, 2014, 3:58 p.m., Jonathan Rose wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3800/ > ----------------------------------------------------------- > > (Updated July 15, 2014, 3:58 p.m.) > > > Review request for Asterisk Developers, Joshua Colp, Matt Jordan, and > opticron. > > > Bugs: ASTERISK-23958 > https://issues.asterisk.org/jira/browse/ASTERISK-23958 > > > Repository: Asterisk > > > Description > ------- > > At some point in the media_formats work, the function edited in this patch > was modified in a way that left it abandoning the preference order of the > codecs and just choosing from the first value in the codecs bitfield. This > patch changes that function to evaluate using the preferred codecs order. > > > Diffs > ----- > > /team/group/media_formats-reviewed-trunk/channels/chan_iax2.c 418711 > > Diff: https://reviewboard.asterisk.org/r/3800/diff/ > > > Testing > ------- > > [lappy] > type=friend > ... > disallow = all > allow = ulaw > allow = gsm > allow = alaw > > context = default > > [deskbox] > type=friend > ... > disallow=all > allow=gsm > allow=alaw > > context = default > > > Using these two peers, I had each call the other. > > lappy -> deskbox > Asterisk 11: Codec chosen for lappy was ulaw. Codec chosen for deskbox was > alaw, presumably because of favorable transcoding costs. > without patch: Codec chosen for both was gsm in spite of preferred order for > lappy favoring ulaw > with patch: Codec chosen for lappy was ulaw. Codec chosen for deskbox was > alaw, again presumably because of favorable transcoding costs. > > deskbox -> lappy > It was GSM for both endpoints in all cases because deskbox favors gsm and > lappy supports it. > > > Thanks, > > Jonathan Rose > >
-- _____________________________________________________________________ -- 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
