[netstatus] Remove dead code, mark __attribute__((unused)) to indicate compilers,
authorJim Huang <jserv.tw@gmail.com>
Wed, 6 Aug 2008 03:47:08 +0000 (03:47 +0000)
committerJim Huang <jserv.tw@gmail.com>
Wed, 6 Aug 2008 03:47:08 +0000 (03:47 +0000)
confirming "-Wall -Wextra".

src/plugins/netstatus/glade-support.c
src/plugins/netstatus/glade-support.h
src/plugins/netstatus/netstatus-icon.c
src/plugins/netstatus/netstatus-iface.c
src/plugins/netstatus/netstatus-sysdeps.c
src/plugins/netstatus/netstatus.c

index 8f85c53..781c861 100644 (file)
@@ -52,6 +52,7 @@ add_pixmap_directory                   (const gchar     *directory)
                                         g_strdup (directory));
 }
 
+#if 0
 /* This is an internally used function to find pixmap files. */
 static gchar*
 find_pixmap_file                       (const gchar     *filename)
@@ -141,4 +142,5 @@ glade_set_atk_action_description       (AtkAction       *action,
         atk_action_set_description (action, i, description);
     }
 }
+#endif
 
index a32649e..aea013e 100644 (file)
@@ -55,6 +55,7 @@ void        add_pixmap_directory       (const gchar     *directory);
  * Private Functions.
  */
 
+#if 0
 /* This is used to create the pixmaps used in the interface. */
 GtkWidget*  create_pixmap              (GtkWidget       *widget,
                                         const gchar     *filename);
@@ -66,4 +67,4 @@ GdkPixbuf*  create_pixbuf              (const gchar     *filename);
 void        glade_set_atk_action_description (AtkAction       *action,
                                               const gchar     *action_name,
                                               const gchar     *description);
-
+#endif
index 81bf9fe..593f410 100644 (file)
@@ -93,7 +93,7 @@ static GObjectClass *parent_class;
 static guint icon_signals [LAST_SIGNAL] = { 0 };
 
 static GdkPixbuf *
-netstatus_icon_get_default_pixbuf (NetstatusIcon *icon)
+netstatus_icon_get_default_pixbuf (NetstatusIcon *icon __attribute__((unused)))
 {
   static GdkPixbuf *fallback_pixbuf = NULL;
 
@@ -114,7 +114,7 @@ netstatus_icon_get_default_pixbuf (NetstatusIcon *icon)
 
 static void
 netstatus_icon_theme_changed (NetstatusIcon *icon,
-                             GtkIconTheme  *icon_theme)
+                             GtkIconTheme  *icon_theme __attribute__((unused)))
 {
   int i;
 
@@ -365,8 +365,8 @@ netstatus_icon_update_image (NetstatusIcon *icon)
 }
 
 static void
-netstatus_icon_name_changed (NetstatusIface *iface,
-                            GParamSpec     *pspec,
+netstatus_icon_name_changed (NetstatusIface *iface __attribute__((unused)),
+                            GParamSpec     *pspec __attribute__((unused)),
                             NetstatusIcon  *icon)
 {
   const char *iface_name;
@@ -391,7 +391,7 @@ netstatus_icon_name_changed (NetstatusIface *iface,
 
 static void
 netstatus_icon_state_changed (NetstatusIface *iface,
-                             GParamSpec     *pspec,
+                             GParamSpec     *pspec __attribute__((unused)),
                              NetstatusIcon  *icon)
 {
   NetstatusState state;
@@ -410,7 +410,7 @@ netstatus_icon_state_changed (NetstatusIface *iface,
 
 static void
 netstatus_icon_is_wireless_changed (NetstatusIface *iface,
-                                   GParamSpec     *pspec,
+                                   GParamSpec     *pspec __attribute__((unused)),
                                    NetstatusIcon  *icon)
 {
   if (netstatus_iface_get_is_wireless (iface) && icon->priv->show_signal)
@@ -421,7 +421,7 @@ netstatus_icon_is_wireless_changed (NetstatusIface *iface,
 
 static void
 netstatus_icon_signal_changed (NetstatusIface *iface,
-                              GParamSpec     *pspec,
+                              GParamSpec     *pspec __attribute__((unused)),
                               NetstatusIcon  *icon)
 {
   NetstatusSignal signal_strength;
@@ -851,7 +851,7 @@ netstatus_icon_class_init (NetstatusIconClass *klass)
 
 static void
 netstatus_icon_instance_init (NetstatusIcon      *icon,
-                             NetstatusIconClass *klass)
+                             NetstatusIconClass *klass __attribute__((unused)))
 {
   icon->priv = g_new0 (NetstatusIconPrivate, 1);
 
index e89efc9..b0009c7 100644 (file)
@@ -125,7 +125,7 @@ netstatus_iface_get_type (void)
 
 static void
 netstatus_iface_instance_init (NetstatusIface      *iface,
-                              NetstatusIfaceClass *klass)
+                              NetstatusIfaceClass *klass __attribute__((unused)))
 {
   iface->priv = g_new0 (NetstatusIfacePrivate, 1);
   iface->priv->state = NETSTATUS_STATE_DISCONNECTED;
@@ -1021,7 +1021,7 @@ netstatus_iface_get_hw_details (NetstatusIface  *iface,
   static struct HwType *hw_type = NULL;
   struct ifreq          if_req;
   int                   fd;
-  int                   i;
+  unsigned int                   i;
 
   if (hw_addr)
     *hw_addr = NULL;
index a9b31c5..4321a0c 100644 (file)
@@ -223,7 +223,7 @@ netstatus_sysdeps_read_iface_statistics (const char  *iface,
       break;
     }
 
-  if ((*in_packets == -1 || *out_packets == -1 || *in_bytes == -1 || *out_bytes == -1) && !error_message)
+  if ((*in_packets == (gulong) -1 || *out_packets == (gulong) -1 || *in_bytes == (gulong) -1 || *out_bytes == (gulong) -1) && !error_message)
     error_message = g_strdup_printf ("Could not find information on interface '%s' in /proc/net/dev", iface);
 
   rewind (fh);
index 7a8674f..ca5c0ff 100644 (file)
@@ -147,7 +147,7 @@ netstatus_constructor(Plugin *p, char** fp)
     RET(0);
 }
 
-static void apply_config( Plugin* p )
+static void apply_config( Plugin* p __attribute__((unused)))
 {
 
 }