1. Replace hardcoded "-rdynamic" with proper "-export-dynamic" in
authorJim Huang <jserv.tw@gmail.com>
Mon, 9 Oct 2006 16:04:46 +0000 (16:04 +0000)
committerJim Huang <jserv.tw@gmail.com>
Mon, 9 Oct 2006 16:04:46 +0000 (16:04 +0000)
lxpanel_LDFLAGS. (Bug #1573603)

2. Fixes to various compilation warnings (make CFLAGS="-Wall -Werror").

12 files changed:
src/Makefile.am
src/gtk-run.c
src/lxpanelctl.c
src/misc.c
src/plugins/menu.c
src/plugins/netstatus/netstatus-dialog.c
src/plugins/netstatus/netstatus-icon.c
src/plugins/netstatus/netstatus-iface.c
src/plugins/netstatus/netstatus.c
src/plugins/ptk-app-menu.c
src/plugins/ptk-app-menu.h [new file with mode: 0644]
src/plugins/volume/volume.c

index cce1539..481b183 100644 (file)
@@ -16,7 +16,7 @@ BUILTIN_PLUGINS = $(top_builddir)/src/plugins/libbuiltin_plugins.a
 TRAY_SOURCES= \
        systray/eggmarshalers.c systray/eggmarshalers.h \
        systray/eggtraymanager.c systray/eggtraymanager.h \
-       systray/fixedtip.c systray/fixedtip.h\
+       systray/fixedtip.c systray/fixedtip.h \
        systray/tray.c
 
 lxpanel_SOURCES = \
@@ -33,7 +33,7 @@ lxpanel_SOURCES = \
        gtk-run.c
 
 if BUILD_PLUGIN_LOADER
-DYNAMIC_FLAGS = -rdynamic
+DYNAMIC_FLAGS = -export-dynamic
 endif
 lxpanel_LDFLAGS = $(DYNAMIC_FLAGS)
 
index c611db7..096c39b 100644 (file)
@@ -44,10 +44,9 @@ static gboolean setup_auto_complete( gpointer entry )
     {
         GDir *dir = g_dir_open( *dirname, 0, NULL );
         const char *name;
-        int i = 0;
         if( ! dir )
             continue;
-        while( name = g_dir_read_name( dir ) )
+        while( ( name = g_dir_read_name( dir ) ) )
         {
             char* filename = g_build_filename( *dirname, name, NULL );
             if( g_file_test( filename, G_FILE_TEST_IS_EXECUTABLE ) )
@@ -134,6 +133,6 @@ void gtk_run()
     gtk_widget_show_all( win );
     /* g_timeout_add( 500, setup_auto_complete, entry ); */
     setup_auto_complete( entry );
-    gtk_widget_show( (GtkDialog*)win );
+    gtk_widget_show( win );
 }
 
index 39de2ed..f5788b8 100644 (file)
@@ -4,10 +4,14 @@
 #include <X11/Xlib.h>
 #include <X11/Xutil.h>
 #include <X11/Xatom.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
 
 static Display* dpy;
 
-const char usage[] = "\nlxpanelctl - LXPanel Controller\n"
+static const char usage[] = 
+        "\nlxpanelctl - LXPanel Controller\n"
         "Usage: lxpanelctl <command>\n\n"
         "Available commands:\n"
         "menu\tshow system menu\n"
@@ -16,7 +20,7 @@ const char usage[] = "\nlxpanelctl - LXPanel Controller\n"
         "restart\trestart lxpanel\n"
         "exit\texit lxpanel\n\n";
 
-int get_cmd( const char* cmd )
+static int get_cmd( const char* cmd )
 {
     if( ! strcmp( cmd, "menu") )
         return LXPANEL_CMD_SYS_MENU;
index 4f80a61..dbc4b3d 100644 (file)
@@ -782,8 +782,8 @@ gtk_image_new_from_file_scaled(const gchar *file, gint width,
       gint height, gboolean keep_ratio)
 {
     GtkWidget *img;
-    GdkPixbuf *pb, *pb_scaled;
-    gfloat w, h, rw, rh;
+    GdkPixbuf /*- *pb, -*/ *pb_scaled;
+    // gfloat w, h, rw, rh;
     GtkIconInfo *inf = NULL;
 
     ENTER;
index b10425d..78baaf6 100644 (file)
@@ -11,6 +11,8 @@
 #include "bg.h"
 #include "gtkbgbox.h"
 
+#include "ptk-app-menu.h"
+
 //#define DEBUG
 #include "dbg.h"
 
@@ -123,7 +125,7 @@ reload_system_menu( GtkMenu* menu )
             if( ! child )
                 break;
         }
-        else if( sub_menu = gtk_menu_item_get_submenu( item ) ) {
+        else if( ( sub_menu = gtk_menu_item_get_submenu( item ) ) ) {
             reload_system_menu( GTK_MENU(sub_menu) );
         }
     }
@@ -134,9 +136,9 @@ static void show_menu( GtkWidget* widget, menup* m, int btn, guint32 time )
 {
     /* reload system menu items if needed */
     if( m->has_system_menu && ptk_app_menu_need_reload() ) {
-        reload_system_menu( m->menu );
+        reload_system_menu( GTK_MENU(m->menu) );
     }
-    gtk_menu_popup(m->menu,
+    gtk_menu_popup(GTK_MENU(m->menu),
                    NULL, NULL,
                    (GtkMenuPositionFunc)menu_pos, widget,
                    btn, time);
@@ -354,7 +356,7 @@ read_submenu(plugin *p, gboolean as_item)
             } else if (!g_ascii_strcasecmp(s.t[0], "separator")) {
                 mi = read_separator(p);
             } else if (!g_ascii_strcasecmp(s.t[0], "system")) {
-                read_system_menu(menu, p); /* add system menu items */
+                read_system_menu(GTK_MENU(menu), p); /* add system menu items */
                 continue;
             } else if (!g_ascii_strcasecmp(s.t[0], "menu")) {
                 mi = read_submenu(p, TRUE);
index 07e0295..c90a5a3 100644 (file)
 #define NETWORK_CONFIG_TOOL_DIR "/apps/netstatus_applet"
 #define NETWORK_CONFIG_TOOL_KEY NETWORK_CONFIG_TOOL_DIR "/config_tool"
 
+#if 0 /* stripped-down version does nothing to configurators. */
 static const char *network_config_tools[] = {
   "network-admin --configure %i",
   "redhat-config-network",
   "system-control-network"
 };
+#endif
 
 #define UNKNOWN_STR(t,s) G_STMT_START {       \
        if (!((t) = (s))) (t) = _("Unknown"); \
@@ -498,10 +500,10 @@ netstatus_dialog_config_tool_notify (GConfClient         *client,
 static void
 netstatus_dialog_detect_configuration_tool (NetstatusDialogData *dialog_data)
 {
+  /*
   char *config_tool;
   int   i;
 
-  /*
   dialog_data->client = gconf_client_get_default ();
 
   gconf_client_add_dir (dialog_data->client,
index c9a75d6..87f4e83 100644 (file)
@@ -915,7 +915,7 @@ netstatus_icon_new (NetstatusIface *iface)
     GtkWidget* obj = g_object_new (NETSTATUS_TYPE_ICON,
                              /*"iface", iface,*/
                              NULL);
-    netstatus_icon_set_iface( obj, iface );
+    netstatus_icon_set_iface( NETSTATUS_ICON(obj), iface );
     return obj;
 }
 
index 339d635..1f5abee 100644 (file)
@@ -831,7 +831,7 @@ print_rose_addr (guchar *p)
 static char *
 print_x25_addr (guchar *p)
 {
-  return g_strdup (p);
+  return g_strdup ((const gchar *) p);
 }
 
 static char *
@@ -1070,7 +1070,7 @@ netstatus_iface_get_hw_details (NetstatusIface  *iface,
     }
 
   if (hw_addr && if_req.ifr_hwaddr.sa_data && hw_type->print_hw_addr)
-    *hw_addr = hw_type->print_hw_addr (if_req.ifr_hwaddr.sa_data);
+    *hw_addr = hw_type->print_hw_addr ((guchar *) if_req.ifr_hwaddr.sa_data);
 
   return hw_type;
 
index 9abccdd..bca80bb 100644 (file)
@@ -40,7 +40,7 @@ static void on_response( GtkDialog* dlg, gint response, netstatus *ns )
         case GTK_RESPONSE_CLOSE:
         case GTK_RESPONSE_DELETE_EVENT:
         case GTK_RESPONSE_NONE:
-            gtk_widget_destroy( dlg );
+            gtk_widget_destroy( GTK_WIDGET(dlg) );
             ns->dlg = NULL;
     }
 }
@@ -57,7 +57,7 @@ static void on_button_press( GtkWidget* widget, GdkEventButton* evt, plugin* p )
             iface = netstatus_icon_get_iface( NETSTATUS_ICON(widget) );
             ns->dlg = netstatus_dialog_new(iface);
             netstatus_dialog_set_configuration_tool( ns->dlg, ns->config_tool );
-            g_signal_connect( ns->dlg, "response", on_response, ns );
+            g_signal_connect( ns->dlg, "response", G_CALLBACK(on_response), ns );
         }
         gtk_window_present( GTK_WINDOW(ns->dlg) );
     }
@@ -68,9 +68,7 @@ netstatus_constructor(plugin *p)
 {
     netstatus *ns;
     line s;
-    int w, h;
     NetstatusIface* iface;
-    GtkWidget* icon;
 
     ENTER;
     s.len = 256;  
index 0ab498e..558675c 100644 (file)
@@ -15,6 +15,7 @@
 #include <stdio.h>
 #include <sys/stat.h>
 #include <string.h>
+#include "ptk-app-menu.h"
 
 /* Compatibility macros for older versions of glib */
 #if ! GLIB_CHECK_VERSION(2, 10, 0)
@@ -42,7 +43,7 @@ typedef struct _CatInfo
 {
     char* title;
     char* icon;
-    char** sub_cats;
+    const char** sub_cats;
 }CatInfo;
 
 typedef struct _PtkAppMenuItem
@@ -164,7 +165,7 @@ int find_cat( char** cats )
         /* Skip other */
         for( i = 1; i < G_N_ELEMENTS(known_cats); ++i )
         {
-            char** sub_cats = known_cats[i].sub_cats;
+            const char** sub_cats = known_cats[i].sub_cats;
             while( *sub_cats )
             {
                 if( 0 == strncmp(*cat, "X-", 2) ) /*  Desktop specific*/
@@ -474,7 +475,7 @@ void on_app_menu_destroy( gpointer user_data, GObject* menu )
 
 gboolean ptk_app_menu_item_has_data( GtkMenuItem* item )
 {
-   return (g_object_get_qdata( item, data_id ) != NULL);
+   return (g_object_get_qdata( G_OBJECT(item), data_id ) != NULL);
 }
 
 /*
@@ -493,39 +494,38 @@ void ptk_app_menu_insert_items( GtkMenu* menu, int position )
 
    if( ! data_id )
       data_id = g_quark_from_static_string("PtkAppMenuItem");
-   app_dirs_foreach( load_dir, sub_menus );
+   app_dirs_foreach( (GFunc) load_dir, sub_menus );
    for( i = 0; i < G_N_ELEMENTS(known_cats); ++i )
    {
       GtkMenu* sub_menu;
       GtkWidget* menu_item;
-      GtkWidget* img;
       PtkAppMenuItem* data;
       if( ! (sub_items = sub_menus[i]) )
          continue;
       sub_menu = gtk_menu_new();
 
       for( l = sub_items; l; l = l->next )
-         gtk_menu_shell_append( sub_menu, GTK_WIDGET(l->data) );
+         gtk_menu_shell_append( GTK_MENU_SHELL(sub_menu), GTK_WIDGET(l->data) );
       g_list_free( sub_items );
       menu_item = gtk_image_menu_item_new_with_label( _(known_cats[i].title) );
       data = g_slice_new0( PtkAppMenuItem );
       data->icon = g_strdup(known_cats[i].icon);
-      g_object_set_qdata_full( menu_item, data_id, data, ptk_app_menu_item_free );
-      g_signal_connect( menu_item, "expose-event", on_menu_item_expose, data );
-      g_signal_connect( menu_item, "size-request", on_menu_item_size_request, data );
+      g_object_set_qdata_full( G_OBJECT(menu_item), data_id, data, (GDestroyNotify) ptk_app_menu_item_free );
+      g_signal_connect( menu_item, "expose-event", G_CALLBACK(on_menu_item_expose), data );
+      g_signal_connect( menu_item, "size-request", G_CALLBACK(on_menu_item_size_request), data );
       on_menu_item_expose( menu_item, NULL, data );
-      gtk_menu_item_set_submenu( menu_item, sub_menu );
+      gtk_menu_item_set_submenu( GTK_MENU_ITEM(menu_item), GTK_WIDGET(sub_menu) );
       if( position == -1 )
-         gtk_menu_shell_append( menu, menu_item );
+         gtk_menu_shell_append( GTK_MENU_SHELL(menu), menu_item );
       else
       {
-         gtk_menu_shell_insert( menu, menu_item, position );
+         gtk_menu_shell_insert( GTK_MENU_SHELL(menu), menu_item, position );
          ++position;
       }
    }
-   gtk_widget_show_all(menu);
-   change_handler = g_signal_connect_swapped( gtk_icon_theme_get_default(), "changed", unload_old_icons, menu );
-   g_object_weak_ref( menu, on_app_menu_destroy, GINT_TO_POINTER(change_handler) );
+   gtk_widget_show_all(GTK_WIDGET(menu));
+   change_handler = g_signal_connect_swapped( gtk_icon_theme_get_default(), "changed", G_CALLBACK(unload_old_icons), menu );
+   g_object_weak_ref( G_OBJECT(menu), on_app_menu_destroy, GINT_TO_POINTER(change_handler) );
    ++n_ref;
 }
 
@@ -533,7 +533,7 @@ GtkWidget* ptk_app_menu_new()
 {
     GtkWidget* menu;
     menu = gtk_menu_new();
-    ptk_app_menu_insert_items( menu, -1 );
+    ptk_app_menu_insert_items( GTK_MENU(menu), -1 );
     return menu;
 }
 
@@ -544,7 +544,7 @@ void app_dirs_foreach( GFunc func, gpointer user_data )
     int i, len;
     struct stat dir_stat;
 
-    len = g_strv_length(sys_dirs);
+    len = g_strv_length((gchar **) sys_dirs);
     if( !times )
         times = g_new0( time_t, len + 2 );
     for( i = 0; i < len; ++i )
@@ -594,7 +594,7 @@ gboolean ptk_app_menu_need_reload()
 
     if( !times )
         return TRUE;
-    len = g_strv_length(sys_dirs);
+    len = g_strv_length((gchar **) sys_dirs);
     for( i = 0; i < len; ++i )
     {
         path = g_build_filename( sys_dirs[i], app_dir_name, NULL );
diff --git a/src/plugins/ptk-app-menu.h b/src/plugins/ptk-app-menu.h
new file mode 100644 (file)
index 0000000..7c2b727
--- /dev/null
@@ -0,0 +1,9 @@
+#ifndef PTK_APP_MENU_H
+#define PTK_APP_MENU_H
+
+gboolean ptk_app_menu_item_has_data( GtkMenuItem* item );
+void ptk_app_menu_insert_items( GtkMenu* menu, int position );
+gboolean ptk_app_menu_need_reload();
+
+#endif
+
index 028f032..3820bc3 100644 (file)
@@ -21,7 +21,7 @@ int mixer_fd;
 typedef struct {
        GtkWidget *mainw;
        GtkWidget *dlg;
-    GtkTooltips* tooltips;
+       GtkTooltips* tooltips;
 } volume_t;
 
 static void
@@ -32,7 +32,7 @@ volume_destructor(plugin *p)
        ENTER;
        if (vol->dlg)
                gtk_widget_destroy(vol->dlg);
-    g_object_unref( vol->tooltips );
+       g_object_unref( vol->tooltips );
        gtk_widget_destroy(vol->mainw);
        if (mixer_fd)
                close(mixer_fd);
@@ -54,7 +54,8 @@ static void on_button_press (GtkWidget* widget, GdkEventButton* evt, plugin* p)
        if( evt->button == 1 ) { /*  Left click*/
                if( ! vol->dlg ) {
                        vol->dlg = create_volume_window();
-                       g_signal_connect( vol->dlg, "focus-out-event", on_volume_focus, vol );
+                       g_signal_connect( vol->dlg, "focus-out-event", 
+                                         G_CALLBACK(on_volume_focus), vol );
                }
                gtk_window_present( GTK_WINDOW(vol->dlg) );
        }
@@ -67,8 +68,8 @@ volume_constructor(plugin *p)
        line s;
        GdkPixbuf *icon;
        GtkWidget *image;
-    GtkIconTheme* theme;
-    GtkIconInfo* info;
+       GtkIconTheme* theme;
+       GtkIconInfo* info;
 
        ENTER;
        s.len = 256;  
@@ -84,16 +85,16 @@ volume_constructor(plugin *p)
 
        vol->mainw = gtk_event_box_new();
 
-    theme = gtk_icon_theme_get_default();
-    if( info = gtk_icon_theme_lookup_icon( theme, "stock_volume", 24, 0 ) )
-    {
-        icon = gdk_pixbuf_new_from_file_at_size( gtk_icon_info_get_filename( info ),
-                                                 24, 24, NULL );
-        gtk_icon_info_free( info );
-    }
-    else
-       icon = gdk_pixbuf_new_from_xpm_data(volume_xpm);
-       if(icon) {
+       theme = gtk_icon_theme_get_default();
+       if ( (info = gtk_icon_theme_lookup_icon( theme, "stock_volume", 24, 0 ) ) ) {
+               icon = gdk_pixbuf_new_from_file_at_size( 
+                               gtk_icon_info_get_filename( info ),
+                               24, 24, NULL );
+               gtk_icon_info_free( info );
+       }
+       else
+               icon = gdk_pixbuf_new_from_xpm_data((const char **) volume_xpm);
+       if (icon) {
                image = gtk_image_new_from_pixbuf(icon);
                gtk_container_add (GTK_CONTAINER (vol->mainw), image);
        }
@@ -108,15 +109,11 @@ volume_constructor(plugin *p)
 
        gtk_container_add(GTK_CONTAINER(p->pwid), vol->mainw);
 
-    vol->tooltips = gtk_tooltips_new ();
-    /* FIXME: display current level in tooltip. ex: "Volume Control: 80%"  */
-    gtk_tooltips_set_tip (vol->tooltips, vol->mainw, _("Volume control"), NULL);
+       vol->tooltips = gtk_tooltips_new ();
+       /* FIXME: display current level in tooltip. ex: "Volume Control: 80%"  */
+       gtk_tooltips_set_tip (vol->tooltips, vol->mainw, _("Volume control"), NULL);
 
        RET(1);
-
-error:
-       volume_destructor(p);
-       RET(0);
 }