From 0ac64d66e87e27014b54f21e2d633e4d368ac8cf Mon Sep 17 00:00:00 2001 From: Denis Kenzior Date: Thu, 15 Apr 2010 14:37:17 -0500 Subject: Refactor: use a switch/case to make more readable --- src/modem.c | 25 +++++++++++-------------- 1 file changed, 11 insertions(+), 14 deletions(-) (limited to 'src') diff --git a/src/modem.c b/src/modem.c index 45c2bc47..62ecd3c8 100644 --- a/src/modem.c +++ b/src/modem.c @@ -1152,23 +1152,20 @@ static void modem_sim_ready(void *user, enum ofono_sim_state new_state) { struct ofono_modem *modem = user; - if (new_state == OFONO_SIM_STATE_NOT_PRESENT) { - if (modem->pre_sim_atoms) - remove_post_sim_atoms(modem); - - return; - } - - if (new_state != OFONO_SIM_STATE_READY) - return; - - if (modem->pre_sim_atoms == NULL) + switch (new_state) { + case OFONO_SIM_STATE_NOT_PRESENT: + remove_post_sim_atoms(modem); + break; + case OFONO_SIM_STATE_INSERTED: + break; + case OFONO_SIM_STATE_READY: modem->pre_sim_atoms = g_slist_copy(modem->atoms); - if (modem->driver->post_sim) - modem->driver->post_sim(modem); + if (modem->driver->post_sim) + modem->driver->post_sim(modem); - __ofono_history_probe_drivers(modem); + __ofono_history_probe_drivers(modem); + } } static void sim_watch(struct ofono_atom *atom, -- cgit v1.2.3