From a012011631a04c79e283b494acf91706bfb48f43 Mon Sep 17 00:00:00 2001 From: Přemysl Eric Janouch Date: Fri, 14 Apr 2023 07:11:49 +0200 Subject: Deduplicate file information structures --- fiv-browser.c | 70 ++++++++++-------------- fiv-io.c | 170 +++++++++++++++++++++++++++++++++++----------------------- fiv-io.h | 20 +++---- fiv-sidebar.c | 4 +- fiv.c | 4 +- 5 files changed, 146 insertions(+), 122 deletions(-) diff --git a/fiv-browser.c b/fiv-browser.c index ccc070a..0c14c9b 100644 --- a/fiv-browser.c +++ b/fiv-browser.c @@ -108,14 +108,8 @@ static cairo_user_data_key_t fiv_browser_key_mtime_msec; /// The original file size of source images for thumbnails. static cairo_user_data_key_t fiv_browser_key_filesize; -// TODO(p): Include FivIoModelEntry data by reference. struct entry { - gchar *uri; ///< GIO URI - gchar *target_uri; ///< GIO URI for any target - gchar *display_name; ///< Label for the file - guint64 filesize; ///< Filesize in bytes - gint64 mtime_msec; ///< Modification time in milliseconds - + FivIoModelEntry *e; ///< Reference to model entry cairo_surface_t *thumbnail; ///< Prescaled thumbnail GIcon *icon; ///< If no thumbnail, use this icon }; @@ -123,9 +117,7 @@ struct entry { static void entry_free(Entry *self) { - g_free(self->uri); - g_free(self->target_uri); - g_free(self->display_name); + fiv_io_model_entry_unref(self->e); g_clear_pointer(&self->thumbnail, cairo_surface_destroy); g_clear_object(&self->icon); } @@ -228,7 +220,8 @@ relayout(FivBrowser *self, int width) PangoLayout *label = NULL; if (self->show_labels) { - label = gtk_widget_create_pango_layout(widget, entry->display_name); + label = gtk_widget_create_pango_layout( + widget, entry->e->display_name); pango_layout_set_width( label, (width - 2 * self->glow_w) * PANGO_SCALE); pango_layout_set_alignment(label, PANGO_ALIGN_CENTER); @@ -505,15 +498,15 @@ rescale_thumbnail(cairo_surface_t *thumbnail, double row_height) return scaled; } -static char * +static const char * entry_system_wide_uri(const Entry *self) { // "recent" and "trash", e.g., also have "standard::target-uri" set, // but we'd like to avoid saving their thumbnails. - if (self->target_uri && fiv_collection_uri_matches(self->uri)) - return self->target_uri; + if (self->e->target_uri && fiv_collection_uri_matches(self->e->uri)) + return self->e->target_uri; - return self->uri; + return self->e->uri; } static void @@ -522,10 +515,10 @@ entry_set_surface_user_data(const Entry *self) // This choice of mtime favours unnecessary thumbnail reloading // over retaining stale data (consider both calling functions). cairo_surface_set_user_data(self->thumbnail, - &fiv_browser_key_mtime_msec, (void *) (intptr_t) self->mtime_msec, + &fiv_browser_key_mtime_msec, (void *) (intptr_t) self->e->mtime_msec, NULL); cairo_surface_set_user_data(self->thumbnail, - &fiv_browser_key_filesize, (void *) (uintptr_t) self->filesize, + &fiv_browser_key_filesize, (void *) (uintptr_t) self->e->filesize, NULL); } @@ -538,19 +531,19 @@ entry_add_thumbnail(gpointer data, gpointer user_data) FivBrowser *browser = FIV_BROWSER(user_data); cairo_surface_t *cached = - g_hash_table_lookup(browser->thumbnail_cache, self->uri); + g_hash_table_lookup(browser->thumbnail_cache, self->e->uri); if (cached && (intptr_t) cairo_surface_get_user_data(cached, - &fiv_browser_key_mtime_msec) == (intptr_t) self->mtime_msec && + &fiv_browser_key_mtime_msec) == (intptr_t) self->e->mtime_msec && (uintptr_t) cairo_surface_get_user_data(cached, - &fiv_browser_key_filesize) == (uintptr_t) self->filesize) { + &fiv_browser_key_filesize) == (uintptr_t) self->e->filesize) { self->thumbnail = cairo_surface_reference(cached); // TODO(p): If this hit is low-quality, see if a high-quality thumbnail // hasn't been produced without our knowledge (avoid launching a minion // unnecessarily; we might also shift the concern there). } else { cairo_surface_t *found = fiv_thumbnail_lookup( - entry_system_wide_uri(self), self->mtime_msec, self->filesize, + entry_system_wide_uri(self), self->e->mtime_msec, self->e->filesize, browser->item_size); self->thumbnail = rescale_thumbnail(found, browser->item_height); } @@ -563,7 +556,7 @@ entry_add_thumbnail(gpointer data, gpointer user_data) // Fall back to symbolic icons, though there's only so much we can do // in parallel--GTK+ isn't thread-safe. - GFile *file = g_file_new_for_uri(self->uri); + GFile *file = g_file_new_for_uri(self->e->uri); GFileInfo *info = g_file_query_info(file, G_FILE_ATTRIBUTE_STANDARD_NAME "," G_FILE_ATTRIBUTE_STANDARD_SYMBOLIC_ICON, @@ -640,7 +633,7 @@ reload_thumbnails(FivBrowser *self) for (guint i = 0; i < self->entries->len; i++) { Entry *entry = &g_array_index(self->entries, Entry, i); if (entry->thumbnail) { - g_hash_table_insert(self->thumbnail_cache, g_strdup(entry->uri), + g_hash_table_insert(self->thumbnail_cache, g_strdup(entry->e->uri), cairo_surface_reference(entry->thumbnail)); } @@ -680,7 +673,7 @@ thumbnailer_reprocess_entry(FivBrowser *self, GBytes *output, Entry *entry) } entry_set_surface_user_data(entry); - g_hash_table_insert(self->thumbnail_cache, g_strdup(entry->uri), + g_hash_table_insert(self->thumbnail_cache, g_strdup(entry->e->uri), cairo_surface_reference(entry->thumbnail)); } @@ -1128,7 +1121,7 @@ fiv_browser_draw(GtkWidget *widget, cairo_t *cr) static gboolean open_entry(GtkWidget *self, const Entry *entry, gboolean new_window) { - GFile *location = g_file_new_for_uri(entry->uri); + GFile *location = g_file_new_for_uri(entry->e->uri); g_signal_emit(self, browser_signals[ITEM_ACTIVATED], 0, location, new_window ? GTK_PLACES_OPEN_NEW_WINDOW : GTK_PLACES_OPEN_NORMAL); g_object_unref(location); @@ -1198,7 +1191,7 @@ fiv_browser_button_press_event(GtkWidget *widget, GdkEventButton *event) // no matter what its new location is. gdk_window_set_cursor(gtk_widget_get_window(widget), NULL); - GFile *file = g_file_new_for_uri(entry->uri); + GFile *file = g_file_new_for_uri(entry->e->uri); show_context_menu(widget, file); g_object_unref(file); return GDK_EVENT_STOP; @@ -1342,8 +1335,8 @@ fiv_browser_drag_data_get(GtkWidget *widget, { FivBrowser *self = FIV_BROWSER(widget); if (self->selected) { - (void) gtk_selection_data_set_uris( - data, (gchar *[]) {entry_system_wide_uri(self->selected), NULL}); + (void) gtk_selection_data_set_uris(data, (gchar *[]) + {(gchar *) entry_system_wide_uri(self->selected), NULL}); } } @@ -1520,7 +1513,7 @@ fiv_browser_key_press_event(GtkWidget *widget, GdkEventKey *event) case GDK_KEY_Return: if (self->selected) { GtkWindow *window = GTK_WINDOW(gtk_widget_get_toplevel(widget)); - fiv_context_menu_information(window, self->selected->uri); + fiv_context_menu_information(window, self->selected->e->uri); } return GDK_EVENT_STOP; } @@ -1555,7 +1548,7 @@ fiv_browser_query_tooltip(GtkWidget *widget, gint x, gint y, if (!entry) return FALSE; - gtk_tooltip_set_text(tooltip, entry->display_name); + gtk_tooltip_set_text(tooltip, entry->e->display_name); return TRUE; } @@ -1569,7 +1562,7 @@ fiv_browser_popup_menu(GtkWidget *widget) GFile *file = NULL; GdkRectangle rect = {}; if (self->selected) { - file = g_file_new_for_uri(self->selected->uri); + file = g_file_new_for_uri(self->selected->e->uri); rect = entry_rect(self, self->selected); rect.x += rect.width / 2; rect.y += rect.height / 2; @@ -1607,7 +1600,7 @@ on_long_press(GtkGestureLongPress *lp, gdouble x, gdouble y, gpointer user_data) // It might also be possible to have long-press just select items, // and show some kind of toolbar with available actions. - GFile *file = g_file_new_for_uri(entry->uri); + GFile *file = g_file_new_for_uri(entry->e->uri); gtk_menu_popup_at_rect(fiv_context_menu_new(widget, file), window, &(GdkRectangle) {.x = x, .y = y}, GDK_GRAVITY_NORTH_WEST, GDK_GRAVITY_NORTH_WEST, event); @@ -1823,21 +1816,16 @@ on_model_files_changed(FivIoModel *model, FivBrowser *self) gchar *selected_uri = NULL; if (self->selected) - selected_uri = g_strdup(self->selected->uri); + selected_uri = g_strdup(self->selected->e->uri); thumbnailers_abort(self); g_array_set_size(self->entries, 0); g_array_set_size(self->layouted_rows, 0); gsize len = 0; - const FivIoModelEntry *files = fiv_io_model_get_files(self->model, &len); + FivIoModelEntry *const *files = fiv_io_model_get_files(self->model, &len); for (gsize i = 0; i < len; i++) { - Entry e = {.thumbnail = NULL, - .uri = g_strdup(files[i].uri), - .target_uri = g_strdup(files[i].target_uri), - .display_name = g_strdup(files[i].display_name), - .filesize = files[i].filesize, - .mtime_msec = files[i].mtime_msec}; + Entry e = {.e = fiv_io_model_entry_ref(files[i])}; g_array_append_val(self->entries, e); } @@ -1888,7 +1876,7 @@ fiv_browser_select(FivBrowser *self, const char *uri) for (guint i = 0; i < self->entries->len; i++) { const Entry *entry = &g_array_index(self->entries, Entry, i); - if (!g_strcmp0(entry->uri, uri)) { + if (!g_strcmp0(entry->e->uri, uri)) { self->selected = entry; scroll_to_selection(self); break; diff --git a/fiv-io.c b/fiv-io.c index 87a4e01..c4914a2 100644 --- a/fiv-io.c +++ b/fiv-io.c @@ -3044,21 +3044,10 @@ fiv_io_serialize_for_search(cairo_surface_t *surface, GError **error) #include "xdg.h" -static void -model_entry_finalize(FivIoModelEntry *entry) -{ - g_free(entry->uri); - g_free(entry->target_uri); - g_free(entry->display_name); - g_free(entry->collate_key); -} - -static GArray * +static GPtrArray * model_entry_array_new(void) { - GArray *a = g_array_new(FALSE, TRUE, sizeof(FivIoModelEntry)); - g_array_set_clear_func(a, (GDestroyNotify) model_entry_finalize); - return a; + return g_ptr_array_new_with_free_func(g_rc_box_release); } struct _FivIoModel { @@ -3067,8 +3056,8 @@ struct _FivIoModel { GFile *directory; ///< Currently loaded directory GFileMonitor *monitor; ///< "directory" monitoring - GArray *subdirs; ///< "directory" contents - GArray *files; ///< "directory" contents + GPtrArray *subdirs; ///< "directory" contents + GPtrArray *files; ///< "directory" contents FivIoModelSort sort_field; ///< How to sort gboolean sort_descending; ///< Whether to sort in reverse @@ -3150,8 +3139,8 @@ model_compare_entries(FivIoModel *self, static gint model_compare(gconstpointer a, gconstpointer b, gpointer user_data) { - const FivIoModelEntry *entry1 = a; - const FivIoModelEntry *entry2 = b; + const FivIoModelEntry *entry1 = *(const FivIoModelEntry **) a; + const FivIoModelEntry *entry2 = *(const FivIoModelEntry **) b; GFile *file1 = g_file_new_for_uri(entry1->uri); GFile *file2 = g_file_new_for_uri(entry2->uri); int result = model_compare_entries(user_data, entry1, file1, entry2, file2); @@ -3160,14 +3149,81 @@ model_compare(gconstpointer a, gconstpointer b, gpointer user_data) return result; } +static size_t +model_strsize(const char *string) +{ + if (!string) + return 0; + + return strlen(string) + 1; +} + +static char * +model_strappend(char **p, const char *string, size_t size) +{ + if (!string) + return *p; + + char *destination = memcpy(*p, string, size); + *p += size; + return destination; +} + +static FivIoModelEntry * +model_entry_new(GFile *file, GFileInfo *info) +{ + gchar *uri = g_file_get_uri(file); + const gchar *target_uri = g_file_info_get_attribute_string( + info, G_FILE_ATTRIBUTE_STANDARD_TARGET_URI); + const gchar *display_name = g_file_info_get_display_name(info); + + // TODO(p): Make it possible to use g_utf8_collate_key() instead, + // which does not use natural sorting. + gchar *parse_name = g_file_get_parse_name(file); + gchar *collate_key = g_utf8_collate_key_for_filename(parse_name, -1); + g_free(parse_name); + + // The entries are immutable. Packing them into the structure + // should help memory usage as well as performance. + size_t size_uri = model_strsize(uri); + size_t size_target_uri = model_strsize(target_uri); + size_t size_display_name = model_strsize(display_name); + size_t size_collate_key = model_strsize(collate_key); + + FivIoModelEntry *entry = g_rc_box_alloc0(sizeof *entry + + size_uri + + size_target_uri + + size_display_name + + size_collate_key); + + gchar *p = (gchar *) entry + sizeof *entry; + entry->uri = model_strappend(&p, uri, size_uri); + entry->target_uri = model_strappend(&p, target_uri, size_target_uri); + entry->display_name = model_strappend(&p, display_name, size_display_name); + entry->collate_key = model_strappend(&p, collate_key, size_collate_key); + + entry->filesize = (guint64) g_file_info_get_size(info); + + GDateTime *mtime = g_file_info_get_modification_date_time(info); + if (mtime) { + entry->mtime_msec = g_date_time_to_unix(mtime) * 1000 + + g_date_time_get_microsecond(mtime) / 1000; + g_date_time_unref(mtime); + } + + g_free(uri); + g_free(collate_key); + return entry; +} + static gboolean model_reload_to(FivIoModel *self, GFile *directory, - GArray *subdirs, GArray *files, GError **error) + GPtrArray *subdirs, GPtrArray *files, GError **error) { if (subdirs) - g_array_set_size(subdirs, 0); + g_ptr_array_set_size(subdirs, 0); if (files) - g_array_set_size(files, 0); + g_ptr_array_set_size(files, 0); GFileEnumerator *enumerator = g_file_enumerate_children(directory, G_FILE_ATTRIBUTE_STANDARD_TYPE "," @@ -3197,42 +3253,22 @@ model_reload_to(FivIoModel *self, GFile *directory, if (self->filtering && g_file_info_get_is_hidden(info)) continue; - GArray *target = NULL; + GPtrArray *target = NULL; if (g_file_info_get_file_type(info) == G_FILE_TYPE_DIRECTORY) target = subdirs; else if (!self->filtering || model_supports(self, g_file_info_get_name(info))) target = files; - if (!target) - continue; - - FivIoModelEntry entry = {.uri = g_file_get_uri(child), - .target_uri = g_strdup(g_file_info_get_attribute_string( - info, G_FILE_ATTRIBUTE_STANDARD_TARGET_URI)), - .display_name = g_strdup(g_file_info_get_display_name(info)), - .filesize = (guint64) g_file_info_get_size(info)}; - - GDateTime *mtime = g_file_info_get_modification_date_time(info); - if (mtime) { - entry.mtime_msec = g_date_time_to_unix(mtime) * 1000 + - g_date_time_get_microsecond(mtime) / 1000; - g_date_time_unref(mtime); - } - - gchar *parse_name = g_file_get_parse_name(child); - // TODO(p): Make it possible to use g_utf8_collate_key() instead, - // which does not use natural sorting. - entry.collate_key = g_utf8_collate_key_for_filename(parse_name, -1); - g_free(parse_name); - g_array_append_val(target, entry); + if (target) + g_ptr_array_add(target, model_entry_new(child, info)); } g_object_unref(enumerator); if (subdirs) - g_array_sort_with_data(subdirs, model_compare, self); + g_ptr_array_sort_with_data(subdirs, model_compare, self); if (files) - g_array_sort_with_data(files, model_compare, self); + g_ptr_array_sort_with_data(files, model_compare, self); return TRUE; } @@ -3251,8 +3287,8 @@ model_reload(FivIoModel *self, GError **error) static void model_resort(FivIoModel *self) { - g_array_sort_with_data(self->subdirs, model_compare, self); - g_array_sort_with_data(self->files, model_compare, self); + g_ptr_array_sort_with_data(self->subdirs, model_compare, self); + g_ptr_array_sort_with_data(self->files, model_compare, self); g_signal_emit(self, model_signals[FILES_CHANGED], 0); g_signal_emit(self, model_signals[SUBDIRECTORIES_CHANGED], 0); @@ -3265,13 +3301,13 @@ static GFile * model_last_deep_subdirectory(FivIoModel *self, GFile *directory) { GFile *result = NULL; - GArray *subdirs = model_entry_array_new(); + GPtrArray *subdirs = model_entry_array_new(); if (!model_reload_to(self, directory, subdirs, NULL, NULL)) goto out; if (subdirs->len) { - GFile *last = g_file_new_for_uri( - g_array_index(subdirs, FivIoModelEntry, subdirs->len - 1).uri); + FivIoModelEntry *entry = g_ptr_array_index(subdirs, subdirs->len - 1); + GFile *last = g_file_new_for_uri(entry->uri); result = model_last_deep_subdirectory(self, last); g_object_unref(last); } else { @@ -3279,7 +3315,7 @@ model_last_deep_subdirectory(FivIoModel *self, GFile *directory) } out: - g_array_free(subdirs, TRUE); + g_ptr_array_free(subdirs, TRUE); return result; } @@ -3293,13 +3329,13 @@ fiv_io_model_get_previous_directory(FivIoModel *self) return NULL; GFile *result = NULL; - GArray *subdirs = model_entry_array_new(); + GPtrArray *subdirs = model_entry_array_new(); if (!model_reload_to(self, parent_directory, subdirs, NULL, NULL)) goto out; for (gsize i = 0; i < subdirs->len; i++) { - GFile *file = g_file_new_for_uri( - g_array_index(subdirs, FivIoModelEntry, i).uri); + FivIoModelEntry *entry = g_ptr_array_index(subdirs, i); + GFile *file = g_file_new_for_uri(entry->uri); if (g_file_equal(file, self->directory)) { g_object_unref(file); break; @@ -3318,7 +3354,7 @@ fiv_io_model_get_previous_directory(FivIoModel *self) out: g_object_unref(parent_directory); - g_array_free(subdirs, TRUE); + g_ptr_array_free(subdirs, TRUE); return result; } @@ -3331,14 +3367,14 @@ model_next_directory_within_parents(FivIoModel *self, GFile *directory) return NULL; GFile *result = NULL; - GArray *subdirs = model_entry_array_new(); + GPtrArray *subdirs = model_entry_array_new(); if (!model_reload_to(self, parent_directory, subdirs, NULL, NULL)) goto out; gboolean found_self = FALSE; for (gsize i = 0; i < subdirs->len; i++) { - result = g_file_new_for_uri( - g_array_index(subdirs, FivIoModelEntry, i).uri); + FivIoModelEntry *entry = g_ptr_array_index(subdirs, i); + result = g_file_new_for_uri(entry->uri); if (found_self) goto out; @@ -3350,7 +3386,7 @@ model_next_directory_within_parents(FivIoModel *self, GFile *directory) out: g_object_unref(parent_directory); - g_array_free(subdirs, TRUE); + g_ptr_array_free(subdirs, TRUE); return result; } @@ -3360,8 +3396,8 @@ fiv_io_model_get_next_directory(FivIoModel *self) g_return_val_if_fail(FIV_IS_IO_MODEL(self), NULL); if (self->subdirs->len) { - return g_file_new_for_uri( - g_array_index(self->subdirs, FivIoModelEntry, 0).uri); + FivIoModelEntry *entry = g_ptr_array_index(self->subdirs, 0); + return g_file_new_for_uri(entry->uri); } return model_next_directory_within_parents(self, self->directory); @@ -3379,8 +3415,8 @@ fiv_io_model_finalize(GObject *gobject) g_clear_object(&self->directory); g_clear_object(&self->monitor); - g_array_free(self->subdirs, TRUE); - g_array_free(self->files, TRUE); + g_ptr_array_free(self->subdirs, TRUE); + g_ptr_array_free(self->files, TRUE); G_OBJECT_CLASS(fiv_io_model_parent_class)->finalize(gobject); } @@ -3513,18 +3549,18 @@ fiv_io_model_get_location(FivIoModel *self) return self->directory; } -const FivIoModelEntry * +FivIoModelEntry *const * fiv_io_model_get_files(FivIoModel *self, gsize *len) { *len = self->files->len; - return (const FivIoModelEntry *) self->files->data; + return (FivIoModelEntry *const *) self->files->pdata; } -const FivIoModelEntry * +FivIoModelEntry *const * fiv_io_model_get_subdirs(FivIoModel *self, gsize *len) { *len = self->subdirs->len; - return (const FivIoModelEntry *) self->subdirs->data; + return (FivIoModelEntry *const *) self->subdirs->pdata; } // --- Export ------------------------------------------------------------------ diff --git a/fiv-io.h b/fiv-io.h index 71917da..3f8c65b 100644 --- a/fiv-io.h +++ b/fiv-io.h @@ -135,21 +135,21 @@ GFile *fiv_io_model_get_previous_directory(FivIoModel *self); /// Returns the next VFS directory in order, or NULL. GFile *fiv_io_model_get_next_directory(FivIoModel *self); -// TODO(p): Turn this into a reference-counted object. -// - If using g_rc_box_*(), we should wrap the {_acquire,_release_full}() -// functions as fiv_io_model_entry_{ref,unref}(). -// - Ideally, all the strings would follow the struct immediately. +// These objects are reference-counted using GRcBox. typedef struct { - gchar *uri; ///< GIO URI - gchar *target_uri; ///< GIO URI for any target - gchar *display_name; ///< Label for the file - gchar *collate_key; ///< Collate key for the filename + const char *uri; ///< GIO URI + const char *target_uri; ///< GIO URI for any target + const char *display_name; ///< Label for the file + const char *collate_key; ///< Collate key for the filename guint64 filesize; ///< Filesize in bytes gint64 mtime_msec; ///< Modification time in milliseconds } FivIoModelEntry; -const FivIoModelEntry *fiv_io_model_get_files(FivIoModel *self, gsize *len); -const FivIoModelEntry *fiv_io_model_get_subdirs(FivIoModel *self, gsize *len); +#define fiv_io_model_entry_ref(e) g_rc_box_acquire(e) +#define fiv_io_model_entry_unref(e) g_rc_box_release(e) + +FivIoModelEntry *const *fiv_io_model_get_files(FivIoModel *self, gsize *len); +FivIoModelEntry *const *fiv_io_model_get_subdirs(FivIoModel *self, gsize *len); // --- Export ------------------------------------------------------------------ diff --git a/fiv-sidebar.c b/fiv-sidebar.c index f688720..6525067 100644 --- a/fiv-sidebar.c +++ b/fiv-sidebar.c @@ -357,10 +357,10 @@ update_location(FivSidebar *self) gtk_container_add(GTK_CONTAINER(self->listbox), row); gsize len = 0; - const FivIoModelEntry *subdirs = + FivIoModelEntry *const *subdirs = fiv_io_model_get_subdirs(self->model, &len); for (gsize i = 0; i < len; i++) { - GFile *file = g_file_new_for_uri(subdirs[i].uri); + GFile *file = g_file_new_for_uri(subdirs[i]->uri); if ((row = create_row(self, file, "go-down-symbolic"))) gtk_container_add(GTK_CONTAINER(self->listbox), row); g_object_unref(file); diff --git a/fiv.c b/fiv.c index 475d40b..8d25eff 100644 --- a/fiv.c +++ b/fiv.c @@ -783,11 +783,11 @@ on_model_files_changed(FivIoModel *model, G_GNUC_UNUSED gpointer user_data) g_return_if_fail(model == g.model); gsize len = 0; - const FivIoModelEntry *files = fiv_io_model_get_files(g.model, &len); + FivIoModelEntry *const *files = fiv_io_model_get_files(g.model, &len); g_ptr_array_free(g.files, TRUE); g.files = g_ptr_array_new_full(len, g_free); for (gsize i = 0; i < len; i++) - g_ptr_array_add(g.files, g_strdup(files[i].uri)); + g_ptr_array_add(g.files, g_strdup(files[i]->uri)); update_files_index(); -- cgit v1.2.3-54-g00ecf