diff options
author | Vivek Dasmohapatra <vivek@collabora.co.uk> | 2011-04-21 12:43:26 +0100 |
---|---|---|
committer | Vivek Dasmohapatra <vivek@collabora.co.uk> | 2011-05-04 15:28:33 +0100 |
commit | d830cae86f8979bad772e8217b247a3a25c0dfbd (patch) | |
tree | b4f8af6005f2b27a13f56151212cf2fdaf16a8ca | |
parent | 9aca7421037b878ce572e4ddc8f2f6200fb463df (diff) | |
download | telepathy-mission-control-d830cae86f8979bad772e8217b247a3a25c0dfbd.tar.gz |
Use utility function to check if it's the right signal
-rw-r--r-- | src/client-registry.c | 15 |
1 files changed, 1 insertions, 14 deletions
diff --git a/src/client-registry.c b/src/client-registry.c index dbc7e15d..f0947089 100644 --- a/src/client-registry.c +++ b/src/client-registry.c @@ -314,21 +314,8 @@ mcd_client_registry_name_owner_filter (DBusConnection *conn, const gchar *interface_name = NULL; const gchar *member_name = NULL; - if (dbus_message_get_type (msg) != DBUS_MESSAGE_TYPE_SIGNAL) - return DBUS_HANDLER_RESULT_NOT_YET_HANDLED; - - /* these two look weird but can happen during shutdown, I think */ - interface_name = dbus_message_get_interface (msg); - if (interface_name == NULL) - return DBUS_HANDLER_RESULT_NOT_YET_HANDLED; - - member_name = dbus_message_get_member (msg); - if (member_name == NULL) - return DBUS_HANDLER_RESULT_NOT_YET_HANDLED; - /* make sure this is the right kind of signal: */ - if (!tp_strdiff (interface_name, DBUS_INTERFACE_DBUS) && - !tp_strdiff (member_name, "NameOwnerChanged")) + if (dbus_message_is_signal (msg, DBUS_INTERFACE_DBUS, "NameOwnerChanged")) { const gchar *dbus_name = NULL; const gchar *old_owner = NULL; |