Attention is currently required from: fixeria.
pespin has posted comments on this change by pespin. (
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42520?usp=email )
Change subject: stp: STP_Tests_IPA: Introduce tests
TC_ipa_tcp_{cli,srv}_adm_shutdown
......................................................................
Patch Set 1:
(2 comments)
File stp/STP_Tests_IPA.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42520/comment/801b36b2_f6b478d0?usp=email
:
PS1, Line 209: //if (not g_ipa_configs[i].is_server) {
> Maybe add a comment explaining why the if is commented out?
Done
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42520/comment/f87e065c_4b96f0fe?usp=email
:
PS1, Line 230: f_listen_ipa
> I am not strictly against abusing this function, but IMO a separate function
> with a self-explaining […]
Done
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42520?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: Ie30de6caa3e2b680aff9e50898e63f615b526500
Gerrit-Change-Number: 42520
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Mon, 30 Mar 2026 08:00:26 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <[email protected]>
