Attention is currently required from: neels, pespin. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36456?usp=email )
Change subject: msc: rework f_expect_paging(): handle mismatch and timeout ...................................................................... Patch Set 2: (1 comment) File msc/BSC_ConnectionHandler.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36456/comment/751bf5b1_4cd7e76c PS1, Line 1366: if (g_pars.ran_is_geran) { > Instead of going into this code again i'd rather drop this mild convenience > patch that only helps wh […] We had a similar discussion in ttcn3-pcu-test, where we had lots of `f_expect_foo()` and `f_expect_bar()` functions. At some point we ended up with quite a complex hierarchy of one function calling several others. Each function would have its own timers and catch-over-all statements. This become a nightmare when we started hitting sporadic failures due to receiving unexpected messages. So we decided to use altsteps, which you can stack up into an alt-statement and equip with whatever else logic you need in a specific situation. -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36456?usp=email 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: I30273e3882e348a2ded88b7b96a5ec1473a56913 Gerrit-Change-Number: 36456 Gerrit-PatchSet: 2 Gerrit-Owner: neels <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <[email protected]> Gerrit-CC: pespin <[email protected]> Gerrit-Attention: neels <[email protected]> Gerrit-Attention: pespin <[email protected]> Gerrit-Comment-Date: Fri, 12 Apr 2024 09:10:14 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: neels <[email protected]> Comment-In-Reply-To: pespin <[email protected]> Comment-In-Reply-To: fixeria <[email protected]> Gerrit-MessageType: comment
