From: Inaky Perez-Gonzalez <[email protected]>

'msg' gets too confusing: is it the SMS message, a D-Bus message?
---
 src/sms.c |   25 +++++++++++++------------
 1 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/src/sms.c b/src/sms.c
index faaacbe..4ad9e61 100644
--- a/src/sms.c
+++ b/src/sms.c
@@ -134,7 +134,7 @@ struct tx_queue_entry {
        unsigned char cur_pdu;
        unsigned int msg_id;
        unsigned int retry;
-       DBusMessage *msg;
+       DBusMessage *dbus_msg;
        gboolean status_report;
        struct sms_address receiver;
        char *name;
@@ -436,10 +436,10 @@ static DBusMessage *dbus_sms_msg_cancel(
                return __ofono_error_invalid_args(msg);
        sms_msg_cancel(sms_msg,
                       do_network_cancel ? SMS_MSG_CANCEL_IN_NETWORK : 0);
-       sms_msg->msg = dbus_message_ref(msg);
+       sms_msg->dbus_msg = dbus_message_ref(msg);
        __ofono_dbus_pending_reply(
-               &sms_msg->msg,
-               dbus_message_new_method_return(sms_msg->msg));
+               &sms_msg->dbus_msg,
+               dbus_message_new_method_return(sms_msg->dbus_msg));
        return NULL;
 }
 
@@ -507,9 +507,9 @@ static void tx_queue_entry_destroy(struct tx_queue_entry 
*entry)
        g_free(entry->pdus);
        g_free(entry->name);
        entry->state = __OFONO_SMS_TX_ST_INVALID;
-       if (entry->msg) {
-               dbus_message_unref(entry->msg);
-               entry->msg = NULL;
+       if (entry->dbus_msg) {
+               dbus_message_unref(entry->dbus_msg);
+               entry->dbus_msg = NULL;
        }
 }
 
@@ -543,8 +543,8 @@ static void tx_finished(const struct ofono_error *error, 
int mr, void *data)
 
                ofono_sms_tx_state_set(entry, OFONO_SMS_TX_ST_CANCELING);
                entry = g_queue_pop_head(sms->txq);
-               __ofono_dbus_pending_reply(&entry->msg,
-                                       __ofono_error_failed(entry->msg));
+               __ofono_dbus_pending_reply(&entry->dbus_msg,
+                                       __ofono_error_failed(entry->dbus_msg));
 
                __ofono_history_sms_send_status(modem, entry->msg_id,
                                        time(NULL),
@@ -578,8 +578,9 @@ static void tx_finished(const struct ofono_error *error, 
int mr, void *data)
        }
 
        entry = g_queue_pop_head(sms->txq);
-       __ofono_dbus_pending_reply(&entry->msg,
-                               dbus_message_new_method_return(entry->msg));
+       __ofono_dbus_pending_reply(
+               &entry->dbus_msg,
+               dbus_message_new_method_return(entry->dbus_msg));
        __ofono_history_sms_send_status(modem, entry->msg_id,
                                        time(NULL),
                                        OFONO_HISTORY_SMS_STATUS_SUBMITTED);
@@ -817,7 +818,7 @@ static DBusMessage *dbus_SendMessage(DBusConnection *conn, 
DBusMessage *msg,
        if (sms_msg == NULL)
                return __ofono_error_invalid_format(msg);
        /* Unreferenced upon tx_queue_entry_destroy() */
-       sms_msg->msg = dbus_message_ref(msg);
+       sms_msg->dbus_msg = dbus_message_ref(msg);
        return NULL;
 }
 
-- 
1.6.6.1

_______________________________________________
ofono mailing list
[email protected]
http://lists.ofono.org/listinfo/ofono

Reply via email to