<p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/17996">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/osmo-ttcn3-hacks/+/17996/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/osmo-ttcn3-hacks/+/17996/1/bts/BTS_Tests.ttcn@4908">Patch Set #1, Line 4908:</a> <code style="font-family:monospace,monospace">private function f_si3_has_gprs_indicator(Si3RestOctets ro) return boolean {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">That's really confusing! Si3RestOctets vs SI3RestOctets. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">ACK. We should either rename the old symbol(s) (to SI3RestOctetsRaw?), or completely drop them as they're basically sub-types of RestOctets with length limitations (we can apply those limitations within the related records).</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  type record SystemInformationType3 {<br>    // ...<br>    RestOctets    rest_octets<br>  } with { variant (rest_octets) "FIELDLENGTH(4)" };</pre><p style="white-space: pre-wrap; word-wrap: break-word;">What do you think?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/17996/1/bts/BTS_Tests.ttcn@4909">Patch Set #1, Line 4909:</a> <code style="font-family:monospace,monospace">      var SI3RestOctets rest := dec_SI3RestOctets(ro);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I'm wondering whether it makes sense to decode it each time inside the function instead of simply pa […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I was thinking about this, but for now I just want to get rid of manual bit manipulations and make use of the new records. Given that this function is used in several test cases and we only care about the GPRS indicator for now, I think it's ok to have this function.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/17996">change 17996</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/+/17996"/><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: I2dc970d3d34a7d70c6845558544259e387f1d6fc </div>
<div style="display:none"> Gerrit-Change-Number: 17996 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <axilirator@gmail.com> </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-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 02 May 2020 18:26:42 +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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>