Change in osmo-ttcn3-hacks[master]: msc: cipher mode: move cipher expect to new function

neels gerrit-no-reply at
Tue Jun 29 14:21:06 UTC 2021

neels has posted comments on this change. ( )

Change subject: msc: cipher mode: move cipher expect to new function

Patch Set 2:


applied review (and added some comments) 
File library/BSSMAP_Templates.ttcn: 
PS1, Line 549: template BSSMAP_IE_KC128 tr_BSSMAP_IE_Kc128(template OCT16 kc128) := {
> could be template (present) both as return and as argument, as "omit" is not permitted in both place […]
i haven't yet understood the benefit of adding keywords like that.

(i do experience a disadvantage via tooling: universal-ctags fails to pick up templates with "(foo)" keywords.
so looking up templates with such keywords becomes annoying, my usual tag jump fails and i need to start grepping...) 
File msc/BSC_ConnectionHandler.ttcn: 
PS1, Line 568:      runs
> strange indent of 4 chars? just move to previous line?
often the 'runs on' is written without any indent, in which case the 'git diff' no longer shows the function name in the diff chunk headers -- it shows the 'runs on' line instead. So that's why I added indent.
Wanted it to be less indented than the arguments ... i guess that's how i came up with 4 spaces here.

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I1f775889fb801d441ea6c8b0f0c34718b814c09e
Gerrit-Change-Number: 24758
Gerrit-PatchSet: 2
Gerrit-Owner: neels <nhofmeyr at>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at>
Gerrit-Comment-Date: Tue, 29 Jun 2021 14:21:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge at>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list