diff options
author | Denis Kenzior <denkenz@gmail.com> | 2010-04-15 14:37:17 -0500 |
---|---|---|
committer | Denis Kenzior <denkenz@gmail.com> | 2010-04-15 16:49:42 -0500 |
commit | 0ac64d66e87e27014b54f21e2d633e4d368ac8cf (patch) | |
tree | 6c41c0f7e8be8687dafd02dfd72a4763ec1de1a5 /src/modem.c | |
parent | f27946b4b0d71a6c12f5341b3af026715ea42fba (diff) | |
download | ofono-0ac64d66e87e27014b54f21e2d633e4d368ac8cf.tar.bz2 |
Refactor: use a switch/case to make more readable
Diffstat (limited to 'src/modem.c')
-rw-r--r-- | src/modem.c | 25 |
1 files changed, 11 insertions, 14 deletions
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, |