Thread (2 messages) 2 messages, 2 authors, 2012-11-21
STALE4990d

[PATCH] gdbus: Remove not needed NULL pointer checks

From: Szymon Janc <hidden>
Date: 2012-11-21 10:21:03
Subsystem: the rest · Maintainer: Linus Torvalds

g_strdup returns NULL if argument passed is NULL and there is no need
to double check that.
---
 gdbus/watch.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/gdbus/watch.c b/gdbus/watch.c
index 07feb61..1cd1211 100644
--- a/gdbus/watch.c
+++ b/gdbus/watch.c
@@ -240,8 +240,8 @@ proceed:
 	data = g_new0(struct filter_data, 1);
 
 	data->connection = dbus_connection_ref(connection);
-	data->name = name ? g_strdup(name) : NULL;
-	data->owner = owner ? g_strdup(owner) : NULL;
+	data->name = g_strdup(name);
+	data->owner = g_strdup(owner);
 	data->path = g_strdup(path);
 	data->interface = g_strdup(interface);
 	data->member = g_strdup(member);
-- 
1.7.9.5
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help