diff options
author | Timm Bäder <mail@baedert.org> | 2020-04-30 09:08:48 +0200 |
---|---|---|
committer | Timm Bäder <mail@baedert.org> | 2020-05-05 08:20:10 +0200 |
commit | 0e73bfa202eb0fd82b954c39b53aead481cd837e (patch) | |
tree | 2fc20edddd6290cdf019c5f15cb7ba0eb92ed1f3 /gtk/gtkshortcutmanager.c | |
parent | fb9b54d4b209afc10d7794dd7356e436ee9aa36e (diff) | |
download | gtk+-0e73bfa202eb0fd82b954c39b53aead481cd837e.tar.gz |
shortcutmanager: Speed up default_remove_controller a bit
Only call get_n_items on the model once and remove two unnecessary
casts.
Diffstat (limited to 'gtk/gtkshortcutmanager.c')
-rw-r--r-- | gtk/gtkshortcutmanager.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/gtk/gtkshortcutmanager.c b/gtk/gtkshortcutmanager.c index c6530c0efe..9aa3577703 100644 --- a/gtk/gtkshortcutmanager.c +++ b/gtk/gtkshortcutmanager.c @@ -113,16 +113,16 @@ gtk_shortcut_manager_default_remove_controller (GtkShortcutManager *self, if (model) { GListModel *store; - guint position; + guint position, len; - store = gtk_flatten_list_model_get_model (model); + store = gtk_flatten_list_model_get_model (model); #if 0 && GLIB_CHECK_VERSION(2,64,0) if (_g_list_store_find (G_LIST_STORE (store), controller, &position)) g_list_store_remove (G_LIST_STORE (store), position); #else - for (position = 0; position < g_list_model_get_n_items (G_LIST_MODEL (store)); position++) + for (position = 0, len = g_list_model_get_n_items (store); position < len; position++) { - GtkShortcutController *item = g_list_model_get_item (G_LIST_MODEL (store), position); + GtkShortcutController *item = g_list_model_get_item (store, position); g_object_unref (item); if (item == controller) { |