<p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22857">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22857/1/pcu/PCU_Tests.ttcn">File pcu/PCU_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/osmo-ttcn3-hacks/+/22857/1/pcu/PCU_Tests.ttcn@4285">Patch Set #1, Line 4285:</a> <code style="font-family:monospace,monospace">    cell_chf_notif := ts_RlcMacUlCtrl_PKT_CELL_CHG_NOTIF(ms.ul_tbf.tfi, req_arfcn, req_bsic);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I know next to nothing of NACC but it seems to me like you could refactor the code above into a sepa […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I already thought about it but in general most of the stuf above is already test specific stuff like initing different parts for the test (RLCMAC, BSSGP, etc.) and hence it's not something really to be applied for this bunch of new tests but to most of the tests, so I'd leave it for a different patch.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Moreover, one finds that some variables here are later on used by the code below, so it also makes some stuff more complex because then one needs to retrieve the variable back from the new propose _nacc_init function.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also some nacc test have slight differences here since  for instance the neigh cache is initialized differently on some tests.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So I'm not sure it's a super benefit, and in any case I think it can be done at a later step for all tests in general if required.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22857">change 22857</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/+/22857"/><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: Ib83eacfab7a73a2a51ab08801ff1c00c0058057c </div>
<div style="display:none"> Gerrit-Change-Number: 22857 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 12 Feb 2021 14:40:48 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>