diff --git a/ChangeLog b/ChangeLog index 08085f91c4..8f08f1b1f1 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +2006-08-29 Emmanuele Bassi + + Backport from HEAD + + * gtk/gtkrecentchoosermenu.c + (gtk_recent_chooser_menu_set_current_uri): Break when an item + is found and activated. (#353449, based on a patch from Jan Arne + Petersen) + 2006-08-29 Paolo Borelli * tests/autotestfilesystem.c: add this file from HEAD to fix diff --git a/gtk/gtkrecentchoosermenu.c b/gtk/gtkrecentchoosermenu.c index 0e04b86575..78e452ab84 100644 --- a/gtk/gtkrecentchoosermenu.c +++ b/gtk/gtkrecentchoosermenu.c @@ -411,26 +411,28 @@ gtk_recent_chooser_menu_set_current_uri (GtkRecentChooser *chooser, if (!info) continue; - if (0 == strcmp (uri, gtk_recent_info_get_uri (info))) - found = TRUE; + if (strcmp (uri, gtk_recent_info_get_uri (info)) == 0) + { + gtk_menu_shell_activate_item (GTK_MENU_SHELL (menu), + menu_item, + TRUE); + found = TRUE; + + break; + } } g_list_free (children); - if (!found) + if (!found) { g_set_error (error, GTK_RECENT_CHOOSER_ERROR, GTK_RECENT_CHOOSER_ERROR_NOT_FOUND, _("No recently used resource found with URI `%s'"), uri); - return FALSE; - } - else - { - gtk_menu_shell_activate_item (GTK_MENU_SHELL (menu), menu_item, TRUE); - - return TRUE; } + + return found; } static gchar *