<p>dexter has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/9961">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">MSC_ConnectionHandler: split up as_media into separate altsteps<br><br>as_media() tests both, IPACC/RSL media handling and MGCP media<br>handling. These two domains are technically quite separate, which<br>means we can split them up into two separate altsteps in order<br>to increase readability of the code.<br><br>- Split as_media() into as_Media_ipacc() and as_Media_mgw()<br><br>Change-Id: I539e8ffdb9f99d5a8e730dd918df502614b9e84d<br>Related: OS#3292<br>---<br>M bsc/MSC_ConnectionHandler.ttcn<br>1 file changed, 17 insertions(+), 7 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/61/9961/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/bsc/MSC_ConnectionHandler.ttcn b/bsc/MSC_ConnectionHandler.ttcn</span><br><span>index 1261a4d..f38a3e9 100644</span><br><span>--- a/bsc/MSC_ConnectionHandler.ttcn</span><br><span>+++ b/bsc/MSC_ConnectionHandler.ttcn</span><br><span>@@ -149,12 +149,10 @@</span><br><span>         self.stop;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-/* altstep for handling of IPA + MGCP media related commands. Can be activated by a given</span><br><span style="color: hsl(0, 100%, 40%);">- * test case if it expects to see media related handling (i.e. voice calls */</span><br><span style="color: hsl(0, 100%, 40%);">-altstep as_Media() runs on MSC_ConnHdlr {</span><br><span style="color: hsl(120, 100%, 40%);">+/* altstep for handling of IPACC media related commands. Activated by as_Media() to test</span><br><span style="color: hsl(120, 100%, 40%);">+ * RSL level media handling */</span><br><span style="color: hsl(120, 100%, 40%);">+altstep as_Media_ipacc() runs on MSC_ConnHdlr {</span><br><span>        var RSL_Message rsl;</span><br><span style="color: hsl(0, 100%, 40%);">-    var MgcpCommand mgcp_cmd;</span><br><span>    var RSL_IE_Body ie;</span><br><span>  var boolean b_unused;</span><br><span>        [not g_media.bts.ipa_crcx_seen] RSL.receive(tr_RSL_IPA_CRCX(g_chan_nr)) -> value rsl {</span><br><span>@@ -243,6 +241,14 @@</span><br><span>             repeat;</span><br><span>              }</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+/* altstep for handling of MGCP media related commands. Activated by as_Media() to test</span><br><span style="color: hsl(120, 100%, 40%);">+ * MGW level media handling */</span><br><span style="color: hsl(120, 100%, 40%);">+altstep as_Media_mgw() runs on MSC_ConnHdlr {</span><br><span style="color: hsl(120, 100%, 40%);">+  var MgcpCommand mgcp_cmd;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>  [] MGCP.receive(tr_CRCX) -> value mgcp_cmd {</span><br><span>              var SDP_Message sdp;</span><br><span>                 var integer cid := f_get_free_mgcp_conn();</span><br><span>@@ -298,8 +304,12 @@</span><br><span>    }</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(120, 100%, 40%);">+/* Altsteps for handling of media related commands. Can be activated by a given</span><br><span style="color: hsl(120, 100%, 40%);">+ * test case if it expects to see media related handling (i.e. voice calls) */</span><br><span style="color: hsl(120, 100%, 40%);">+altstep as_Media() runs on MSC_ConnHdlr {</span><br><span style="color: hsl(120, 100%, 40%);">+        [] as_Media_ipacc();</span><br><span style="color: hsl(120, 100%, 40%);">+  [] as_Media_mgw();</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span> </span><br><span> /* this component represents a single subscriber connection at the MSC.</span><br><span>  * There is a 1:1 mapping between SCCP connections and BSSAP_ConnHdlr components.</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/9961">change 9961</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/9961"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I539e8ffdb9f99d5a8e730dd918df502614b9e84d </div>
<div style="display:none"> Gerrit-Change-Number: 9961 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>