diff options
author | Matthias Clasen <mclasen@redhat.com> | 2023-05-17 19:13:12 -0400 |
---|---|---|
committer | Matthias Clasen <mclasen@redhat.com> | 2023-05-17 20:12:37 -0400 |
commit | c817d3c8a0081135cdaf18795440548f1755a98f (patch) | |
tree | bbd836148c0f1f0da6f6a3348bf73d666feb6dfc | |
parent | 27c824edb527934ecb93f699204d3da8c037badb (diff) | |
download | gtk+-c817d3c8a0081135cdaf18795440548f1755a98f.tar.gz |
Drop the FILLER tile type
It is not used anymore.
-rw-r--r-- | gtk/gtklistitemmanager.c | 51 | ||||
-rw-r--r-- | gtk/gtklistitemmanagerprivate.h | 3 | ||||
-rw-r--r-- | gtk/gtklistview.c | 1 | ||||
-rw-r--r-- | testsuite/gtk/listitemmanager.c | 9 |
4 files changed, 4 insertions, 60 deletions
diff --git a/gtk/gtklistitemmanager.c b/gtk/gtklistitemmanager.c index 8fc0476370..a8b12cef7f 100644 --- a/gtk/gtklistitemmanager.c +++ b/gtk/gtklistitemmanager.c @@ -198,7 +198,6 @@ gtk_list_item_manager_augment_node (GtkRbTree *tree, aug->has_footer = TRUE; break; case GTK_LIST_TILE_ITEM: - case GTK_LIST_TILE_FILLER: case GTK_LIST_TILE_REMOVED: aug->has_header = FALSE; aug->has_footer = FALSE; @@ -633,7 +632,7 @@ static GtkListTile * gtk_list_tile_get_next_skip (GtkListTile *tile) { for (tile = gtk_rb_tree_node_get_next (tile); - tile && (tile->type == GTK_LIST_TILE_FILLER || tile->type == GTK_LIST_TILE_REMOVED); + tile && tile->type == GTK_LIST_TILE_REMOVED; tile = gtk_rb_tree_node_get_next (tile)) { } @@ -644,7 +643,7 @@ static GtkListTile * gtk_list_tile_get_previous_skip (GtkListTile *tile) { for (tile = gtk_rb_tree_node_get_previous (tile); - tile && (tile->type == GTK_LIST_TILE_FILLER || tile->type == GTK_LIST_TILE_REMOVED); + tile && tile->type == GTK_LIST_TILE_REMOVED; tile = gtk_rb_tree_node_get_previous (tile)) { } @@ -885,7 +884,6 @@ gtk_list_item_manager_remove_items (GtkListItemManager *self, gtk_list_tile_set_type (tile, GTK_LIST_TILE_REMOVED); break; - case GTK_LIST_TILE_FILLER: case GTK_LIST_TILE_REMOVED: default: g_assert_not_reached (); @@ -928,7 +926,7 @@ gtk_list_item_manager_add_items (GtkListItemManager *self, { /* at end of list, pick the footer */ for (tile = gtk_rb_tree_get_last (self->items); - tile && (tile->type == GTK_LIST_TILE_REMOVED || tile->type == GTK_LIST_TILE_FILLER); + tile && tile->type == GTK_LIST_TILE_REMOVED; tile = gtk_rb_tree_node_get_previous (tile)) { } @@ -1030,34 +1028,6 @@ gtk_list_tile_split (GtkListItemManager *self, } /* - * gtk_list_tile_append_filler: - * @self: the listitemmanager - * @previous: tile to append to - * - * Appends a filler tile. - * - * Filler tiles don't refer to any items or header and exist - * just to take up space, so that finding items by position gets - * easier. - * - * They ave a special garbage-collection behavior, see - * gtk_list_tile_gc(). - * - * Returns: The new filler tile - **/ -GtkListTile * -gtk_list_tile_append_filler (GtkListItemManager *self, - GtkListTile *previous) -{ - GtkListTile *result; - - result = gtk_rb_tree_insert_after (self->items, previous); - result->type = GTK_LIST_TILE_FILLER; - - return result; -} - -/* * gtk_list_tile_gc: * @self: the listitemmanager * @tile: a tile or NULL @@ -1083,13 +1053,6 @@ gtk_list_tile_gc (GtkListItemManager *self, if (tile == NULL) return NULL; - if (tile->type == GTK_LIST_TILE_FILLER) - { - next = gtk_rb_tree_node_get_next (tile); - gtk_rb_tree_remove (self->items, tile); - tile = next; - } - while (tile) { next = gtk_rb_tree_node_get_next (tile); @@ -1113,7 +1076,6 @@ gtk_list_tile_gc (GtkListItemManager *self, case GTK_LIST_TILE_FOOTER: case GTK_LIST_TILE_UNMATCHED_HEADER: case GTK_LIST_TILE_UNMATCHED_FOOTER: - case GTK_LIST_TILE_FILLER: break; case GTK_LIST_TILE_REMOVED: @@ -1180,7 +1142,6 @@ gtk_list_item_manager_release_items (GtkListItemManager *self, deleted_section = TRUE; break; - case GTK_LIST_TILE_FILLER: case GTK_LIST_TILE_REMOVED: default: g_assert_not_reached (); @@ -1414,7 +1375,6 @@ gtk_list_item_manager_ensure_items (GtkListItemManager *self, break; case GTK_LIST_TILE_UNMATCHED_FOOTER: - case GTK_LIST_TILE_FILLER: case GTK_LIST_TILE_REMOVED: default: g_assert_not_reached (); @@ -1660,9 +1620,7 @@ gtk_list_item_manager_clear_model (GtkListItemManager *self) for (tile = gtk_list_tile_gc (self, gtk_list_item_manager_get_first (self)); tile; tile = gtk_list_tile_gc (self, tile)) - { - g_assert (tile->type == GTK_LIST_TILE_FILLER); - } + { } g_assert (gtk_rb_tree_get_root (self->items) == NULL); } @@ -1775,7 +1733,6 @@ gtk_list_item_manager_set_has_sections (GtkListItemManager *self, footer = tile; break; case GTK_LIST_TILE_ITEM: - case GTK_LIST_TILE_FILLER: case GTK_LIST_TILE_REMOVED: break; default: diff --git a/gtk/gtklistitemmanagerprivate.h b/gtk/gtklistitemmanagerprivate.h index fb8991fb69..ea855f4cc5 100644 --- a/gtk/gtklistitemmanagerprivate.h +++ b/gtk/gtklistitemmanagerprivate.h @@ -51,7 +51,6 @@ typedef enum GTK_LIST_TILE_FOOTER, GTK_LIST_TILE_UNMATCHED_HEADER, GTK_LIST_TILE_UNMATCHED_FOOTER, - GTK_LIST_TILE_FILLER, GTK_LIST_TILE_REMOVED, } GtkListTileType; @@ -127,8 +126,6 @@ void gtk_list_tile_set_area_size (GtkListItemMana GtkListTile * gtk_list_tile_split (GtkListItemManager *self, GtkListTile *tile, guint n_items); -GtkListTile * gtk_list_tile_append_filler (GtkListItemManager *self, - GtkListTile *previous); GtkListTile * gtk_list_tile_gc (GtkListItemManager *self, GtkListTile *tile); diff --git a/gtk/gtklistview.c b/gtk/gtklistview.c index c3ac2f681f..28c4706814 100644 --- a/gtk/gtklistview.c +++ b/gtk/gtklistview.c @@ -235,7 +235,6 @@ gtk_list_view_update_factories_with (GtkListView *self, case GTK_LIST_TILE_UNMATCHED_HEADER: case GTK_LIST_TILE_FOOTER: case GTK_LIST_TILE_UNMATCHED_FOOTER: - case GTK_LIST_TILE_FILLER: case GTK_LIST_TILE_REMOVED: g_assert (tile->widget == NULL); break; diff --git a/testsuite/gtk/listitemmanager.c b/testsuite/gtk/listitemmanager.c index ba5fbd54ba..40c6d2c298 100644 --- a/testsuite/gtk/listitemmanager.c +++ b/testsuite/gtk/listitemmanager.c @@ -71,9 +71,6 @@ print_list_item_manager_tiles (GtkListItemManager *items) g_string_append_c (string, ')'); break; - case GTK_LIST_TILE_FILLER: - g_string_append_c (string, '_'); - break; case GTK_LIST_TILE_REMOVED: g_string_append_c (string, '.'); break; @@ -160,11 +157,6 @@ check_list_item_manager (GtkListItemManager *items, n_items += tile->n_items; break; - case GTK_LIST_TILE_FILLER: - /* We don't add fillers */ - g_assert_not_reached (); - break; - case GTK_LIST_TILE_REMOVED: g_assert_cmpint (tile->n_items, ==, 0); g_assert_false (tile->widget); @@ -245,7 +237,6 @@ check_list_item_manager (GtkListItemManager *items, n_items += tile->n_items; break; - case GTK_LIST_TILE_FILLER: case GTK_LIST_TILE_REMOVED: default: g_assert_not_reached (); |