forked from cellular-infrastructure/osmo-pcu
misc: Remove if (timer_pending) stop_timer idiom from the code
osmo_timer_del is an idempotent operation. There is no requirement to check if it is running. If you don't want a timer to run, delete it. Maybe one should have called the method _unschedule, _cancel to make this more clear.
This commit is contained in:
parent
a004e6a823
commit
51c57045e5
|
@ -515,8 +515,7 @@ static int nsvc_signal_cb(unsigned int subsys, unsigned int signal,
|
|||
case S_NS_BLOCK:
|
||||
if (nsvc_unblocked) {
|
||||
nsvc_unblocked = 0;
|
||||
if (osmo_timer_pending(&bvc_timer))
|
||||
osmo_timer_del(&bvc_timer);
|
||||
osmo_timer_del(&bvc_timer);
|
||||
bvc_sig_reset = 0;
|
||||
bvc_reset = 0;
|
||||
bvc_unblocked = 0;
|
||||
|
@ -646,8 +645,7 @@ void gprs_bssgp_destroy(void)
|
|||
if (!bssgp_nsi)
|
||||
return;
|
||||
|
||||
if (osmo_timer_pending(&bvc_timer))
|
||||
osmo_timer_del(&bvc_timer);
|
||||
osmo_timer_del(&bvc_timer);
|
||||
|
||||
osmo_signal_unregister_handler(SS_L_NS, nsvc_signal_cb, NULL);
|
||||
|
||||
|
|
|
@ -304,8 +304,7 @@ void pcu_l1if_close(void)
|
|||
if (!state)
|
||||
return;
|
||||
|
||||
if (osmo_timer_pending(&state->timer))
|
||||
osmo_timer_del(&state->timer);
|
||||
osmo_timer_del(&state->timer);
|
||||
|
||||
bfd = &state->conn_bfd;
|
||||
if (bfd->fd > 0)
|
||||
|
|
Loading…
Reference in New Issue