Attention is currently required from: lynxis lazus, pespin. neels has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-msc/+/38490?usp=email )
Change subject: vlr: add PS support ...................................................................... Patch Set 9: Code-Review+1 (2 comments) File src/libvlr/vlr_lu_fsm.c: https://gerrit.osmocom.org/c/osmo-msc/+/38490/comment/e7c31aa4_d4264ef6?usp=email : PS5, Line 771: /*! count times timer T timed out */ > I'm sure whatever else you put in there will be easier to read, that's really > a tongue twister. i will veto block this patch unless you make this: "Tally times timer T triggered timeout" https://gerrit.osmocom.org/c/osmo-msc/+/38490/comment/dd7d5d05_53c9a443?usp=email : PS5, Line 1611: fi->T > In this case, you could also pass -1 to it. […] I'm a bit confused about what i meant with my original comment. But I'm also still confused by the two arguments for CS and PS, particularly because there is no VLR that is both CS and PS at the same time, is there? I see in my original comment i tried to find out how by looking for callers... So even though i don't have a good point to make, my gut feel is that this could be less confusing? (if only a comment explaining why this is cool) About the code change you hint, I do see that vlr_timer_secs() says * TODO: we should start using osmo_tdef_fsm_inst_state_chg() */ and that sounds about right to me. -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/38490?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: Ie9ffeb140c9d354b3a0f4822e2619f623235add0 Gerrit-Change-Number: 38490 Gerrit-PatchSet: 9 Gerrit-Owner: lynxis lazus <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <[email protected]> Gerrit-Reviewer: laforge <[email protected]> Gerrit-Reviewer: neels <[email protected]> Gerrit-Reviewer: pespin <[email protected]> Gerrit-Attention: pespin <[email protected]> Gerrit-Attention: lynxis lazus <[email protected]> Gerrit-Comment-Date: Wed, 05 Mar 2025 15:59:58 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: neels <[email protected]> Comment-In-Reply-To: pespin <[email protected]> Comment-In-Reply-To: lynxis lazus <[email protected]>
