<p><a href="https://gerrit.osmocom.org/14221">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/14221/1/bts/BTS_Tests.ttcn">File bts/BTS_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/14221/1/bts/BTS_Tests.ttcn@2005">Patch Set #1, Line 2005:</a> <code style="font-family:monospace,monospace">more_ies :={valueof(t_RSL_IE(RSL_IE_MS_POWER, RSL_IE_Body:{ms_power := ms_power}))} </code></p><p style="white-space: pre-wrap; word-wrap: break-word;">this looks a bit 'clumsy' in terms of stuffing a lot in a single line without good reason.  I'd sugest to first define a derived template like</p><p style="white-space: pre-wrap; word-wrap: break-word;">template t_RSL_IE_MsPower() := t_RSL_IE(RSL_IE_MS_POWER ...</p><p style="white-space: pre-wrap; word-wrap: break-word;">and then use that, or simply split it in two lines:</p><ul><li>first assigne the valueof(...) to a local variable</li><li>then pass that variable as f_est_dchan(more_ies := {my_var});</li></ul></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/14221/1/bts/BTS_Tests.ttcn@2013">Patch Set #1, Line 2013:</a> <code style="font-family:monospace,monospace">l1_dl.payload.data_ind.payload[0]</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I'v pretty much just copied what other tests do, I guess in this particular case there is no real ne […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">agreed we can avoid going through decode + template match here.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/14221">change 14221</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/14221"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: Icecf18c77bfe8b049cc0896c4c5b441187be1c0e </div>
<div style="display:none"> Gerrit-Change-Number: 14221 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Eric Wild <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Eric Wild <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 28 May 2019 17:03:53 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>