GtkFileChooserWidget: check for empty instead of NULL GListModel
authorLuca Bacci <luca.bacci982@gmail.com>
Sun, 1 May 2022 14:31:47 +0000 (16:31 +0200)
committerSimon McVittie <smcv@debian.org>
Wed, 4 May 2022 10:30:16 +0000 (11:30 +0100)
While porting GtkFileChooserWidget from GList to GListModel we did not
change some checks for NULL to checks for empty list.

Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/4851
Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/4858
Origin: upstream, 4.6.4, commit:6b740c86f907e171d5bec227677ccf81b0091d73

Gbp-Pq: Name GtkFileChooserWidget-check-for-empty-instead-of-NULL-GLis.patch

gtk/gtkfilechooserwidget.c

index be084da4212260d140d5a4744c1e59dbd4eee6c1..4c34c498408fa99fa1f89df59e7bf7ce97a142b0 100644 (file)
@@ -5448,7 +5448,7 @@ gtk_file_chooser_widget_get_files (GtkFileChooser *chooser)
        * So we want the selection to be "bar/foo.txt".  Jump to the case for the
        * filename entry to see if that is the case.
        */
-      if (info.result == NULL && impl->location_entry)
+      if (g_list_model_get_n_items (G_LIST_MODEL (info.result)) == 0 && impl->location_entry)
         goto file_entry;
     }
   else if (impl->location_entry &&
@@ -5497,7 +5497,7 @@ gtk_file_chooser_widget_get_files (GtkFileChooser *chooser)
    * then we fall back to the current directory
    */
   if (impl->action == GTK_FILE_CHOOSER_ACTION_SELECT_FOLDER &&
-      info.result == NULL)
+      g_list_model_get_n_items (G_LIST_MODEL (info.result)) == 0)
     {
       GFile *current_folder;