diff options
author | Benjamin Otte <otte@redhat.com> | 2020-07-03 04:21:07 +0200 |
---|---|---|
committer | Benjamin Otte <otte@redhat.com> | 2020-07-07 02:23:09 +0200 |
commit | 1ae939cf5a46701412974312ab2feb88aedd496a (patch) | |
tree | 71f9187121798170882ceed2538ded9b8d216d7f | |
parent | edf523fe305090a3b5abb05690c3aa11c70ab4cd (diff) | |
download | gtk+-1ae939cf5a46701412974312ab2feb88aedd496a.tar.gz |
icontheme: Use GtkVector
-rw-r--r-- | gtk/gtkicontheme.c | 55 |
1 files changed, 30 insertions, 25 deletions
diff --git a/gtk/gtkicontheme.c b/gtk/gtkicontheme.c index 7c46cb3267..2bdbd4a7c7 100644 --- a/gtk/gtkicontheme.c +++ b/gtk/gtkicontheme.c @@ -55,6 +55,14 @@ #include "gdk/gdktextureprivate.h" #include "gdk/gdkprofilerprivate.h" +#define GTK_VECTOR_ELEMENT_TYPE char * +#define GTK_VECTOR_NULL_TERMINATED 1 +#define GTK_VECTOR_FREE_FUNC g_free +#define GTK_VECTOR_TYPE_NAME GtkStrvBuilder +#define GTK_VECTOR_NAME gtk_strv_builder +#define GTK_VECTOR_PREALLOC 16 +#include "gtkvectorimpl.c" + /** * SECTION:gtkicontheme * @Short_description: Looking up icons by name @@ -2276,13 +2284,13 @@ real_choose_icon (GtkIconTheme *self, } static void -icon_name_list_add_icon (GPtrArray *icons, - const gchar *dir_suffix, - gchar *icon_name) +icon_name_list_add_icon (GtkStrvBuilder *icons, + const gchar *dir_suffix, + gchar *icon_name) { if (dir_suffix) - g_ptr_array_add (icons, g_strconcat (icon_name, dir_suffix, NULL)); - g_ptr_array_add (icons, icon_name); + gtk_strv_builder_append (icons, g_strconcat (icon_name, dir_suffix, NULL)); + gtk_strv_builder_append (icons, icon_name); } static GtkIconPaintable * @@ -2296,7 +2304,7 @@ choose_icon (GtkIconTheme *self, { gboolean has_regular = FALSE, has_symbolic = FALSE; GtkIconPaintable *icon; - GPtrArray *new_names; + GtkStrvBuilder new_names; const gchar *dir_suffix; guint i; @@ -2327,73 +2335,70 @@ choose_icon (GtkIconTheme *self, if ((flags & GTK_ICON_LOOKUP_FORCE_REGULAR) && has_symbolic) { - new_names = g_ptr_array_new_with_free_func (g_free); + gtk_strv_builder_init (&new_names); for (i = 0; icon_names[i]; i++) { if (icon_name_is_symbolic (icon_names[i], -1)) - icon_name_list_add_icon (new_names, dir_suffix, g_strndup (icon_names[i], strlen (icon_names[i]) - strlen ("-symbolic"))); + icon_name_list_add_icon (&new_names, dir_suffix, g_strndup (icon_names[i], strlen (icon_names[i]) - strlen ("-symbolic"))); else - icon_name_list_add_icon (new_names, dir_suffix, g_strdup (icon_names[i])); + icon_name_list_add_icon (&new_names, dir_suffix, g_strdup (icon_names[i])); } for (i = 0; icon_names[i]; i++) { if (icon_name_is_symbolic (icon_names[i], -1)) - icon_name_list_add_icon (new_names, dir_suffix, g_strdup (icon_names[i])); + icon_name_list_add_icon (&new_names, dir_suffix, g_strdup (icon_names[i])); } - g_ptr_array_add (new_names, NULL); icon = real_choose_icon (self, - (const gchar **) new_names->pdata, + (const char **) gtk_strv_builder_get_data (&new_names), size, scale, flags & ~(GTK_ICON_LOOKUP_FORCE_REGULAR | GTK_ICON_LOOKUP_FORCE_SYMBOLIC), non_blocking); - g_ptr_array_free (new_names, TRUE); + gtk_strv_builder_clear (&new_names); } else if ((flags & GTK_ICON_LOOKUP_FORCE_SYMBOLIC) && has_regular) { - new_names = g_ptr_array_new_with_free_func (g_free); + gtk_strv_builder_init (&new_names); for (i = 0; icon_names[i]; i++) { if (!icon_name_is_symbolic (icon_names[i], -1)) - icon_name_list_add_icon (new_names, dir_suffix, g_strconcat (icon_names[i], "-symbolic", NULL)); + icon_name_list_add_icon (&new_names, dir_suffix, g_strconcat (icon_names[i], "-symbolic", NULL)); else - icon_name_list_add_icon (new_names, dir_suffix, g_strdup (icon_names[i])); + icon_name_list_add_icon (&new_names, dir_suffix, g_strdup (icon_names[i])); } for (i = 0; icon_names[i]; i++) { if (!icon_name_is_symbolic (icon_names[i], -1)) - icon_name_list_add_icon (new_names, dir_suffix, g_strdup (icon_names[i])); + icon_name_list_add_icon (&new_names, dir_suffix, g_strdup (icon_names[i])); } - g_ptr_array_add (new_names, NULL); icon = real_choose_icon (self, - (const gchar **) new_names->pdata, + (const char **) gtk_strv_builder_get_data (&new_names), size, scale, flags & ~(GTK_ICON_LOOKUP_FORCE_REGULAR | GTK_ICON_LOOKUP_FORCE_SYMBOLIC), non_blocking); - g_ptr_array_free (new_names, TRUE); + gtk_strv_builder_clear (&new_names); } else if (dir_suffix) { - new_names = g_ptr_array_new_with_free_func (g_free); + gtk_strv_builder_init (&new_names); for (i = 0; icon_names[i]; i++) { - icon_name_list_add_icon (new_names, dir_suffix, g_strdup (icon_names[i])); + icon_name_list_add_icon (&new_names, dir_suffix, g_strdup (icon_names[i])); } - g_ptr_array_add (new_names, NULL); icon = real_choose_icon (self, - (const gchar **) new_names->pdata, + (const char **) gtk_strv_builder_get_data (&new_names), size, scale, flags & ~(GTK_ICON_LOOKUP_FORCE_REGULAR | GTK_ICON_LOOKUP_FORCE_SYMBOLIC), non_blocking); - g_ptr_array_free (new_names, TRUE); + gtk_strv_builder_clear (&new_names); } else { |