From e40fbfbf35ca4d150a61c62901e529940b33690a Mon Sep 17 00:00:00 2001 From: Pekka Pessi Date: Fri, 10 Sep 2010 15:11:34 +0300 Subject: [PATCH] element: remove unnecessary checks Do not check for null pointers after accessing data. --- src/element.c | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/src/element.c b/src/element.c index 3309ae8..264b5a7 100644 --- a/src/element.c +++ b/src/element.c @@ -1138,8 +1138,7 @@ static gboolean remove_element(GNode *node, gpointer user_data) if (element == root) return FALSE; - if (node != NULL) - g_node_unlink(node); + g_node_unlink(node); if (element->driver) { if (element->driver->remove) @@ -1148,8 +1147,7 @@ static gboolean remove_element(GNode *node, gpointer user_data) element->driver = NULL; } - if (node != NULL) - g_node_destroy(node); + g_node_destroy(node); connman_element_unref(element); @@ -1176,8 +1174,7 @@ static gboolean remove_element_type(GNode *node, gpointer user_data) if(element->type != type) return FALSE; - if (node != NULL) - g_node_unlink(node); + g_node_unlink(node); if (element->driver) { if (element->driver->remove) @@ -1186,8 +1183,7 @@ static gboolean remove_element_type(GNode *node, gpointer user_data) element->driver = NULL; } - if (node != NULL) - g_node_destroy(node); + g_node_destroy(node); connman_element_unref(element); -- 2.7.4