From 60b2e582097591547728e7c81d20460dd41c1466 Mon Sep 17 00:00:00 2001 From: Mikel Astiz Date: Tue, 12 Jun 2012 14:10:25 +0200 Subject: obexd: Remove obc_transfer_abort Refactor the code to remove function obc_transfer_abort, which is used only once and is anyway coupled to the D-Bus API. --- obexd/client/transfer.c | 26 +++++++++++--------------- 1 file changed, 11 insertions(+), 15 deletions(-) (limited to 'obexd/client/transfer.c') diff --git a/obexd/client/transfer.c b/obexd/client/transfer.c index 0686afec6..f296638c5 100644 --- a/obexd/client/transfer.c +++ b/obexd/client/transfer.c @@ -187,20 +187,6 @@ static void abort_complete(GObex *obex, GError *err, gpointer user_data) } } -static gboolean obc_transfer_abort(struct obc_transfer *transfer) -{ - if (transfer->xfer == 0) - return FALSE; - - if (transfer->progress_id != 0) { - g_source_remove(transfer->progress_id); - transfer->progress_id = 0; - } - - return g_obex_cancel_transfer(transfer->xfer, abort_complete, - transfer); -} - static DBusMessage *obc_transfer_cancel(DBusConnection *connection, DBusMessage *message, void *user_data) { @@ -218,7 +204,17 @@ static DBusMessage *obc_transfer_cancel(DBusConnection *connection, ERROR_INTERFACE ".InProgress", "Cancellation already in progress"); - if (!obc_transfer_abort(transfer)) + if (transfer->xfer == 0) + return g_dbus_create_error(message, + ERROR_INTERFACE ".Failed", + "Failed"); + + if (transfer->progress_id != 0) { + g_source_remove(transfer->progress_id); + transfer->progress_id = 0; + } + + if (!g_obex_cancel_transfer(transfer->xfer, abort_complete, transfer)) return g_dbus_create_error(message, ERROR_INTERFACE ".Failed", "Failed"); -- cgit v1.2.1