--- audio/headset.c | 9 +++------ plugins/service.c | 16 +++------------- src/error.c | 7 +++++++ src/error.h | 1 + 4 files changed, 14 insertions(+), 19 deletions(-) diff --git a/audio/headset.c b/audio/headset.c index fd358a0..017f02e 100644 --- a/audio/headset.c +++ b/audio/headset.c @@ -1796,8 +1796,7 @@ static DBusMessage *hs_play(DBusConnection *conn, DBusMessage *msg, if (sco_hci) { error("Refusing Headset.Play() because SCO HCI routing " "is enabled"); - return g_dbus_create_error(msg, ERROR_INTERFACE ".NotAvailable", - "Operation not Available"); + return __btd_error_not_available(msg); } switch (hs->state) { @@ -1842,8 +1841,7 @@ static DBusMessage *hs_get_speaker_gain(DBusConnection *conn, dbus_uint16_t gain; if (hs->state < HEADSET_STATE_CONNECTED) - return g_dbus_create_error(msg, ERROR_INTERFACE ".NotAvailable", - "Operation not Available"); + return __btd_error_not_available(msg); reply = dbus_message_new_method_return(msg); if (!reply) @@ -1868,8 +1866,7 @@ static DBusMessage *hs_get_mic_gain(DBusConnection *conn, dbus_uint16_t gain; if (hs->state < HEADSET_STATE_CONNECTED || slc == NULL) - return g_dbus_create_error(msg, ERROR_INTERFACE ".NotAvailable", - "Operation not Available"); + return __btd_error_not_available(msg); reply = dbus_message_new_method_return(msg); if (!reply) diff --git a/plugins/service.c b/plugins/service.c index 6f6a275..f4b71ae 100644 --- a/plugins/service.c +++ b/plugins/service.c @@ -339,12 +339,6 @@ static void exit_callback(DBusConnection *conn, void *user_data) g_free(user_record); } -static inline DBusMessage *not_available(DBusMessage *msg) -{ - return g_dbus_create_error(msg, ERROR_INTERFACE ".NotAvailable", - "Not Available"); -} - static inline DBusMessage *failed(DBusMessage *msg) { return g_dbus_create_error(msg, ERROR_INTERFACE ".Failed", "Failed"); @@ -419,9 +413,7 @@ static DBusMessage *update_record(DBusConnection *conn, DBusMessage *msg, if (remove_record_from_server(handle) < 0) { sdp_record_free(sdp_record); - return g_dbus_create_error(msg, - ERROR_INTERFACE ".NotAvailable", - "Not Available"); + return __btd_error_not_available(msg); } if (serv_adapter->adapter) @@ -465,9 +457,7 @@ static DBusMessage *update_xml_record(DBusConnection *conn, user_record = find_record(serv_adapter, handle, dbus_message_get_sender(msg)); if (!user_record) - return g_dbus_create_error(msg, - ERROR_INTERFACE ".NotAvailable", - "Not Available"); + return __btd_error_not_available(msg); sdp_record = sdp_xml_parse_record(record, len); if (!sdp_record) { @@ -552,7 +542,7 @@ static DBusMessage *remove_service_record(DBusConnection *conn, sender = dbus_message_get_sender(msg); if (remove_record(conn, sender, serv_adapter, handle) < 0) - return not_available(msg); + return __btd_error_not_available(msg); return dbus_message_new_method_return(msg); } diff --git a/src/error.c b/src/error.c index 11bc520..8b91a03 100644 --- a/src/error.c +++ b/src/error.c @@ -83,3 +83,10 @@ DBusMessage *__btd_error_in_progress(DBusMessage *msg) ".InProgress", "In Progress"); } + +DBusMessage *__btd_error_not_available(DBusMessage *msg) +{ + return g_dbus_create_error(msg, ERROR_INTERFACE + ".NotAvailable", + "Operation currently not available"); +} diff --git a/src/error.h b/src/error.h index e241b92..dfdc933 100644 --- a/src/error.h +++ b/src/error.h @@ -34,4 +34,5 @@ DBusMessage *__btd_error_invalid_args(DBusMessage *msg); DBusMessage *__btd_error_already_exists(DBusMessage *msg); DBusMessage *__btd_error_not_supported(DBusMessage *msg); DBusMessage *__btd_error_not_connected(DBusMessage *msg); +DBusMessage *__btd_error_not_available(DBusMessage *msg); DBusMessage *__btd_error_in_progress(DBusMessage *msg); -- 1.7.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html