<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/6569">View Change</a></p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think this needs to be rethought.</p><p style="white-space: pre-wrap; word-wrap: break-word;">For an initial outgoing INVITE the offered codecs should be the intersection of the calling channel's codecs and the endpoint's configured codecs.  Probably if the resulting set is empty we should offer the endpoint's configured codecs.  I do know that if we unconditionally offered the endpoint's configured codecs we have received complaints about unnecessary codec translations.</p><p style="white-space: pre-wrap; word-wrap: break-word;">For a reINVITE, I'm not sure what we should really do.  Take the intersection of the bridged peer's codecs and the channel's currently negotiated codecs.  If it results in an empty set then either don't change the current codecs or offer the endpoint's configured codecs.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Setting to -1 so the comment is not overlooked.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/6569">change 6569</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/6569"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 15 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Iffabc373bd94cd1dc700925dcfe406e12918c696 </div>
<div style="display:none"> Gerrit-Change-Number: 6569 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 22 Sep 2017 22:50:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>