changeset 7962:cc77bd88cd72

[gaim-migrate @ 8639] Honestly, I don't know *what* I was thinking. We no longer delete all items in a multi-select list when trying to clear the selection. Sorry, faceprint :) committer: Tailor Script <tailor@pidgin.im>
author Christian Hammond <chipx86@chipx86.com>
date Wed, 31 Dec 2003 07:47:01 +0000
parents aca39f227ea0
children 96b71ea64386
files src/request.c
diffstat 1 files changed, 5 insertions(+), 15 deletions(-) [+]
line wrap: on
line diff
--- a/src/request.c	Wed Dec 31 07:03:18 2003 +0000
+++ b/src/request.c	Wed Dec 31 07:47:01 2003 +0000
@@ -727,17 +727,12 @@
 	g_return_if_fail(field != NULL);
 	g_return_if_fail(field->type == GAIM_REQUEST_FIELD_LIST);
 
-	if (field->u.list.items != NULL)
+	if (field->u.list.selected != NULL)
 	{
-		g_list_foreach(field->u.list.items, (GFunc)g_free, NULL);
-		g_list_free(field->u.list.items);
-		field->u.list.items = NULL;
+		g_list_foreach(field->u.list.selected, (GFunc)g_free, NULL);
+		g_list_free(field->u.list.selected);
+		field->u.list.selected = NULL;
 	}
-
-	g_hash_table_destroy(field->u.list.item_data);
-
-	field->u.list.item_data = g_hash_table_new_full(g_str_hash, g_str_equal,
-													g_free, NULL);
 }
 
 void
@@ -747,12 +742,7 @@
 	g_return_if_fail(items != NULL);
 	g_return_if_fail(field->type == GAIM_REQUEST_FIELD_LIST);
 
-	if (field->u.list.selected != NULL)
-	{
-		g_list_foreach(field->u.list.selected, (GFunc)g_free, NULL);
-		g_list_free(field->u.list.selected);
-		field->u.list.selected = NULL;
-	}
+	gaim_request_field_list_clear_selected(field);
 
 	field->u.list.selected = items;
 }