Re: desktop icon patch

[Dave Camp <dave ximian com>]

This part of the patch:

        if (vfs_uri == NULL) {
-               directory_vfs_uri = NULL;
+               if (eel_uri_is_desktop (uri) &&
+                   strcmp (uri, EEL_DESKTOP_URI) != 0) {
+                       directory_uri = g_strdup (EEL_DESKTOP_URI);
+               }

Broke if x-nautilus-desktop:/// was loaded rather than
x-nautilus-desktop:.  I just committed this patch:

        /* Make VFS version of directory URI. */
        if (vfs_uri == NULL) {
                if (eel_uri_is_desktop (uri) &&
-                   strcmp (uri, EEL_DESKTOP_URI) != 0) {
+                   !eel_uris_match (uri, EEL_DESKTOP_URI)) {
                        directory_uri = g_strdup (EEL_DESKTOP_URI);
                }
 
-dave 

On Thu, 2003-05-15 at 14:30, Alexander Larsson wrote:
> Ok, here is the patch with all the desktop changes i talked about in the
> last email. If anyone want to look for silly bugs in it, be my guest.
> 
> =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
>  Alexander Larsson                                            Red Hat, Inc 
>                    alexl redhat com    alla lysator liu se 
> He's a time-tossed guerilla farmboy for the 21st century. She's a high-kicking 
> winged mechanic married to the Mob. They fight crime!