<p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19216">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/osmo-ttcn3-hacks/+/19216/1/library/GSM_RR_Types.ttcn">File library/GSM_RR_Types.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/+/19216/1/library/GSM_RR_Types.ttcn@1074">Patch Set #1, Line 1074:</a> <code style="font-family:monospace,monospace">CellSelectionIndicatorValue</code></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">looking around this file the style seems to be shortened names. so ok.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yeah, please keep them short.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">but there's also something to using the exact naming from the specs...</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Better add the spec. references in comments.</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/+/19216/1/library/GSM_RR_Types.ttcn@1087">Patch Set #1, Line 1087:</a> <code style="font-family:monospace,monospace">CellSelectionIndicatorValueEnum</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I see only this one. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Well, for sure it's not really important. But still improves readability. If it's private, I know that I can change it in any way and there are no potentially broken dependencies. This also speeds up rebuilding the test suite. Without 'private', if you change this definition, TITAN would rebuild all modules that implicitly import it, even if they don't actually use it.</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/+/19216/1/library/GSM_RR_Types.ttcn@1109">Patch Set #1, Line 1109:</a> <code style="font-family:monospace,monospace">arfcn</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">in the spec this is called "FDD-ARFCN" and "TDD-ARFCN" -- i noticed that FDD and TDD are exactly the […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Not critical, we can leave it as is. In the end, it's still ARFCN regardless of the network generation it relates to.</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/+/19216/1/library/GSM_RR_Types.ttcn@1134">Patch Set #1, Line 1134:</a> <code style="font-family:monospace,monospace">target_pcid</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">44.018 says […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Ok, sorry for confusion. Indeed it seems to be a different IE.</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/+/19216/1/library/GSM_RR_Types.ttcn@1159">Patch Set #1, Line 1159:</a> <code style="font-family:monospace,monospace">meas_bw_presence := ?,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">The problem is that this field is unconditionally set to '?', so your template parameter does not affect it. You don't really need this argument because you can explicitly pass 'meas_bw := omit' if it is expected to be absent.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19216">change 19216</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/+/19216"/><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: I882c5e1f70bcc4833fc837a95c900ce291919cc5 </div>
<div style="display:none"> Gerrit-Change-Number: 19216 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 14 Jul 2020 09:56:54 +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: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>