Thread (4 messages) 4 messages, 2 authors, 2013-10-14
STALE4670d

[PATCH 2/2] gdbus: Remove not needed check for NULL DBusPendingCall

From: Szymon Janc <hidden>
Date: 2013-09-26 08:02:35
Subsystem: the rest · Maintainer: Linus Torvalds

It is now checked by g_dbus_send_message_with_reply() so there is no
need to double check that in caller.
---
 gdbus/client.c | 5 -----
 1 file changed, 5 deletions(-)
diff --git a/gdbus/client.c b/gdbus/client.c
index 7bffdad..be8cc29 100644
--- a/gdbus/client.c
+++ b/gdbus/client.c
@@ -112,11 +112,6 @@ static gboolean modify_match(DBusConnection *conn, const char *member,
 		return FALSE;
 	}
 
-	if (call == NULL) {
-		dbus_message_unref(msg);
-		return FALSE;
-	}
-
 	dbus_pending_call_set_notify(call, modify_match_reply, NULL, NULL);
 	dbus_pending_call_unref(call);
 
-- 
1.8.4
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help