[gaim-migrate @ 8639]

Wed, 31 Dec 2003 07:47:01 +0000

author
Christian Hammond <chipx86@chipx86.com>
date
Wed, 31 Dec 2003 07:47:01 +0000
changeset 7962
92ef549901f6
parent 7961
3b87f00fc0c9
child 7963
f982dbac2ccf

[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
:)

src/request.c file | annotate | diff | comparison | revisions
--- 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;
 }

mercurial