Unify all the logging: remove ERR() and LOG(), all log_level* variables, --log option.
authorAndriy Grytsenko <andrej@rep.kiev.ua>
Wed, 20 Aug 2014 08:36:24 +0000 (11:36 +0300)
committerAndriy Grytsenko <andrej@rep.kiev.ua>
Wed, 20 Aug 2014 08:36:24 +0000 (11:36 +0300)
The macros ENTER, RET(), and DBG() are left for 'make -DDEBUG' with
excess flood log for detailed debugging if someone would want it.

In case if someone would want to change logging there are G_MESSAGES_DEBUG
and G_MESSAGES_PREFIXED environment variables for GLib logging.

18 files changed:
plugins/batt/batt.c
plugins/cpufreq/cpufreq.c
plugins/launchtaskbar.c
plugins/menu.c
plugins/monitors/monitors.c
plugins/netstat/devproc.c
plugins/thermal/thermal.c
plugins/tray.c
plugins/volumealsa/volumealsa.c
plugins/wincmd.c
src/bg.c
src/configurator.c
src/dbg.c
src/dbg.h
src/misc.c
src/panel.c
src/plugin.c
src/private.h

index 427856c..408de79 100644 (file)
@@ -469,7 +469,7 @@ static GtkWidget * constructor(LXPanel *panel, config_setting_t *settings)
     lx_b->panel = panel;
     lx_b->settings = settings;
 
-    lx_b->show_extended_information = false;
+    lx_b->show_extended_information = FALSE;
 
     if (config_setting_lookup_int(settings, "HideIfNoBattery", &tmp_int))
         lx_b->hide_if_no_battery = (tmp_int != 0);
index 203f8e8..8a3b94a 100644 (file)
@@ -298,7 +298,7 @@ cpufreq_menu(cpufreq *cf){
 static  gboolean
 clicked(GtkWidget *widget, GdkEventButton *evt, LXPanel *panel)
 {
-    ENTER2;
+    ENTER;
 
     /* Standard right-click handling. */
     if( evt->button == 1 )
@@ -309,7 +309,7 @@ clicked(GtkWidget *widget, GdkEventButton *evt, LXPanel *panel)
       return TRUE;
     }
 
-    RET2(FALSE);
+    RET(FALSE);
 }
 
 static gboolean
index 53f4175..8bf6c6e 100644 (file)
@@ -29,7 +29,8 @@
  * 2. Raise window when files get dragged over taskbar buttons.
  * 3. Add Restore & Maximize menu items to popup menu of task bar buttons.
  */
-//#define DEBUG_WITH_GPRINTS // killall lxpanel && lxpanel --profile Lubuntu &
+
+//#define DEBUG // killall lxpanel && lxpanel --profile Lubuntu &
 
 #ifdef HAVE_CONFIG_H
 #include <config.h>
@@ -62,8 +63,6 @@
 # include "menu-policy.h"
 #endif
 
-#include "dbg.h" /* for ERR macro. FIXME: is it required? */
-
 
 #define PANEL_ICON_SIZE 24 /* see the private.h */
 
@@ -234,9 +233,9 @@ static void f_get_exec_cmd_from_pid(GPid pid, gchar *buffer_128, const gchar *pr
     snprintf(command, 64, "cat /proc/%u/%s", pid, proc_file);
     pipe = popen(command, "r");
     if(pipe == NULL)
-        ERR("ltbp: popen '%s'\n", command);
+        g_warning("ltbp: popen '%s'", command);
     else if(fgets(buffer_128, 128, pipe) == NULL)
-        ERR("ltbp: fgets '%s'\n", command);
+        g_warning("ltbp: fgets '%s'", command);
     else
     {
         gchar *p_char = strchr(buffer_128, '\n');
@@ -446,7 +445,7 @@ static void launchbar_update_after_taskbar_class_added(LaunchTaskBarPlugin *ltbp
     }
     tk->exec_bin = g_strdup(exec_bin_full);
 
-#ifdef DEBUG_WITH_GPRINTS
+#ifdef DEBUG
     if(ltbp->mode == LAUNCHTASKBAR)
     {
         FmFileInfo *fi = f_find_menu_launchbutton_recursive(tk->exec_bin);
@@ -461,7 +460,7 @@ static void launchbar_update_after_taskbar_class_added(LaunchTaskBarPlugin *ltbp
 
 static void launchbar_update_after_taskbar_class_removed(LaunchTaskBarPlugin *ltbp, Task *tk)
 {
-#ifdef DEBUG_WITH_GPRINTS
+#ifdef DEBUG
     if(ltbp->mode == LAUNCHTASKBAR)
     {
         FmFileInfo *fi = f_find_menu_launchbutton_recursive(tk->exec_bin);
@@ -1701,9 +1700,8 @@ static void task_set_names(Task * tk, Atom source)
     if (name != NULL)
     {
         task_free_names(tk);
-        tk->name = g_strdup(name);
+        tk->name = name;
         tk->name_iconified = g_strdup_printf("[%s]", name);
-        g_free(name);
 
         /* Redraw the button. */
         task_button_redraw(tk, tk->tb);
@@ -2500,7 +2498,7 @@ static gboolean taskbar_task_control_event(GtkWidget * widget, GdkEventButton *
                 FmFileInfo *fi = f_find_menu_launchbutton_recursive(tk->exec_bin);
                 LaunchButton *btn = launchbar_exec_bin_exists(ltbp, fi);
                 /* FIXME: shouldn't we make file info at task button creation? */
-#ifdef DEBUG_WITH_GPRINTS
+#ifdef DEBUG
                 g_print("\nTB '%s' right-click, in LB: %c\n", tk->exec_bin, btn != NULL ? 'Y':'N');
 #endif
                 if(btn != NULL)
index accc1f0..4149f4c 100644 (file)
@@ -832,7 +832,7 @@ read_system_menu(GtkMenu *menu, menup *m, config_setting_t *s)
         m->menu_cache = panel_menu_cache_new(&flags);
         if (m->menu_cache == NULL)
         {
-            ERR("error loading applications menu");
+            g_warning("error loading applications menu");
             return;
         }
         m->visibility_flags = flags;
@@ -919,11 +919,11 @@ read_submenu(menup *m, config_setting_t *s, gboolean as_item)
             continue;
 #endif
         } else {
-            ERR("menu: unknown block %s\n", str);
+            g_warning("menu: unknown block %s", str);
             goto error;
         }
         if (!mi) {
-            ERR("menu: can't create menu item\n");
+            g_warning("menu: can't create menu item");
             goto error;
         }
         gtk_widget_show(mi);
@@ -993,7 +993,7 @@ menu_constructor(LXPanel *panel, config_setting_t *settings)
     }
 
     if (!read_submenu(m, m->settings, FALSE)) {
-        ERR("menu: plugin init failed\n");
+        g_warning("menu: plugin init failed");
         gtk_widget_destroy(m->box);
         return NULL;
     }
index c127cf9..fa76731 100644 (file)
@@ -299,8 +299,8 @@ mem_update(Monitor * m)
 
     meminfo = fopen("/proc/meminfo", "r");
     if (!meminfo) {
-        ERR("monitors: Could not open /proc/meminfo: %d, %s\n",
-                errno, strerror(errno));
+        g_warning("monitors: Could not open /proc/meminfo: %d, %s",
+                  errno, strerror(errno));
         RET(FALSE);
     }
 
@@ -326,8 +326,8 @@ mem_update(Monitor * m)
     fclose(meminfo);
 
     if (readmask) {
-        ERR("monitors: Couldn't read all values from /proc/meminfo: "
-                "readmask %x\n", readmask);
+        g_warning("monitors: Couldn't read all values from /proc/meminfo: "
+                  "readmask %x", readmask);
         RET(FALSE);
     }
 
index 03786f7..f6ff7c0 100644 (file)
@@ -225,10 +225,10 @@ int netproc_scandevice(int sockfd, int iwsockfd, FILE *fp, NETDEVLIST_PTR *netde
 
        status = fgets (buffer, sizeof(buffer), fp);
        if (!status)
-               ERR("netstat: netproc_scnadevice(): Error reading first line from stream!\n");
+               g_warning("netstat: netproc_scnadevice(): Error reading first line from stream!");
        status = fgets (buffer, sizeof(buffer), fp);
        if (!status)
-               ERR("netstat: netproc_scnadevice(): Error reading second line from stream!\n");
+               g_warning("netstat: netproc_scnadevice(): Error reading second line from stream!");
        netproc_parse_stats_header(buffer, &prx_idx, &ptx_idx, &brx_idx, &btx_idx);
 
        while (fgets(buffer, sizeof(buffer), fp)) {
index f028f1d..07767d1 100644 (file)
 #define MAX_NUM_SENSORS 10
 #define MAX_AUTOMATIC_CRITICAL_TEMP 150 /* in degrees Celsius */
 
+#if !GLIB_CHECK_VERSION(2, 40, 0)
+# define g_info(...) g_log(G_LOG_DOMAIN, G_LOG_LEVEL_INFO, __VA_ARGS__)
+#endif
+
 typedef gint (*GetTempFunc)(char const *);
 
 typedef struct thermal {
@@ -84,7 +88,7 @@ proc_get_critical(char const* sensor_path){
     snprintf(sstmp,sizeof(sstmp),"%s%s",sensor_path,PROC_THERMAL_TRIP);
 
     if (!(state = fopen( sstmp, "r"))) {
-        ERR("thermal: cannot open %s\n", sstmp);
+        g_warning("thermal: cannot open %s", sstmp);
         return -1;
     }
 
@@ -116,7 +120,7 @@ proc_get_temperature(char const* sensor_path){
     snprintf(sstmp,sizeof(sstmp),"%s%s",sensor_path,PROC_THERMAL_TEMPF);
 
     if (!(state = fopen( sstmp, "r"))) {
-        ERR("thermal: cannot open %s\n", sstmp);
+        g_warning("thermal: cannot open %s", sstmp);
         return -1;
     }
 
@@ -144,7 +148,7 @@ static gint _get_reading(const char *path)
     char* pstr;
 
     if (!(state = fopen(path, "r"))) {
-        ERR("thermal: cannot open %s\n", path);
+        g_warning("thermal: cannot open %s", path);
         return -1;
     }
 
@@ -284,7 +288,7 @@ add_sensor(thermal* th, char const* sensor_path, const char *sensor_name,
            GetTempFunc get_temp, GetTempFunc get_crit)
 {
     if (th->numsensors + 1 > MAX_NUM_SENSORS){
-        ERR("thermal: Too many sensors (max %d), ignoring '%s'\n",
+        g_warning("thermal: Too many sensors (max %d), ignoring '%s'",
                 MAX_NUM_SENSORS, sensor_path);
         return -1;
     }
@@ -295,7 +299,7 @@ add_sensor(thermal* th, char const* sensor_path, const char *sensor_name,
     th->get_temperature[th->numsensors] = get_temp;
     th->numsensors++;
 
-    LOG(LOG_ALL, "thermal: Added sensor %s\n", sensor_path);
+    g_debug("thermal: Added sensor %s", sensor_path);
 
     return 0;
 }
@@ -375,7 +379,7 @@ remove_all_sensors(thermal *th)
 {
     int i;
 
-    LOG(LOG_ALL, "thermal: Removing all sensors (%d)\n", th->numsensors);
+    g_debug("thermal: Removing all sensors (%d)", th->numsensors);
 
     for (i = 0; i < th->numsensors; i++)
     {
@@ -393,7 +397,7 @@ check_sensors( thermal *th )
     find_sensors(th, SYSFS_THERMAL_DIRECTORY, SYSFS_THERMAL_SUBDIR_PREFIX, sysfs_get_temperature, sysfs_get_critical);
     if (th->numsensors == 0)
         find_hwmon_sensors(th);
-    LOG(LOG_INFO, "thermal: Found %d sensors\n", th->numsensors);
+    g_info("thermal: Found %d sensors", th->numsensors);
 }
 
 
index 5e066d5..68051aa 100644 (file)
@@ -44,9 +44,6 @@
 #define SYSTEM_TRAY_ORIENTATION_HORZ 0
 #define SYSTEM_TRAY_ORIENTATION_VERT 1
 
-//#define DEBUG
-#include "dbg.h"
-
 struct _balloon_message;
 struct _tray_client;
 struct _tray_plugin;
@@ -106,19 +103,16 @@ static TrayClient * client_lookup(TrayPlugin * tr, Window window)
 
 static void client_print(TrayPlugin * tr, char c, TrayClient * tc, XClientMessageEvent * xevent)
 {
-    int const log_at_level = LOG_ALL;
-    if (log_at_level <= log_level) {
         char *name = get_utf8_property(tc->window, a_NET_WM_NAME);
         int pid = get_net_wm_pid(tc->window);
         XClientMessageEvent xcm = {0};
         if (!xevent)
             xevent = &xcm;
-        LOG(log_at_level, "tray: %c%p, winid 0x%lx: %s (PID %d), plug %p, serial no %lu, send_event %c, format %d\n",
+        g_debug("tray: %c%p, winid 0x%lx: %s (PID %d), plug %p, serial no %lu, send_event %c, format %d",
                 c, tc, tc->window, name, pid,
                 gtk_socket_get_plug_window(GTK_SOCKET(tc->socket)),
                 xevent->serial, xevent->send_event ? 'y' : 'n', xevent->format);
         g_free(name);
-    }
 }
 
 /* Delete a client. */
@@ -569,7 +563,7 @@ static GtkWidget *tray_constructor(LXPanel *panel, config_setting_t *settings)
     /* If the selection is already owned, there is another tray running. */
     if (XGetSelectionOwner(GDK_DISPLAY_XDISPLAY(display), selection_atom) != None)
     {
-        ERR("tray: another systray already running\n");
+        g_warning("tray: another systray already running");
         return NULL;
     }
 
index 90116a8..cf2df0e 100644 (file)
@@ -27,7 +27,6 @@
 #include <poll.h>
 #include <libfm/fm-gtk.h>
 #include "plugin.h"
-#include "dbg.h"
 
 #define ICONS_VOLUME_HIGH   PACKAGE_DATA_DIR "/images/volume-high.png"
 #define ICONS_VOLUME_MEDIUM PACKAGE_DATA_DIR "/images/volume-medium.png"
@@ -137,9 +136,9 @@ static gboolean asound_mixer_event(GIOChannel * channel, GIOCondition cond, gpoi
     if ((cond & G_IO_HUP) || (res < 0))
     {
         /* This means there're some problems with alsa. */
-        ERR("volumealsa: ALSA (or pulseaudio) had a problem: \n"
+        g_warning("volumealsa: ALSA (or pulseaudio) had a problem: "
                 "volumealsa: snd_mixer_handle_events() = %d,"
-                " cond 0x%x (IN: 0x%x, HUP: 0x%x).\n", res, cond,
+                " cond 0x%x (IN: 0x%x, HUP: 0x%x).", res, cond,
                 G_IO_IN, G_IO_HUP);
         gtk_widget_set_tooltip_text(vol->plugin, "ALSA (or pulseaudio) had a problem."
                 " Please check the lxpanel logs.");
@@ -163,11 +162,11 @@ static gboolean asound_restart(gpointer vol_gpointer)
     asound_deinitialize(vol);
 
     if (!asound_initialize(vol)) {
-        ERR("volumealsa: Re-initialization failed.\n");
+        g_warning("volumealsa: Re-initialization failed.");
         return TRUE; // try again in a second
     }
 
-    ERR("volumealsa: Restarted ALSA interface...\n");
+    g_warning("volumealsa: Restarted ALSA interface...");
 
     vol->restart_idle = 0;
     return FALSE;
index 9e08522..c80819c 100644 (file)
@@ -79,7 +79,7 @@ static void wincmd_execute(WinCmdPlugin * wc, WindowCommand command)
             /* If the task is visible on the current desktop and it is an ordinary window,
              * execute the requested Iconify or Shade change. */
             if (((task_desktop == -1) || (task_desktop == current_desktop))
-            && (( ! nwwt.dock) && ( ! nwwt.desktop) && ( ! nwwt.splash)))
+                && (( ! nwwt.dock) && ( ! nwwt.desktop) && ( ! nwwt.splash)))
             {
                 switch (command)
                 {
index 59ed594..906fdf8 100644 (file)
--- a/src/bg.c
+++ b/src/bg.c
@@ -193,14 +193,14 @@ fb_bg_get_xroot_pix_for_win(FbBg *bg, GtkWidget *widget)
     win = GDK_WINDOW_XWINDOW(gtk_widget_get_window(widget));
     if (!XGetGeometry(bg->dpy, win, &dummy, &x, &y, &width, &height, &border,
               &depth)) {
-        DBG2("XGetGeometry failed\n");
+        g_warning("XGetGeometry failed\n");
         RET(NULL);
     }
     XTranslateCoordinates(bg->dpy, win, bg->xroot, 0, 0, &x, &y, &dummy);
     DBG("win=%x %dx%d%+d%+d\n", win, width, height, x, y);
     gbgpix = gdk_pixmap_new(NULL, width, height, depth);
     if (!gbgpix) {
-        ERR("gdk_pixmap_new failed\n");
+        g_critical("gdk_pixmap_new failed");
         RET(NULL);
     }
     bgpix =  gdk_x11_drawable_get_xid(gbgpix);
index 9cf067f..85acbaa 100644 (file)
@@ -289,16 +289,6 @@ static void set_width_type( GtkWidget *item, LXPanel* panel )
 
 /* FIXME: heighttype and spacing and RoundCorners */
 
-static void set_log_level( GtkWidget *cbox, Panel* p)
-{
-    configured_log_level = gtk_combo_box_get_active(GTK_COMBO_BOX(cbox));
-    if (!log_level_set_on_commandline)
-        log_level = configured_log_level;
-    ERR("panel-pref: log level configured to %d, log_level is now %d\n",
-            configured_log_level, log_level);
-    UPDATE_GLOBAL_INT(p, "loglevel", configured_log_level);
-}
-
 static void transparency_toggle( GtkWidget *b, Panel* p)
 {
     GtkWidget* tr = (GtkWidget*)g_object_get_data(G_OBJECT(b), "tint_clr");
@@ -1209,11 +1199,6 @@ void panel_configure( LXPanel* panel, int sel_page )
                         &logout_cmd);
     }
 
-    w = (GtkWidget*)gtk_builder_get_object( builder, "log_level" );
-    update_opt_menu(w, configured_log_level);
-    g_signal_connect(w, "changed", G_CALLBACK(set_log_level), p);
-
-
     panel_adjust_geometry_terminology(p);
     gtk_widget_show(GTK_WIDGET(p->pref_dialog));
     w = (GtkWidget*)gtk_builder_get_object( builder, "notebook" );
@@ -1230,9 +1215,9 @@ void panel_config_save( Panel* p )
     /* existance of 'panels' dir ensured in main() */
 
     if (!config_write_file(p->config, fname)) {
-        ERR("can't open for write %s:", fname);
+        g_warning("can't open for write %s:", fname);
         g_free( fname );
-        RET();
+        return;
     }
     g_free( fname );
 
index 2e9153b..e18bbe8 100644 (file)
--- a/src/dbg.c
+++ b/src/dbg.c
 
 #include "dbg.h"
 
-int log_level = LOG_WARN;
-bool log_level_set_on_commandline = false;
-int configured_log_level = LOG_WARN;
-
-
+#include <glib.h>
 
 void _check_cairo_status(cairo_t* cr, char const* file, char const* func, int line)
 {
     cairo_status_t status = cairo_status(cr);
     if (status != CAIRO_STATUS_SUCCESS)
-        ERR("%s:%s:%-5d: cairo had error %d: %s\n", file, func, line, status,
+        g_critical("%s:%s:%-5d: cairo had error %d: %s", file, func, line, status,
                 cairo_status_to_string(status));
 }
 
@@ -36,7 +32,7 @@ void _check_cairo_surface_status(cairo_surface_t** surf, char const* file, char
 {
     cairo_status_t status = cairo_surface_status(*surf);
     if (status != CAIRO_STATUS_SUCCESS) {
-        ERR("%s:%s:%-5d: cairo had error %d: %s\n", file, func, line, status,
+        g_critical("%s:%s:%-5d: cairo had error %d: %s", file, func, line, status,
                 cairo_status_to_string(status));
         cairo_surface_destroy(*surf);
         *surf = NULL;
index 50dae0f..3539fff 100644 (file)
--- a/src/dbg.h
+++ b/src/dbg.h
  */
 
 #include <stdio.h>
-#include <stdbool.h>
 #include <cairo/cairo.h>
 
-#define ERR(fmt, args...) fprintf(stderr, fmt, ## args)
-#define DBG2(fmt, args...) fprintf(stderr, "%s:%s:%-5d: " fmt, __FILE__,  __FUNCTION__, __LINE__, ## args)
-#define ENTER2          do { fprintf(stderr, "%s:%s:%-5d: ENTER\n",  __FILE__,__FUNCTION__, __LINE__); } while(0)
-#define RET2(args...)   do { fprintf(stderr, "%s:%s:%-5d: RETURN\n",  __FILE__,__FUNCTION__, __LINE__);\
-return args; } while(0)
-
-enum { LOG_NONE, LOG_ERR, LOG_WARN, LOG_INFO, LOG_ALL };
-
-extern int log_level;
-extern bool log_level_set_on_commandline;
-extern int configured_log_level;
+/*
+ * Macros ENTER, RET(), and DBG() are for low-level debug. Use them in
+ * combination with defining DEBUG to add some messages that normally
+ * would never be shown. Usually those messages will spam your console
+ * and are just for finding problem, not for production usage.
+ */
 
 #ifdef DEBUG
 
@@ -38,14 +32,12 @@ extern int configured_log_level;
 #define RET(args...)   do { fprintf(stderr, "%s:%s:%-5d: RETURN\n", __FILE__, __FUNCTION__, __LINE__);\
 return args; } while(0)
 #define DBG(fmt, args...) fprintf(stderr, "%s:%s:%-5d: " fmt,  __FILE__,__FUNCTION__, __LINE__, ## args)
-#define LOG(level, fmt, args...) fprintf(stderr, fmt, ## args)
 
 #else
 
 #define ENTER         do {  } while(0)
 #define RET(args...)   return args
 #define DBG(fmt, args...)   do {  } while(0)
-#define LOG(level, fmt, args...) do { if (level <= log_level) fprintf(stderr, fmt, ## args); } while(0)
 
 #endif
 
index 65da827..adb396d 100644 (file)
@@ -301,7 +301,7 @@ lxpanel_get_line(char**fp, line *s)
             *tmp = 0;
             s->type = LINE_BLOCK_START;
         } else {
-            ERR( "parser: unknown token: '%c'\n", *tmp2);
+            g_warning( "parser: unknown token: '%c'", *tmp2);
         }
         break;
     }
@@ -776,10 +776,8 @@ int panel_handle_x_error(Display * d, XErrorEvent * ev)
 {
     char buf[256];
 
-    if (log_level >= LOG_WARN) {
-        XGetErrorText(d, ev->error_code, buf, 256);
-        LOG(LOG_WARN, "lxpanel : X error: %s\n", buf);
-    }
+    XGetErrorText(d, ev->error_code, buf, 256);
+    g_warning("lxpanel : X error: %s", buf);
     return 0;    /* Ignored */
 }
 
@@ -808,7 +806,7 @@ calculate_width(int scrw, int wtype, int allign, int margin,
     }
     if (allign != ALLIGN_CENTER) {
         if (margin > scrw) {
-            ERR( "margin is bigger then edge size %d > %d. Ignoring margin\n",
+            g_warning( "margin is bigger then edge size %d > %d. Ignoring margin",
                   margin, scrw);
             margin = 0;
         }
@@ -1435,7 +1433,7 @@ gboolean spawn_command_async(GtkWindow *parent_window, gchar const* workdir,
     GError* err = NULL;
     gchar** argv = NULL;
 
-    LOG(LOG_INFO, "lxpanel: spawning \"%s\"...\n", cmd);
+    g_info("lxpanel: spawning \"%s\"...", cmd);
 
     g_shell_parse_argv(cmd, NULL, &argv, &err);
     if (!err)
@@ -1443,7 +1441,7 @@ gboolean spawn_command_async(GtkWindow *parent_window, gchar const* workdir,
 
     if (err)
     {
-        ERR("%s\n", err->message);
+        g_warning("%s\n", err->message);
         fm_show_error(parent_window, NULL, err->message);
         g_error_free(err);
     }
index 6e5e2a0..9e4cc10 100644 (file)
@@ -922,7 +922,7 @@ static void panel_popupmenu_create_panel( GtkMenuItem* item, LXPanel* panel )
     }
 
     gtk_widget_destroy(GTK_WIDGET(new_panel));
-    ERR("Error adding panel: There is no room for another panel. All the edges are taken.\n");
+    g_warning("Error adding panel: There is no room for another panel. All the edges are taken.");
     fm_show_error(NULL, NULL, _("There is no room for another panel. All the edges are taken."));
     return;
 
@@ -1468,7 +1468,7 @@ panel_parse_global(Panel *p, config_setting_t *cfg)
     /* check Global config */
     if (!cfg || strcmp(config_setting_get_name(cfg), "Global") != 0)
     {
-        ERR( "lxpanel: Global section not found\n");
+        g_warning( "lxpanel: Global section not found");
         RET(0);
     }
     if (config_setting_lookup_string(cfg, "edge", &str))
@@ -1527,11 +1527,6 @@ panel_parse_global(Panel *p, config_setting_t *cfg)
     if (config_setting_lookup_string(cfg, "backgroundfile", &str))
         p->background_file = g_strdup(str);
     config_setting_lookup_int(cfg, "iconsize", &p->icon_size);
-    if (config_setting_lookup_int(cfg, "loglevel", &configured_log_level))
-    {
-        if (!log_level_set_on_commandline)
-            log_level = configured_log_level;
-    }
 
     panel_normalize_configuration(p);
 
@@ -1548,7 +1543,7 @@ panel_parse_plugin(LXPanel *p, config_setting_t *cfg)
     DBG("plug %s\n", type);
 
     if (!type || lxpanel_add_plugin(p, type, cfg, -1) == NULL) {
-        ERR( "lxpanel: can't load %s plugin\n", type);
+        g_warning( "lxpanel: can't load %s plugin", type);
         goto error;
     }
     RET(1);
@@ -1600,7 +1595,7 @@ static LXPanel* panel_new( const char* config_file, const char* config_name )
         if (!config_read_file(panel->priv->config, config_file) ||
             !panel_start(panel))
         {
-            ERR( "lxpanel: can't start panel\n");
+            g_warning( "lxpanel: can't start panel");
             gtk_widget_destroy(GTK_WIDGET(panel));
             panel = NULL;
         }
@@ -1615,7 +1610,7 @@ usage()
     g_print(_("Command line options:\n"));
     g_print(_(" --help      -- print this help and exit\n"));
     g_print(_(" --version   -- print version and exit\n"));
-    g_print(_(" --log <number> -- set log level 0-5. 0 - none 5 - chatty\n"));
+//    g_print(_(" --log <number> -- set log level 0-5. 0 - none 5 - chatty\n"));
 //    g_print(_(" --configure -- launch configuration utility\n"));
     g_print(_(" --profile name -- use specified profile\n"));
     g_print("\n");
@@ -1791,19 +1786,18 @@ int main(int argc, char *argv[], char *env[])
         } else if (!strcmp(argv[i], "--log")) {
             i++;
             if (i == argc) {
-                ERR( "lxpanel: missing log level\n");
+                g_critical( "lxpanel: missing log level");
                 usage();
                 exit(1);
             } else {
-                log_level = atoi(argv[i]);
-                log_level_set_on_commandline = true;
+                /* deprecated */
             }
         } else if (!strcmp(argv[i], "--configure") || !strcmp(argv[i], "-C")) {
             config = 1;
         } else if (!strcmp(argv[i], "--profile") || !strcmp(argv[i], "-p")) {
             i++;
             if (i == argc) {
-                ERR( "lxpanel: missing profile name\n");
+                g_critical( "lxpanel: missing profile name");
                 usage();
                 exit(1);
             } else {
index 5c1f195..703267f 100644 (file)
@@ -168,7 +168,7 @@ static void plugin_load_dynamic(const char * type, const gchar * path)
         || (strcmp(type, pc->type) != 0))                      /* Then and only then access other fields; check name */
         {
             g_module_close(m);
-            ERR("%s.so is not a lxpanel plugin\n", type);
+            g_warning("%s.so is not a lxpanel plugin", type);
             return;
         }
 
index 20214ac..136d0d1 100644 (file)
 #include "bg.h"
 #include "ev.h"
 
+#if !GLIB_CHECK_VERSION(2, 40, 0)
+# define g_info(...) g_log(G_LOG_DOMAIN, G_LOG_LEVEL_INFO, __VA_ARGS__)
+#endif
+
 /* -----------------------------------------------------------------------------
  *   Definitions used by lxpanel main code internally */