<p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/13866">View Change</a></p><p>5 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13866/4/sgsn/SGSN_Tests.ttcn">File sgsn/SGSN_Tests.ttcn:</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/13866/4/sgsn/SGSN_Tests.ttcn@466">Patch Set #4, Line 466:</a> <code style="font-family:monospace,monospace">private function f_send_l3_gmm_llc(template (value) PDU_L3_MS_SGSN l3_mo, integer gb_index := 0) runs on BSSGP_ConnHdlr {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This looks unrelated?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13866/4/sgsn/SGSN_Tests.ttcn@492">Patch Set #4, Line 492:</a> <code style="font-family:monospace,monospace">         rNC_ID := 2342 /* FIXME */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Can you extend the FIXME here and a few lines above?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13866/4/sgsn/SGSN_Tests.ttcn@508">Patch Set #4, Line 508:</a> <code style="font-family:monospace,monospace">        if (gb_index >= NUM_GB) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">AFAIU, you use gb_index >= NUM_GB to know if conn is Iu or Gb? Can you add a function is_iu(gb_index) for that? so it becomes easy to understand. Or perhaps have an extra param signalling which protocol we use (simlar to what we already have for SCCPLITE vs AoIP in other tests).</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13866/4/sgsn/SGSN_Tests.ttcn@628">Patch Set #4, Line 628:</a> <code style="font-family:monospace,monospace">                                //BSSAP.receive(tr_RANAP_CommonId(imsi_hex2oct(g_pars.imsi)));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Why is this commented?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13866/4/sgsn/SGSN_Tests_Iu.ttcn">File sgsn/SGSN_Tests_Iu.ttcn:</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/13866/4/sgsn/SGSN_Tests_Iu.ttcn@534">Patch Set #4, Line 534:</a> <code style="font-family:monospace,monospace"></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This repeat seems unrelated.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/13866">change 13866</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-ttcn3-hacks/+/13866"/><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-Change-Id: I66069e31c30d33934ad57cc2b8794f56ffd5c7d6 </div>
<div style="display:none"> Gerrit-Change-Number: 13866 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 12 Aug 2019 13:38:16 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>