On Wednesday, May 30, 2001, at 10:19 AM, Frederic Crozat wrote:
I've found where the problem is : when we fill the list with all the available oaf server which can handle text/html mime-type, the order of the list is changed after each insertion. Therefore, when there is no user defined component for text/html, first element of resulting list is returned, which is not the first element of the oaf response to the query. The following patch corrects the problem. Can I commit ? --- nautilus/libnautilus-private/nautilus-mime-actions.c.orig Fri May 11 03:30:31 2001 +++ nautilus/libnautilus-private/nautilus-mime-actions.c Wed May 30 19:13:13 2001 @@ -1632,7 +1632,7 @@ CORBA_free (oaf_result); - return g_list_reverse (retval); + return retval; }
Please don't commit this patch. This does the wrong thing. The reverse is needed in that function nautilus_do_component_query, because the function walks through the query results and prepends each one to the list. So the list ends up in reverse order and must be reversed before returning.
Or maybe I am missing something?My guess is that one of the callers has a mistake in its code. Could you be more specific about which function was called and returned the wrong MIME type?
You say that "the order of the list is changed after each insertion". Can you point me at the code that changes the order of the list?
-- Darin