libpurple/protocols/gg/roster.c

changeset 42172
7c2d151b410d
parent 42085
a76f1668564c
child 42175
4185b4043214
--- a/libpurple/protocols/gg/roster.c	Mon Mar 20 23:06:42 2023 -0500
+++ b/libpurple/protocols/gg/roster.c	Tue Mar 21 00:39:45 2023 -0500
@@ -148,16 +148,11 @@
 {
 	if (content == NULL)
 		return;
-	if (content->xml)
-		purple_xmlnode_free(content->xml);
-	if (content->contact_nodes)
-		g_hash_table_destroy(content->contact_nodes);
-	if (content->group_nodes)
-		g_hash_table_destroy(content->group_nodes);
-	if (content->group_ids)
-		g_hash_table_destroy(content->group_ids);
-	if (content->group_names)
-		g_hash_table_destroy(content->group_names);
+	g_clear_pointer(&content->xml, purple_xmlnode_free);
+	g_clear_pointer(&content->contact_nodes, g_hash_table_destroy);
+	g_clear_pointer(&content->group_nodes, g_hash_table_destroy);
+	g_clear_pointer(&content->group_ids, g_hash_table_destroy);
+	g_clear_pointer(&content->group_names, g_hash_table_destroy);
 	g_free(content->bots_group_id);
 	g_free(content);
 }
@@ -1062,8 +1057,7 @@
 			ggp_roster_set_synchronized(gc, buddy, FALSE);
 	}
 
-	g_list_free_full(rdata->sent_updates, ggp_roster_change_free);
-	rdata->sent_updates = NULL;
+	g_clear_list(&rdata->sent_updates, ggp_roster_change_free);
 
 	/* bump roster version or update it, if needed */
 	g_return_if_fail(content != NULL);

mercurial