Re: GMC 4.5.7 coredumps =O(

[Miguel de Icaza <miguel nuclecu unam mx>]

Thanks a lot for the stack trace and the precise bug report.

Can you please try the patch included at the end.

> Is it really necessary to open and cache directories below like this anyway?  

The reason this is required is because the tree needs to know which of
those directories have childs (and thus, can mark those directories
with an open symbol [+]).

> I ask simply coz this takes many minutes here, and I'd have to wait
> for it each time I open gmc...  Could it be thrown over to a
> separate thread, maybe?

I will add an option to toggle the tree code off.

Unfortunatelly, there is no standard way to figure out what
directories you will like to see in the future and which not.

I have plans on doing the tree computation on the idle loop, but it
might take a while before I do it (mostly because I have a huge list
of other things to fix).

Miguel.

Index: treestore.c
===================================================================
RCS file: /debian/home/gnomecvs/mc/src/treestore.c,v
retrieving revision 1.3
diff -u -u -r1.3 treestore.c
--- treestore.c	1999/01/12 09:08:45	1.3
+++ treestore.c	1999/01/13 20:21:33
@@ -733,6 +733,9 @@
 		if (!entry)
 			return NULL;
 	}
+
+	if (entry->next == NULL)
+		return NULL;
 	
 	scan = xmalloc (sizeof (tree_scan), "");
 	scan->base = entry;