<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1: Code-Review-2</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm pretty sure that the current ordering matches, because we have to remove the 'FIELDORDER("msg")'. I also had a patch changing the spare and discriminator order, and hacked a 1 into the discriminator to verify. So the way to go is to remove the "msb", not change the order.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">In BSSAP_LE_Types.ttcn we defile MSB globally, but in PDU_BSSAP_LE we use LSB order (from right to left), so it's wrong. Either we remove the global MSB attribute (I would rather explicitly change it to LSB and keep in the bottom), or follow the MSB order and place them from left to right.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I hope I didn't get something completely wrong, since you also said you successfully decoded a message with this code? That part still confuses me a little bit.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Given that that in your sample ('000E2B44010005080062F2240017002A'O) contains '00'O at that position, it does not matter which way you place those fields.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20015">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20015">change 20015</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/+/20015"/><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: Ie9bbf99df0b189846c8ea3159bfd3c9a0ee5df2a </div>
<div style="display:none"> Gerrit-Change-Number: 20015 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Assignee: neels <nhofmeyr@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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 08 Sep 2020 15:34:14 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>