<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-ttcn3-hacks/+/15674">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15674/1/pcu/PCU_Tests_RAW.ttcn">File pcu/PCU_Tests_RAW.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/15674/1/pcu/PCU_Tests_RAW.ttcn@a462">Patch Set #1, Line 462:</a> <code style="font-family:monospace,monospace"></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">So this comment is removed (actually not moved), but we still using hard-coded parameters? Please keep it.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15674/1/pcu/PCU_Tests_RAW.ttcn@72">Patch Set #1, Line 72:</a> <code style="font-family:monospace,monospace">f_default_pcuif_info_ind</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">You can do this without a function, just define a global constant instead:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  const PCUIF_info_ind default_info_ind := { ... };</pre><p style="white-space: pre-wrap; word-wrap: break-word;">so then it can be tuned by copying to a variable:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  var PCUIF_info_ind info_ind := default_info_ind;<br>  info_ind.foo := BAR;</pre></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15674/1/pcu/PCU_Tests_RAW.ttcn@114">Patch Set #1, Line 114:</a> <code style="font-family:monospace,monospace">f_init_pcuif</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Not sure if this function really needs to be changed. The new test cases should be based on top of the new components, while the existing test cases do not require us to pass different INFO.ind.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15674/1/pcu/PCU_Tests_RAW.ttcn@484">Patch Set #1, Line 484:</a> <code style="font-family:monospace,monospace">template (omit) PCUIF_info_ind info_ind_tpl := omit</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">So then you will not need this ugly template-omit hack:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  ..., PCUIF_info_ind := default_info_ind);</pre></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15674">change 15674</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/+/15674"/><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: I75c746b822184fdcc966d77a7c6a0a6918c236e6 </div>
<div style="display:none"> Gerrit-Change-Number: 15674 </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: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 04 Oct 2019 18:51:12 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>