Attention is currently required from: laforge, lynxis lazus. fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38586?usp=email )
Change subject: library/gsup: fix wrong IE order in f_gen_tr_ss_ies ...................................................................... Patch Set 3: (1 comment) File library/GSUP_Templates.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38586/comment/5a477918_0a4135c5?usp=email : PS2, Line 1412: ies[idx] := *; > well, the comment says something about a specific IE type > (OSMO_GSUP_MESSAGE_CLASS_IE) while the act […] Right. There's unfortunately no way to express that we optionally expect a specific kind of IE here, i.e. either OSMO_GSUP_MESSAGE_CLASS_IE or nothing. I tried several things: * a) `ies[idx] = tr_GSUP_IE_Message_Class(OSMO_GSUP_MESSAGE_CLASS_USSD) ifpresent` * b) `ies[idx] = (tr_GSUP_IE_Message_Class(OSMO_GSUP_MESSAGE_CLASS_USSD), omit)` * c) `ies[idx] = * length(0..1)` and none of this compiled. The a) used to work, but then was removed in TITAN [1]; the b) does not compile. The c) is permitted by the specs and would allow to at least limit number of IEs in this position (none or one), but TITAN does not implement this syntax [2]. [1] https://www.eclipse.org/forums/index.php/t/1103442/ [2] https://gitlab.eclipse.org/eclipse/titan/titan.core/-/issues/743 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38586?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I5aafbc885d3b78fd37f84feebf5dab1255d51eb1 Gerrit-Change-Number: 38586 Gerrit-PatchSet: 3 Gerrit-Owner: fixeria <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: lynxis lazus <[email protected]> Gerrit-CC: laforge <[email protected]> Gerrit-CC: pespin <[email protected]> Gerrit-Attention: laforge <[email protected]> Gerrit-Attention: lynxis lazus <[email protected]> Gerrit-Comment-Date: Wed, 06 Nov 2024 16:01:15 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: laforge <[email protected]>
