]> spindle.queued.net Git - midori/commitdiff
Implement 'full' debug-level and fix according warnings
authorNick Schermer <nick@xfce.org>
Fri, 9 Jan 2009 21:54:40 +0000 (22:54 +0100)
committerChristian Dywan <christian@twotoasts.de>
Fri, 9 Jan 2009 22:56:00 +0000 (23:56 +0100)
18 files changed:
katze/katze-net.c
katze/katze-throbber.c
katze/katze-utils.c
midori/gjs.c
midori/gtkiconentry.c
midori/main.c
midori/midori-app.c
midori/midori-browser.c
midori/midori-extension.c
midori/midori-locationaction.c
midori/midori-locationentry.c
midori/midori-panel.c
midori/midori-preferences.c
midori/midori-searchaction.c
midori/midori-view.c
midori/sokoke.c
panels/midori-addons.c
wscript

index ee45cfd8c41e94908119d363f55dbe75890beb3a..363ad9d15cb23c38fc9d9f953c1aa930ab924f76 100644 (file)
@@ -239,7 +239,7 @@ katze_net_finished_cb (SoupMessage*  msg,
 }
 #endif
 
-gboolean
+static gboolean
 katze_net_local_cb (KatzeNetPriv* priv)
 {
     KatzeNetRequest* request;
@@ -288,7 +288,7 @@ katze_net_local_cb (KatzeNetPriv* priv)
     return FALSE;
 }
 
-gboolean
+static gboolean
 katze_net_default_cb (KatzeNetPriv* priv)
 {
     KatzeNetRequest* request;
index 68b61bb1670986dbc3098b73ea773963fbf6970f..faaea53ab5fa74669f5214281d2ad2ae8ae34817 100644 (file)
@@ -885,17 +885,17 @@ katze_throbber_expose_event (GtkWidget*      widget,
 
         if (G_LIKELY (cols > 0 && rows > 0))
         {
-            gint index;
+            gint idx;
             guint x, y;
 
             katze_throbber_aligned_coords (widget, &ax, &ay);
 
-            index = throbber->index % (cols * rows);
+            idx = throbber->index % (cols * rows);
             if (G_LIKELY (throbber->timer_id >= 0))
-                index = MAX (index, 1);
+                idx = MAX (idx, 1);
 
-            x = (index % cols) * throbber->width;
-            y = (index / cols) * throbber->height;
+            x = (idx % cols) * throbber->width;
+            y = (idx / cols) * throbber->height;
 
             gdk_draw_pixbuf (event->window, NULL, throbber->pixbuf,
                              x, y, ax, ay,
index ff7a9a8b7ae90ed5d080d946e1ca43e6aabbfc3e..fa685d03cc83ac636e514a92455fd26a9e173b21 100644 (file)
@@ -335,7 +335,7 @@ katze_property_proxy (gpointer     object,
     }
     else if (type == G_TYPE_PARAM_ENUM)
     {
-        gint i;
+        guint i;
         GEnumClass* enum_class = G_ENUM_CLASS (
             g_type_class_ref (pspec->value_type));
         gint value = katze_object_get_enum (object, property);
index 057b9e392614e7520c883e4e56b5f62123371c41..372f3de37e4f4033c9dffc1407ab59bb1a0903b9 100644 (file)
@@ -198,7 +198,7 @@ gjs_value_get_string (GjsValue* value)
     return value->string;
 }
 
-void
+static void
 gjs_value_weak_notify_cb (GjsValue* attribute,
                           GjsValue* value)
 {
@@ -542,7 +542,7 @@ _js_class_get_property_names_cb (JSContextRef                 js_context,
         guint n;
         GParamSpec** pspecs = g_object_class_list_properties (
             G_OBJECT_GET_CLASS (object), &n);
-        gint i;
+        guint i;
         for (i = 0; i < n; i++)
         {
             const gchar* property = g_param_spec_get_name (pspecs[i]);
@@ -556,8 +556,8 @@ _js_class_get_property_names_cb (JSContextRef                 js_context,
             guint* signals = g_signal_list_ids (type, &n);
             for (i = 0; i < n; i++)
             {
-                const gchar* signal = g_signal_name (signals[i]);
-                JSStringRef js_signal = JSStringCreateWithUTF8CString (signal);
+                const gchar* signal_ = g_signal_name (signals[i]);
+                JSStringRef js_signal = JSStringCreateWithUTF8CString (signal_);
                 JSPropertyNameAccumulatorAddName (js_properties, js_signal);
                 JSStringRelease (js_signal);
             }
@@ -622,7 +622,7 @@ _js_object_call_as_function_cb (JSContextRef     js_context,
     GValue* values = g_new0 (GValue, n_arguments + 1);
     g_value_init (&values[0], G_OBJECT_TYPE (object));
     g_value_set_instance (&values[0], object);
-    gint i;
+    guint i;
     for (i = 0; i < n_arguments; i++)
     {
         GValue value = {0, };
@@ -1002,7 +1002,7 @@ _js_module_get_property_cb (JSContextRef js_context,
     return result;
 }
 
-JSObjectRef
+static JSObjectRef
 gjs_module_new (JSContextRef js_context,
                 const gchar* namespace)
 {
index 428fb216db9c5698a9f1a84beac9a15bc2467921..8e1899325836e32aaa2c9676d1bc030972c248fc 100644 (file)
@@ -97,9 +97,7 @@ enum
   PROP_SENSITIVITY_SECONDARY
 };
 
-static void gtk_icon_entry_class_init        (GtkIconEntryClass    *klass);
 static void gtk_icon_entry_editable_init     (GtkEditableClass     *iface);
-static void gtk_icon_entry_init              (GtkIconEntry         *entry);
 static void gtk_icon_entry_finalize          (GObject              *obj);
 static void gtk_icon_entry_dispose           (GObject              *obj);
 static void gtk_icon_entry_map               (GtkWidget            *widget);
index 9614245ad0fda08b2c3459410fa2a63512eca328..17e340ca98f91131d3bb35c240c69f14740fd0ad 100644 (file)
@@ -65,11 +65,11 @@ stock_items_init (void)
 {
     typedef struct
     {
-        gchar* stock_id;
-        gchar* label;
+        const gchar* stock_id;
+        const gchar* label;
         GdkModifierType modifier;
         guint keyval;
-        gchar* fallback;
+        const gchar* fallback;
     } FatStockItem;
     GtkIconSource* icon_source;
     GtkIconSet* icon_set;
@@ -143,7 +143,7 @@ settings_new_from_file (const gchar* filename)
     GParamSpec* pspec;
     GType type;
     const gchar* property;
-    gchar* string;
+    gchar* str;
     gint integer;
     gfloat number;
     gboolean boolean;
@@ -167,11 +167,11 @@ settings_new_from_file (const gchar* filename)
         property = g_param_spec_get_name (pspec);
         if (type == G_TYPE_PARAM_STRING)
         {
-            string = sokoke_key_file_get_string_default (key_file,
+            str = sokoke_key_file_get_string_default (key_file,
                 "settings", property,
                 G_PARAM_SPEC_STRING (pspec)->default_value, NULL);
-            g_object_set (settings, property, string, NULL);
-            g_free (string);
+            g_object_set (settings, property, str, NULL);
+            g_free (str);
         }
         else if (type == G_TYPE_PARAM_INT)
         {
@@ -200,17 +200,17 @@ settings_new_from_file (const gchar* filename)
                 g_type_class_ref (pspec->value_type));
             GEnumValue* enum_value = g_enum_get_value (enum_class,
                 G_PARAM_SPEC_ENUM (pspec)->default_value);
-            gchar* string = sokoke_key_file_get_string_default (key_file,
+            str = sokoke_key_file_get_string_default (key_file,
                 "settings", property,
                 enum_value->value_name, NULL);
-            enum_value = g_enum_get_value_by_name (enum_class, string);
+            enum_value = g_enum_get_value_by_name (enum_class, str);
             if (enum_value)
                  g_object_set (settings, property, enum_value->value, NULL);
              else
                  g_warning (_("Value '%s' is invalid for %s"),
-                            string, property);
+                            str, property);
 
-            g_free (string);
+            g_free (str);
             g_type_class_unref (enum_class);
         }
         else
@@ -243,9 +243,9 @@ settings_save_to_file (MidoriWebSettings* settings,
         property = g_param_spec_get_name (pspec);
         if (!(pspec->flags & G_PARAM_WRITABLE))
         {
-            gchar* comment = g_strdup_printf ("# %s", property);
-            g_key_file_set_string (key_file, "settings", comment, "");
-            g_free (comment);
+            gchar* prop_comment = g_strdup_printf ("# %s", property);
+            g_key_file_set_string (key_file, "settings", prop_comment, "");
+            g_free (prop_comment);
             continue;
         }
         if (type == G_TYPE_PARAM_STRING)
@@ -747,7 +747,7 @@ midori_history_add_items (void*  data,
     /* Test whether have the right number of columns */
     g_return_val_if_fail (argc % ncols == 0, 1);
 
-    for (i = 0; i <= (argc - ncols); i++)
+    for (i = 0; i < (argc - ncols) + 1; i++)
     {
         if (argv[i])
         {
@@ -2141,11 +2141,11 @@ main (int    argc,
         G_CALLBACK (midori_bookmarks_remove_item_cb), NULL);
     if (!katze_array_is_empty (bookmarks))
     {
-        guint i, n;
+        guint n;
         n = katze_array_get_length (bookmarks);
         for (i = 0; i < n; i++)
         {
-            KatzeItem* item = katze_array_get_nth_item (bookmarks, i);
+            item = katze_array_get_nth_item (bookmarks, i);
             if (KATZE_IS_ARRAY (item))
             {
                 g_signal_connect_after (item, "add-item",
index 11a9f895f058b3e1313aad1e7b5d59ab4fa56bd7..8ce5d205e18a2e74e4e5c6ae620b151b9b51097d 100644 (file)
@@ -141,7 +141,7 @@ midori_browser_quit_cb (MidoriBrowser* browser,
     midori_app_quit (app);
 }
 
-void
+static void
 _midori_app_add_browser (MidoriApp*     app,
                          MidoriBrowser* browser)
 {
@@ -164,7 +164,7 @@ _midori_app_add_browser (MidoriApp*     app,
     #endif
 }
 
-void
+static void
 _midori_app_quit (MidoriApp* app)
 {
     gtk_main_quit ();
@@ -282,7 +282,7 @@ static UniqueResponse
 midori_browser_message_received_cb (UniqueApp*         instance,
                                     UniqueCommand      command,
                                     UniqueMessageData* message,
-                                    guint              time,
+                                    guint              timestamp,
                                     MidoriApp*         app)
 {
   UniqueResponse response;
index d4fb7ea49d7c7d807c9ce1daa2be3fbd3b8a5a3b..5cfe5f55391bf328659e942ba302be436edc79e7 100644 (file)
@@ -1990,12 +1990,12 @@ _action_location_activate (GtkAction*     action,
 
 static void
 _action_location_active_changed (GtkAction*     action,
-                                 gint           index,
+                                 gint           idx,
                                  MidoriBrowser* browser)
 {
     const gchar* uri;
 
-    if (index > -1)
+    if (idx > -1)
     {
         uri = midori_location_action_get_uri (MIDORI_LOCATION_ACTION (action));
         midori_browser_set_current_uri (browser, uri);
@@ -2163,16 +2163,16 @@ _action_search_notify_current_item (GtkAction*     action,
 {
     MidoriSearchAction* search_action;
     KatzeItem* item;
-    guint index;
+    guint idx;
 
     search_action = MIDORI_SEARCH_ACTION (action);
     item = midori_search_action_get_current_item (search_action);
     if (item)
-        index = katze_array_get_item_index (browser->search_engines, item);
+        idx = katze_array_get_item_index (browser->search_engines, item);
     else
-        index = 0;
+        idx = 0;
 
-    g_object_set (browser->settings, "last-web-search", index, NULL);
+    g_object_set (browser->settings, "last-web-search", idx, NULL);
 }
 
 static void
@@ -2827,23 +2827,23 @@ static const gchar* license =
 
 static void
 _action_about_activate_link (GtkAboutDialog* about,
-                             const gchar*    link,
+                             const gchar*    uri,
                              gpointer        user_data)
 {
     MidoriBrowser* browser;
     gint n;
 
     browser = MIDORI_BROWSER (user_data);
-    n = midori_browser_add_uri (browser, link);
+    n = midori_browser_add_uri (browser, uri);
     midori_browser_set_current_page (browser, n);
 }
 
 static void
 _action_about_activate_email (GtkAboutDialog* about,
-                              const gchar*    link,
+                              const gchar*    uri,
                               gpointer        user_data)
 {
-    gchar* command = g_strconcat ("xdg-open ", link, NULL);
+    gchar* command = g_strconcat ("xdg-open ", uri, NULL);
     g_spawn_command_line_async (command, NULL);
     g_free (command);
 }
@@ -4070,7 +4070,7 @@ midori_browser_init (MidoriBrowser* browser)
     gtk_tool_button_set_label (GTK_TOOL_BUTTON (toolitem), _("Close Findbar"));
     g_signal_connect (toolitem, "clicked",
         G_CALLBACK (midori_browser_find_button_close_clicked_cb), browser);
-    #ifdef HAVE_OSX
+    #if HAVE_OSX
     gtk_toolbar_insert (GTK_TOOLBAR (browser->find), toolitem, 0);
     #else
     gtk_toolbar_insert (GTK_TOOLBAR (browser->find), toolitem, -1);
index 04228bd1e05ff31fb76ed2c9eaeb9cb9b3759be6..dc54075466100c54ce9997876596e377a2ba4817 100644 (file)
@@ -140,7 +140,7 @@ midori_extension_class_init (MidoriExtensionClass* class)
     g_type_class_add_private (class, sizeof (MidoriExtensionPrivate));
 }
 
-void
+static void
 midori_extension_activate_cb (MidoriExtension* extension,
                               MidoriApp*       app)
 {
index 7fcf78678205f92b1e7d0825b59398b341bb9fc3..631cfc74298a8d027c674b0ecef6a78b8f173915 100644 (file)
@@ -894,7 +894,7 @@ midori_location_action_prepend_item (MidoriLocationAction*    location_action,
     GtkTreeModel* filter_model;
     GtkTreeModel* model;
     GtkTreeIter iter;
-    GtkTreeIter index;
+    GtkTreeIter idx;
     gint n;
     gint visits = 0;
 
@@ -904,17 +904,17 @@ midori_location_action_prepend_item (MidoriLocationAction*    location_action,
     if (midori_location_action_iter_insert (location_action,
         item->uri, &iter, 0))
     {
-        gtk_tree_model_get_iter_first (model, &index);
+        gtk_tree_model_get_iter_first (model, &idx);
         gtk_tree_model_get (model, &iter, VISITS_COL, &visits, -1);
-        gtk_list_store_move_before (GTK_LIST_STORE (model), &iter, &index);
+        gtk_list_store_move_before (GTK_LIST_STORE (model), &iter, &idx);
     }
 
     n = gtk_tree_model_iter_n_children (filter_model, NULL);
     if (n > MAX_ITEMS)
     {
-        gtk_tree_model_iter_nth_child (model, &index, NULL, n - 1);
+        gtk_tree_model_iter_nth_child (model, &idx, NULL, n - 1);
         gtk_list_store_set (GTK_LIST_STORE (model),
-                            &index, VISIBLE_COL, FALSE, -1);
+                            &idx, VISIBLE_COL, FALSE, -1);
     }
 
     /* Only increment the visits when we add the uri */
index eceb4b5669b6f0d1b56c6090b31384a5b5d11ea0..92a342776c1deca33f427d23f7c0ced8f79c2907 100644 (file)
@@ -134,7 +134,7 @@ _gtk_entry_effective_inner_border (GtkEntry  *entry,
   *border = default_inner_border;
 }
 
-void
+static void
 gtk_entry_borders (GtkEntry* entry,
                    gint*     xborder,
                    gint*     yborder,
index 4129d825ebaab1507af67bdbb43b1a970c1130e7..160266b900271dd2a7b9b7991b8cbc39eabe0198 100644 (file)
@@ -595,6 +595,7 @@ typedef struct
     GtkAlignmentClass parent_class;
 }  MidoriDummyViewableClass;
 
+GType midori_dummy_viewable_get_type (void) G_GNUC_CONST;
 #define MIDORI_TYPE_DUMMY_VIEWABLE (midori_dummy_viewable_get_type ())
 #define MIDORI_DUMMY_VIEWABLE(obj) (G_TYPE_CHECK_INSTANCE_CAST ((obj), \
     MIDORI_TYPE_DUMMY_VIEWABLE, MidoriDummyViewable))
index 0d1b4fe66668a61b642c17485f888f3709964520..8ac296edd096671b77c7597e4a0c128e7e60b3fb 100644 (file)
@@ -235,7 +235,7 @@ midori_preferences_notify_identify_as_cb (MidoriWebSettings* settings,
 }
 #endif
 
-#ifdef HAVE_OSX
+#if HAVE_OSX
 static void
 midori_preferences_help_clicked_cb (GtkWidget* button,
                                     GtkDialog* dialog)
@@ -260,7 +260,7 @@ midori_preferences_add_toolbutton (GtkWidget*   toolbar,
                                    const gchar* label,
                                    GtkWidget*   page)
 {
-#ifdef HAVE_OSX
+#if HAVE_OSX
     *toolbutton = GTK_WIDGET (*toolbutton ? gtk_radio_tool_button_new_from_widget (
         GTK_RADIO_TOOL_BUTTON (*toolbutton)) : gtk_radio_tool_button_new (NULL));
     gtk_tool_button_set_label (GTK_TOOL_BUTTON (*toolbutton), label);
@@ -316,7 +316,7 @@ midori_preferences_set_settings (MidoriPreferences* preferences,
     preferences->notebook = gtk_notebook_new ();
     gtk_container_set_border_width (GTK_CONTAINER (preferences->notebook), 6);
 
-    #ifdef HAVE_OSX
+    #if HAVE_OSX
     gtk_notebook_set_show_tabs (GTK_NOTEBOOK (preferences->notebook), FALSE);
     gtk_notebook_set_show_border (GTK_NOTEBOOK (preferences->notebook), FALSE);
     toolbar = gtk_toolbar_new ();
@@ -607,7 +607,7 @@ midori_preferences_set_settings (MidoriPreferences* preferences,
     g_object_unref (sizegroup);
     gtk_box_pack_start (GTK_BOX (GTK_DIALOG (preferences)->vbox),
                         preferences->notebook, FALSE, FALSE, 4);
-    #ifdef HAVE_OSX
+    #if HAVE_OSX
     GtkWidget* icon;
     hbox = gtk_hbox_new (FALSE, 0);
     button = gtk_button_new ();
index 6a9d21c497682321a35d30a3c6a59d8d85e67f1d..d9852d4f205d9c3980eeb4820b301f40fb147834 100644 (file)
@@ -1144,7 +1144,7 @@ midori_search_action_get_dialog (MidoriSearchAction* search_action)
     gtk_widget_set_sensitive (button, FALSE);
     gtk_box_pack_end (GTK_BOX (vbox), button, FALSE, FALSE, 0);
 
-    #ifdef HAVE_OSX
+    #if HAVE_OSX
     GtkWidget* icon;
     hbox = gtk_hbox_new (FALSE, 0);
     button = gtk_button_new ();
index bd114f782ae069e0776dab34e3dac178a73e36f1..b85c79c92fcd4d4c79a20cb5d6e32e17310e7a3f 100644 (file)
@@ -692,7 +692,7 @@ gtk_widget_button_press_event_cb (WebKitWebView*  web_view,
     case 1:
         if (!link_uri)
             return FALSE;
-        #ifdef HAVE_OSX
+        #if HAVE_OSX
         /* FIXME: Test for Command key */
         if (0)
         #else
@@ -1891,8 +1891,9 @@ midori_view_tab_label_parent_set (GtkWidget*  tab_label,
 {
     GtkWidget* parent;
 
+    /* FIXME: Disconnect orientation notification
     if (old_parent)
-        /* FIXME: Disconnect orientation notification */;
+        ; */
 
     if (!(parent = gtk_widget_get_parent (tab_label)))
         return;
index 67b7d46541443360443f980372bcc5e802585bd4..84903b3a5b5da4f9385f4167505d8be4f9bfe1d6 100644 (file)
@@ -251,7 +251,7 @@ typedef enum
 static SokokeDesktop
 sokoke_get_desktop (void)
 {
-    #ifdef HAVE_OSX
+    #if HAVE_OSX
     return SOKOKE_DESKTOP_OSX;
     #else
     static SokokeDesktop desktop = SOKOKE_DESKTOP_UNTESTED;
index 8cc55172be60e059eedbb73a9a07a059ec5582df..67823f6cc63f5a02eb2a238c5386f9b25170ce87 100644 (file)
@@ -308,7 +308,7 @@ _addons_get_files (MidoriAddons* addons)
     return files;
 }
 
-GtkTreePath*
+static GtkTreePath*
 _treeview_first_selected_path (GtkTreeView *treeview)
 {
     GtkTreeSelection* selection;
diff --git a/wscript b/wscript
index a1565052aa4d824702a7d0b076876591a8336c7c..3469c4b9a7d67344e3b5a5a423058ff76bbbe8d0 100644 (file)
--- a/wscript
+++ b/wscript
@@ -141,8 +141,7 @@ def configure (conf):
     check_pkg ('libxml-2.0', '2.6')
 
     conf.check (header_name='unistd.h')
-    if sys.platform == 'darwin':
-        conf.define ('HAVE_OSX', 1)
+    conf.define ('HAVE_OSX', int(sys.platform == 'darwin'))
 
     if conf.find_program ('rsvg-convert', var='RSVG_CONVERT'):
         icons = 'yes'
@@ -162,11 +161,26 @@ def configure (conf):
     conf.write_config_header ('config.h')
     conf.env.append_value ('CCFLAGS', '-DHAVE_CONFIG_H')
     debug_level = Options.options.debug_level
-    if debug_level != '':
+    if debug_level != 'no':
         compiler = conf.env['CC_NAME']
         if compiler == 'gcc':
             if debug_level == 'debug':
                 conf.env.append_value ('CCFLAGS', '-Wall -O0 -g')
+            elif debug_level == 'full':
+                # -Wdeclaration-after-statement
+                # -Wmissing-declarations -Wmissing-prototypes
+                # -Wwrite-strings
+                conf.env.append_value ('CCFLAGS',
+                    '-Wall -Wextra -O1 -g '
+                    '-Waggregate-return -Wno-unused-parameter '
+                    '-Wno-missing-field-initializers '
+                    '-Wunsafe-loop-optimizations '
+                    '-Wredundant-decls -Wmissing-noreturn '
+                    '-Wshadow -Wpointer-arith -Wcast-align '
+                    '-Winline -Wformat-security '
+                    '-Winit-self -Wmissing-include-dirs -Wundef '
+                    '-Wmissing-format-attribute -Wnested-externs '
+                    '-DG_ENABLE_DEBUG')
             else:
                 conf.env.append_value ('CCFLAGS', '-O2')
         else:
@@ -207,9 +221,9 @@ def set_options (opt):
 
     opt.tool_options ('compiler_cc')
     opt.get_option_group ('--check-c-compiler').add_option('-d', '--debug-level',
-        action = 'store', default = '',
-        help = 'Specify the debugging level. [\'debug\', \'release\']',
-        choices = ['', 'debug', 'release'], dest = 'debug_level')
+        action = 'store', default = 'debug',
+        help = 'Specify the debugging level. [\'no\', \'debug\', \'full\']',
+        choices = ['no', 'debug', 'full'], dest = 'debug_level')
     opt.tool_options ('gnu_dirs')
     opt.parser.remove_option ('--oldincludedir')
     opt.parser.remove_option ('--htmldir')