<p>George Joseph <strong>uploaded patch set #3</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14328">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">codec_negotiation: Implement SDP answer preferences<br><br>* Added outgoing_call_answer_pref and incoming_call_answer_pref<br>  to structures, endpoint config, and sample pjsip.conf.<br><br>* Added a new function ast_sip_session_create_joint_cap_from_stream<br>  which takes a stream from pending topology and the capabilities<br>  of the peer and creates the joint caps.<br><br>* Refactored get_codecs() in res_pjsip_sdp_rtp.c so it preserves<br>  the order or codecs specified on SDP "m" attributes.<br><br>* Updated set_caps() and set_incoming_call_offer_cap() in<br>  res_pjsip_sdp_rtp.c to use the new codec functions.<br><br>* Slightly refactored the ast_sip_call_codec_str_to_pref function<br>  in res_pjsip.<br><br>* Added a new function ast_sip_session_get_name which can be used<br>  to get a meaningful name for error/warning/debug messages.<br>  If a session->channel is not NULL, it returns the channel<br>  name.  Then it checks session->endpoint and if not NULL,<br>  returns the endpoint's sorcery id.  If all else fails, it<br>  returns "unknown".<br><br>* Added a lot of debug messages to res_pjsip_session and<br>  res_pjsip_sdp_rtp.<br><br>ASTERISK-28856<br><br>Change-Id: Iad188ae997bdcb5c28e2eb12c6bb2b732538ad45<br>---<br>M configs/samples/pjsip.conf.sample<br>M include/asterisk/res_pjsip.h<br>M include/asterisk/res_pjsip_session.h<br>M include/asterisk/res_pjsip_session_caps.h<br>M res/res_pjsip.c<br>M res/res_pjsip/pjsip_configuration.c<br>M res/res_pjsip_sdp_rtp.c<br>M res/res_pjsip_session.c<br>M res/res_pjsip_session/pjsip_session_caps.c<br>9 files changed, 380 insertions(+), 87 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/28/14328/3</pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/14328">change 14328</a>. To unsubscribe, or for help writing mail filters, 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/c/asterisk/+/14328"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iad188ae997bdcb5c28e2eb12c6bb2b732538ad45 </div>
<div style="display:none"> Gerrit-Change-Number: 14328 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>