diff options
author | Denis Kenzior <denkenz@gmail.com> | 2010-06-21 11:24:30 -0500 |
---|---|---|
committer | Denis Kenzior <denkenz@gmail.com> | 2010-06-21 15:21:39 -0500 |
commit | 74a8ce2e8b8e48aadf6df990794a4f28561dd555 (patch) | |
tree | 73bc739556ba33b02113ad756c3e6cc72e633b48 /src/sms.c | |
parent | 84d095860fdb42a5a7a85ebe5b19649b2e9fae0e (diff) | |
download | ofono-74a8ce2e8b8e48aadf6df990794a4f28561dd555.tar.bz2 |
sms: Fixup minor style issues with status reports
Diffstat (limited to 'src/sms.c')
-rw-r--r-- | src/sms.c | 22 |
1 files changed, 7 insertions, 15 deletions
@@ -69,7 +69,6 @@ struct ofono_sms { struct ofono_atom *atom; ofono_bool_t use_delivery_reports; struct status_report_assembly *sr_assembly; - }; struct pending_pdu { @@ -735,24 +734,17 @@ static void handle_deliver(struct ofono_sms *sms, const struct sms *incoming) static void handle_sms_status_report(struct ofono_sms *sms, const struct sms *incoming) { + struct ofono_modem *modem = __ofono_atom_get_modem(sms->atom); gboolean delivered; unsigned int msg_id; - gboolean update_history; - struct ofono_modem *modem = __ofono_atom_get_modem(sms->atom); - - update_history = status_report_assembly_report(sms->sr_assembly, - incoming, &msg_id, &delivered); - if (update_history) { + if (status_report_assembly_report(sms->sr_assembly, incoming, &msg_id, + &delivered) == FALSE) + return; - if (delivered) - __ofono_history_sms_send_status(modem, msg_id, - time(NULL), OFONO_HISTORY_SMS_STATUS_DELIVERED); - else - __ofono_history_sms_send_status(modem, msg_id, - time(NULL), - OFONO_HISTORY_SMS_STATUS_DELIVER_FAILED); - } + __ofono_history_sms_send_status(modem, msg_id, time(NULL), + delivered ? OFONO_HISTORY_SMS_STATUS_DELIVERED : + OFONO_HISTORY_SMS_STATUS_DELIVER_FAILED); } |