lynxis lazus submitted this change.

View Change

Approvals: neels: Looks good to me, approved Jenkins Builder: Verified fixeria: Looks good to me, approved pespin: Looks good to me, but someone else must approve
vlr: remove old TODO to switch to osmo_tdef_fsm_inst_state_chg

The timer now using osmo_tdef_fsm_inst_state_chg.

Change-Id: I1eecfeec494a2e129c41a6e964c56ed4430611de
---
M src/libvlr/vlr.c
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/libvlr/vlr.c b/src/libvlr/vlr.c
index e90596c..c7b7e63 100644
--- a/src/libvlr/vlr.c
+++ b/src/libvlr/vlr.c
@@ -211,8 +211,7 @@

struct osmo_tdef *vlr_tdefs;

-/* This is just a wrapper around the osmo_tdef API.
- * TODO: we should start using osmo_tdef_fsm_inst_state_chg() */
+/* This is just a wrapper around the osmo_tdef API. */
unsigned long vlr_timer_secs(struct vlr_instance *vlr, int cs_timer, int ps_timer)
{
/* NOTE: since we usually do not need more than one instance of the VLR,

To view, visit change 40422. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I1eecfeec494a2e129c41a6e964c56ed4430611de
Gerrit-Change-Number: 40422
Gerrit-PatchSet: 3
Gerrit-Owner: lynxis lazus <lynxis@fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu>
Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>