<p style="white-space: pre-wrap; word-wrap: break-word;">this is interesting, because it would finally add codec information to the MNCC socket.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Given the period of inactivity on this patch, I guess you know this already, but for completeness' sake:</p><p style="white-space: pre-wrap; word-wrap: break-word;">This is only implementing two codec types, and it is not this trivial to derive the Speech Version just from the payload type number. Doing this right would need to correlate the SDP codec names instead. (the SDP in SIP could assign any payload type numbers to any codec)</p><p style="white-space: pre-wrap; word-wrap: break-word;">Instead, we have another approach for this, i.e. adding complete SDP to the MNCC socket, <br>https://gerrit.osmocom.org/q/topic:%22codecs%22+(project:%22osmo-msc%22%20OR%20project:%22osmo-sip-connector%22)<br>-- those patches are in review and I'm actively working on getting them merged.</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-sip-connector/+/15019">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-sip-connector/+/15019/1/src/mncc.c">File src/mncc.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sip-connector/+/15019/1/src/mncc.c@824">Patch Set #1, Line 824:</a> <code style="font-family:monospace,monospace">struct gsm_mncc_bearer_cap bearer_cap</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Right, I started thinking to set a default bearer cap in a temporary variable that would apply for a […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">FYI, the syntax would have been</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  mncc.bearer_cap = (struct gsm_mncc_bearer_cap){ .transfer = ..., .mode = ... };</pre></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sip-connector/+/15019/1/src/mncc.c@829">Patch Set #1, Line 829:</a> <code style="font-family:monospace,monospace">                   .speech_ver = { GSM48_BCAP_SV_FR,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">the most impact is made by these speech_ver entries.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sip-connector/+/15019">change 15019</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.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.osmocom.org/c/osmo-sip-connector/+/15019"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-sip-connector </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I55cdc4b7c52730d7e998a1d692c7552fd07403ee </div>
<div style="display:none"> Gerrit-Change-Number: 15019 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 26 Nov 2019 03:40:50 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Comment-In-Reply-To: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Comment-In-Reply-To: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>