[Xfce4-commits] <midori:master> Remove pointless FIXME and TODO comments from Midori classes

Christian Dywan noreply at xfce.org
Mon Sep 13 01:42:02 CEST 2010


Updating branch refs/heads/master
         to d34da1c500ec7ab5241891d80a53e8777c3a7681 (commit)
       from db6e5954e8f25f5cef66bddbd8e1ee0c47dba778 (commit)

commit d34da1c500ec7ab5241891d80a53e8777c3a7681
Author: Christian Dywan <christian at twotoasts.de>
Date:   Sun Sep 12 20:23:39 2010 +0200

    Remove pointless FIXME and TODO comments from Midori classes

 midori/midori-app.c            |    5 -----
 midori/midori-browser.c        |    5 -----
 midori/midori-locationaction.c |    1 -
 midori/midori-preferences.c    |    1 -
 midori/midori-view.c           |    1 -
 5 files changed, 0 insertions(+), 13 deletions(-)

diff --git a/midori/midori-app.c b/midori/midori-app.c
index 5214897..9dddad6 100644
--- a/midori/midori-app.c
+++ b/midori/midori-app.c
@@ -777,23 +777,18 @@ midori_app_set_property (GObject*      object,
         break;
     case PROP_SETTINGS:
         katze_object_assign (app->settings, g_value_dup_object (value));
-        /* FIXME: Propagate settings to all browsers */
         break;
     case PROP_BOOKMARKS:
         katze_object_assign (app->bookmarks, g_value_dup_object (value));
-        /* FIXME: Propagate bookmarks to all browsers */
         break;
     case PROP_TRASH:
         katze_object_assign (app->trash, g_value_dup_object (value));
-        /* FIXME: Propagate trash to all browsers */
         break;
     case PROP_SEARCH_ENGINES:
         katze_object_assign (app->search_engines, g_value_dup_object (value));
-        /* FIXME: Propagate search engines to all browsers */
         break;
     case PROP_HISTORY:
         katze_object_assign (app->history, g_value_dup_object (value));
-        /* FIXME: Propagate history to all browsers */
         break;
     case PROP_EXTENSIONS:
         katze_object_assign (app->extensions, g_value_dup_object (value));
diff --git a/midori/midori-browser.c b/midori/midori-browser.c
index 0a17b7a..a2bf527 100644
--- a/midori/midori-browser.c
+++ b/midori/midori-browser.c
@@ -2131,7 +2131,6 @@ _action_private_browsing_activate (GtkAction*     action,
     const gchar* uri = midori_browser_get_current_uri (browser);
     if (uri != NULL)
     {
-        /* FIXME: Use the same binary that is running right now */
         if (*uri != '\0')
             midori_browser_spawn_app (uri);
         else
@@ -2748,7 +2747,6 @@ midori_browser_open_bookmark (MidoriBrowser* browser,
     if (!uri_fixed)
         uri_fixed = g_strdup (uri);
 
-    /* FIXME: Use the same binary that is running right now */
     if (katze_item_get_meta_boolean (item, "app"))
         midori_browser_spawn_app (uri_fixed);
     else
@@ -3225,7 +3223,6 @@ midori_browser_source_transfer_cb (KatzeNetRequest* request,
                 fclose (fp);
                 if ((ret - request->length) != 0)
                 {
-                    /* FIXME: Show an error in the graphical interface */
                     g_warning ("Error writing to file %s "
                                "in midori_browser_source_transfer_cb()", filename);
                 }
@@ -3279,7 +3276,6 @@ _action_source_view_activate (GtkAction*     action,
         #else
         GError* error = NULL;
 
-        /* FIXME: Handling http transparently in the function would be nice */
         if (g_str_has_prefix (uri, "file://"))
         {
             if (!sokoke_show_uri_with_mime_type (gtk_widget_get_screen (view),
@@ -5577,7 +5573,6 @@ midori_browser_init (MidoriBrowser* browser)
     error = NULL;
     if (!gtk_ui_manager_add_ui_from_string (ui_manager, ui_markup, -1, &error))
     {
-        /* TODO: Should this be a message dialog? When does this happen? */
         g_message ("User interface couldn't be created: %s", error->message);
         g_error_free (error);
     }
diff --git a/midori/midori-locationaction.c b/midori/midori-locationaction.c
index cecda90..152b172 100644
--- a/midori/midori-locationaction.c
+++ b/midori/midori-locationaction.c
@@ -545,7 +545,6 @@ midori_location_action_popup_completion (MidoriLocationAction* action,
         G_CALLBACK (gtk_widget_destroyed), &action->entry);
     action->completion_timeout = g_timeout_add (COMPLETION_DELAY,
         midori_location_action_popup_timeout_cb, action);
-    /* TODO: Inline completion */
 }
 
 static void
diff --git a/midori/midori-preferences.c b/midori/midori-preferences.c
index 1a73197..c24f60b 100644
--- a/midori/midori-preferences.c
+++ b/midori/midori-preferences.c
@@ -420,7 +420,6 @@ midori_preferences_set_settings (MidoriPreferences* preferences,
     #endif
     #if WEBKIT_CHECK_VERSION (1, 1, 6)
     FRAME_NEW (_("Spell Checking"));
-    /* FIXME: Provide a nice dictionary selection */
     button = katze_property_proxy (settings, "enable-spell-checking", NULL);
     gtk_button_set_label (GTK_BUTTON (button), _("Enable Spell Checking"));
     gtk_widget_set_tooltip_text (button, _("Enable spell checking while typing"));
diff --git a/midori/midori-view.c b/midori/midori-view.c
index 4ecd00d..918e6eb 100644
--- a/midori/midori-view.c
+++ b/midori/midori-view.c
@@ -1669,7 +1669,6 @@ static void
 midori_web_view_menu_web_app_activate_cb (GtkWidget*  widget,
                                           MidoriView* view)
 {
-    /* FIXME: Use the same binary that is running right now */
     sokoke_spawn_program ("midori -a", view->link_uri, FALSE);
 }
 



More information about the Xfce4-commits mailing list