Mercurial > pidgin

changeset 13228:dd1996c9f3b6

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
[gaim-migrate @ 15593] Cleanups by Sadrul to code like this: if (foo != NULL) g_free(foo); (From SF Patch #1429712) committer: Tailor Script <tailor@pidgin.im>
author Richard Laager <rlaager@wiktel.com>
date Sat, 11 Feb 2006 18:16:58 +0000
parents 87d2d1230ba2
children cc84c3a2fe93
files src/pounce.c
diffstat 1 files changed, 19 insertions(+), 20 deletions(-) [+]
line wrap: on
line diff
--- a/src/pounce.c	Sat Feb 11 18:16:04 2006 +0000
+++ b/src/pounce.c	Sat Feb 11 18:16:58 2006 +0000
@@ -287,14 +287,14 @@
 	if (data->buffer != NULL)
 		g_string_free(data->buffer, TRUE);
 
-	if (data->ui_name      != NULL) g_free(data->ui_name);
-	if (data->pouncee      != NULL) g_free(data->pouncee);
-	if (data->protocol_id  != NULL) g_free(data->protocol_id);
-	if (data->event_type   != NULL) g_free(data->event_type);
-	if (data->option_type  != NULL) g_free(data->option_type);
-	if (data->action_name  != NULL) g_free(data->action_name);
-	if (data->param_name   != NULL) g_free(data->param_name);
-	if (data->account_name != NULL) g_free(data->account_name);
+	g_free(data->ui_name);
+	g_free(data->pouncee);
+	g_free(data->protocol_id);
+	g_free(data->event_type);
+	g_free(data->option_type);
+	g_free(data->action_name);
+	g_free(data->param_name);
+	g_free(data->account_name);
 
 	g_free(data);
 }
@@ -498,14 +498,14 @@
 		data->events  = 0;
 		data->options = 0;
 
-		if (data->ui_name      != NULL) g_free(data->ui_name);
-		if (data->pouncee      != NULL) g_free(data->pouncee);
-		if (data->protocol_id  != NULL) g_free(data->protocol_id);
-		if (data->event_type   != NULL) g_free(data->event_type);
-		if (data->option_type  != NULL) g_free(data->option_type);
-		if (data->action_name  != NULL) g_free(data->action_name);
-		if (data->param_name   != NULL) g_free(data->param_name);
-		if (data->account_name != NULL) g_free(data->account_name);
+		g_free(data->ui_name);
+		g_free(data->pouncee);
+		g_free(data->protocol_id);
+		g_free(data->event_type);
+		g_free(data->option_type);
+		g_free(data->action_name);
+		g_free(data->param_name);
+		g_free(data->account_name);
 
 		data->ui_name      = NULL;
 		data->pounce       = NULL;
@@ -652,8 +652,8 @@
 
 	pounces = g_list_remove(pounces, pounce);
 
-	if (pounce->ui_type != NULL) g_free(pounce->ui_type);
-	if (pounce->pouncee != NULL) g_free(pounce->pouncee);
+	g_free(pounce->ui_type);
+	g_free(pounce->pouncee);
 
 	g_hash_table_destroy(pounce->actions);
 
@@ -723,8 +723,7 @@
 	g_return_if_fail(pounce  != NULL);
 	g_return_if_fail(pouncee != NULL);
 
-	if (pounce->pouncee != NULL)
-		g_free(pounce->pouncee);
+	g_free(pounce->pouncee);
 
 	pounce->pouncee = (pouncee == NULL ? NULL : g_strdup(pouncee));