<p style="white-space: pre-wrap; word-wrap: break-word;">It seems I am the only one who is still against this approach... The proposed encoding is kind of problematic, especially if one would need to add more fields to INFO.ind. And it's still not clear what is the plan for a situation when we would need to send additional SI messages to the PCU, e.g. SI4. Again, why not to introduce a separate PCUIF message type for SI?</p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22369">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/+/22369/4/library/PCUIF_Types.ttcn">File library/PCUIF_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/+/22369/4/library/PCUIF_Types.ttcn@224">Patch Set #4, Line 224:</a> <code style="font-family:monospace,monospace">      OCT23           si1 optional,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This encoding is ambiguous. Imagine that one of the SI kinds is not present for some reason, let's say SI3 is missing. This is possible because those are separate messages on the A-bis/RSL, and the BTS would re-send INFO.ind for each of them. How would you guess on the receive side what is what and where?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22369">change 22369</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/+/22369"/><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: I9054ab0e969c0fbfdc671c92d44cc61360959adc </div>
<div style="display:none"> Gerrit-Change-Number: 22369 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 28 Jan 2021 21:37:52 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>