summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristian Persch <chpe@src.gnome.org>2008-10-02 19:56:01 +0000
committerChristian Persch <chpe@src.gnome.org>2008-10-02 19:56:01 +0000
commit1d9431b920bf094ba587c4bb0a087ad286d7f195 (patch)
tree1c4e0bcc33d6d76822dca514e24489f03e133bab
parente65267157aae9c11ccc83979b9ec2932a31c634a (diff)
downloadgdk-pixbuf-1d9431b920bf094ba587c4bb0a087ad286d7f195.tar.gz
Bug 554704 – gtkfilesystemmodel does too much work
svn path=/branches/gtk-2-14/; revision=21578
-rw-r--r--ChangeLog7
-rw-r--r--gtk/gtkfilesystemmodel.c8
2 files changed, 11 insertions, 4 deletions
diff --git a/ChangeLog b/ChangeLog
index 922df601f..e856a9d4e 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
2008-10-02 Christian Persch
+ Bug 554704 – gtkfilesystemmodel does too much work
+
+ * gtk/gtkfilesystemmodel.c: Replace g_slist_length()<1 check with a
+ simple NULL check.
+
+2008-10-02 Christian Persch
+
Bug 554701 – filechooser spams console with useless warnings
* gtk/gtkfilesystem.c.c: Don't warn if the async call was simply
diff --git a/gtk/gtkfilesystemmodel.c b/gtk/gtkfilesystemmodel.c
index de08a6374..9536d0fa7 100644
--- a/gtk/gtkfilesystemmodel.c
+++ b/gtk/gtkfilesystemmodel.c
@@ -994,7 +994,7 @@ ref_path_cb (GCancellable *cancellable,
if (folder)
info->cleanups = g_slist_prepend (info->cleanups, folder);
- else if (g_slist_length (info->files) == 1
+ else if ((info->files != NULL && info->files->next == NULL) /* g_slist_length == 1 */
&& g_file_equal (info->node->file, info->files->data))
{
/* Done, now call the function */
@@ -1025,7 +1025,7 @@ ref_path_cb (GCancellable *cancellable,
g_object_unref (info->files->data);
info->files = g_slist_remove (info->files, info->files->data);
- if (g_slist_length (info->files) < 1)
+ if (info->files == NULL)
{
/* Done, now call the function */
if (info->node)
@@ -1136,7 +1136,7 @@ _gtk_file_system_model_path_do (GtkFileSystemModel *model,
}
g_object_unref (parent_file);
- if (g_slist_length (files) < 1)
+ if (files == NULL)
return;
/* Now we have all paths, except the root path */
@@ -1153,7 +1153,7 @@ _gtk_file_system_model_path_do (GtkFileSystemModel *model,
g_object_unref (files->data);
files = g_slist_remove (files, files->data);
- if (g_slist_length (files) < 1)
+ if (files == NULL)
{
/* Done, now call the function */
if (node)