[Asterisk-code-review] translate.c: Prefer better codecs upon translate ties. (asterisk[master])

N A asteriskteam at digium.com
Wed Nov 2 07:20:26 CDT 2022


Attention is currently required from: Joshua Colp, Greg Brown, N A, Kevin Harwell.

Hello Joshua Colp, George Joseph, Kevin Harwell, Friendly Automation, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/c/asterisk/+/15953

to look at the new patch set (#7).

Change subject: translate.c: Prefer better codecs upon translate ties.
......................................................................

translate.c: Prefer better codecs upon translate ties.

If multiple codecs are available for the same
resource and the translation costs between
multiple codecs are the same, ties are
currently broken arbitrarily, which means a
lower quality codec would be used. This forces
Asterisk to explicitly use the higher quality
codec, ceteris paribus.

ASTERISK-29455

Change-Id: I4b7297e1baca7aac14fe4a3c7538e18e2dbe9fd6
---
A doc/UPGRADE-staging/translate.txt
M include/asterisk/codec.h
M main/codec.c
M main/codec_builtin.c
M main/translate.c
5 files changed, 85 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/53/15953/7
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/15953
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I4b7297e1baca7aac14fe4a3c7538e18e2dbe9fd6
Gerrit-Change-Number: 15953
Gerrit-PatchSet: 7
Gerrit-Owner: N A <asterisk at phreaknet.org>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Kevin Harwell <default.enum at gmail.com>
Gerrit-CC: Greg Brown <gbrown1002000 at yahoo.com>
Gerrit-Attention: Joshua Colp <jcolp at sangoma.com>
Gerrit-Attention: Greg Brown <gbrown1002000 at yahoo.com>
Gerrit-Attention: N A <asterisk at phreaknet.org>
Gerrit-Attention: Kevin Harwell <default.enum at gmail.com>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20221102/85836765/attachment.html>


More information about the asterisk-code-review mailing list