[Xfce4-commits] <midori:master> Avoid superfluous _midori_browser_update_actions calls

Christian Dywan noreply at xfce.org
Thu Dec 1 20:06:04 CET 2011


Updating branch refs/heads/master
         to 5e784e9bcb23c22243a7c9cab05f30784c39dbc0 (commit)
       from e2a2f96cf80855d9226a0ff98f972b7e27fc6c7f (commit)

commit 5e784e9bcb23c22243a7c9cab05f30784c39dbc0
Author: Christian Dywan <christian at twotoasts.de>
Date:   Thu Dec 1 20:03:41 2011 +0100

    Avoid superfluous _midori_browser_update_actions calls

 midori/midori-browser.c |   17 +++++++----------
 1 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/midori/midori-browser.c b/midori/midori-browser.c
index b91a30d..8c40cb5 100644
--- a/midori/midori-browser.c
+++ b/midori/midori-browser.c
@@ -1468,7 +1468,6 @@ midori_browser_tab_destroy_cb (GtkWidget*     widget,
         if (browser->trash && !midori_view_is_blank (MIDORI_VIEW (widget)))
             katze_array_add_item (browser->trash, item);
         katze_array_remove_item (browser->proxy_array, item);
-        g_object_unref (item);
     }
 
     _midori_browser_update_actions (browser);
@@ -1536,7 +1535,6 @@ _midori_browser_add_tab (MidoriBrowser* browser,
     gtk_widget_set_can_focus (view, TRUE);
     tab_label = midori_view_get_proxy_tab_label (MIDORI_VIEW (view));
     item = midori_view_get_proxy_item (MIDORI_VIEW (view));
-    g_object_ref (item);
     katze_array_add_item (browser->proxy_array, item);
 
     g_object_connect (view,
@@ -2845,17 +2843,17 @@ _action_trash_activate_item_alt (GtkAction*     action,
 {
     if (MIDORI_EVENT_NEW_TAB (gtk_get_current_event ()))
     {
-        gint n = midori_browser_add_item (browser, item);
-        midori_browser_set_current_page_smartly (browser, n);
+        guint n;
         katze_array_remove_item (browser->trash, item);
-        _midori_browser_update_actions (browser);
+        n = midori_browser_add_item (browser, item);
+        midori_browser_set_current_page_smartly (browser, n);
     }
     else if (button == 1)
     {
-        guint n = midori_browser_add_item (browser, item);
-        midori_browser_set_current_page (browser, n);
+        guint n;
         katze_array_remove_item (browser->trash, item);
-        _midori_browser_update_actions (browser);
+        n = midori_browser_add_item (browser, item);
+        midori_browser_set_current_page (browser, n);
     }
 
     return TRUE;
@@ -5141,10 +5139,9 @@ _action_undo_tab_close_activate (GtkAction*     action,
     /* Reopen the most recent trash item */
     last = katze_array_get_length (browser->trash) - 1;
     item = katze_array_get_nth_item (browser->trash, last);
+    katze_array_remove_item (browser->trash, item);
     n = midori_browser_add_item (browser, item);
     midori_browser_set_current_page (browser, n);
-    katze_array_remove_item (browser->trash, item);
-    _midori_browser_update_actions (browser);
 }
 
 static void


More information about the Xfce4-commits mailing list