Attention is currently required from: pespin, dexter.
laforge has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34075 )
Change subject: GTP_Templates: Add templates for RIM ROUTING ADDRESS
......................................................................
Patch Set 1: Code-Review+1
(1 comment)
File library/GTP_Templates.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34075/comment/a07125ff_8f45…
PS1, Line 1456: template RIM_RoutingAddress_Discriminator
tr_RIM_RoutingAddress_Discriminator(template bitstring addr_discr := ?) := {
template (present)
that is correct, though
historically we didn't always use those qualifiers particularly for receive templates,
and it seems the remainder of this file also fails to use the qualifier. So it's
"consistency with surrounding code" vs. "adopt latest best-known
practices". Ideally one would submit a patch (before or after this one) to bring all
of the templates in this module up-to-date.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34075
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: If79f94ac3b7ec9a76763141ee2d8cac50c69d60b
Gerrit-Change-Number: 34075
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Sat, 05 Aug 2023 07:49:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment