pespin submitted this change.

View Change


Approvals: Jenkins Builder: Verified pespin: Looks good to me, approved
sm_ms_fsm: fix double timer_cb assignment

sm_ms_fsm.c:287:14: warning: initializer overrides prior initialization
of this subobject [-Winitializer-overrides]
.timer_cb = sm_ms_fsm_timer_cb,
^~~~~~~~~~~~~~~~~~
sm_ms_fsm.c:284:14: note: previous initialization is here
.timer_cb = sm_ms_fsm_timer_cb,
^~~~~~~~~~~~~~~~~~

Change-Id: Ic1bffe410a4ce5ddafe740185a5a894e056d7879
---
M src/sm/sm_ms_fsm.c
1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/src/sm/sm_ms_fsm.c b/src/sm/sm_ms_fsm.c
index dc063b9..8d994eb 100644
--- a/src/sm/sm_ms_fsm.c
+++ b/src/sm/sm_ms_fsm.c
@@ -281,7 +281,6 @@
.name = "SM_MS",
.states = sm_ms_fsm_states,
.num_states = ARRAY_SIZE(sm_ms_fsm_states),
- .timer_cb = sm_ms_fsm_timer_cb,
.event_names = sm_ms_fsm_event_names,
.log_subsys = DLGLOBAL, /* updated dynamically through gprs_sm_ms_fsm_set_log_cat() */
.timer_cb = sm_ms_fsm_timer_cb,

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

Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: Ic1bffe410a4ce5ddafe740185a5a894e056d7879
Gerrit-Change-Number: 34777
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged