Coverity fixes

[claws.git] / src / statusbar.c

diff --git a/src/statusbar.c b/src/statusbar.c
index 129d308e846d00e45735c5620a01b2e9b033b397..4ced87f329cc0923746b2a14df119a15e9af6036 100644 (file)
--- a/src/statusbar.c
+++ b/src/statusbar.c
@@ -1,10 +1,10 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2006 Hiroyuki Yamamoto and the Sylpheed-Claws team
+ * Copyright (C) 1999-2013 Hiroyuki Yamamoto and the Claws Mail team
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
+ * the Free Software Foundation; either version 3 of the License, or
  * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful,
@@ -13,18 +13,18 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ * 
  */
 
 #ifdef HAVE_CONFIG_H
 #  include "config.h"
+#include "claws-features.h"
 #endif
 
 #include <glib.h>
 #include <glib/gi18n.h>
-#include <gtk/gtkstatusbar.h>
-#include <gtk/gtkprogressbar.h>
+#include <gtk/gtk.h>
 #include <stdarg.h>
 
 #include "mainwindow.h"
@@ -42,12 +42,26 @@ gint statusbar_puts_all_hook_id = -1;
 GtkWidget *statusbar_create(void)
 {
        GtkWidget *statusbar;
+       GtkWidget *child;
+       GtkWidget *parent;
+       GtkWidget *hbox;
 
        statusbar = gtk_statusbar_new();
        gtk_widget_set_size_request(statusbar, 1, -1);
        statusbar_list = g_list_append(statusbar_list, statusbar);
+#if !GTK_CHECK_VERSION(3, 0, 0)
        gtk_statusbar_set_has_resize_grip(GTK_STATUSBAR(statusbar), 
                                          FALSE);
+#endif
+       gtk_container_set_border_width(GTK_CONTAINER(statusbar), 1);
+       child = gtk_statusbar_get_message_area(GTK_STATUSBAR(statusbar));
+       parent = gtk_widget_get_parent(child);
+       gtk_container_remove(GTK_CONTAINER(parent), g_object_ref(child));
+       hbox = gtk_hbox_new(FALSE, 0);
+       gtk_container_add(GTK_CONTAINER(parent), hbox);
+       gtk_widget_show(hbox);
+       gtk_box_pack_start(GTK_BOX(hbox), child, TRUE, TRUE, 0);
+       g_object_unref(child);  
 
        return statusbar;
 }
@@ -117,12 +131,12 @@ void statusbar_pop_all(void)
        }
 }
 
-gboolean statusbar_puts_all_hook (gpointer source, gpointer data)
+static gboolean statusbar_puts_all_hook (gpointer source, gpointer data)
 {
        LogText *logtext = (LogText *) source;
 
-       g_return_val_if_fail(logtext != NULL, TRUE);
-       g_return_val_if_fail(logtext->text != NULL, TRUE);
+       cm_return_val_if_fail(logtext != NULL, TRUE);
+       cm_return_val_if_fail(logtext->text != NULL, TRUE);
 
        statusbar_pop_all();
        if (logtext->type == LOG_NORMAL) {
@@ -148,19 +162,26 @@ void statusbar_verbosity_set(gboolean verbose)
 
 void statusbar_progress_all (gint done, gint total, gint step) 
 {
+       GtkProgressBar *progressbar = GTK_PROGRESS_BAR(
+                                       mainwindow_get_mainwindow()->progressbar);
        gchar buf[32];
-       g_snprintf(buf, sizeof(buf), "%d / %d", done, total);
+       
        if (total && done % step == 0) {
-               gtk_progress_bar_set_text
-                       (GTK_PROGRESS_BAR(mainwindow_get_mainwindow()->progressbar), buf);
-               gtk_progress_bar_set_fraction
-                       (GTK_PROGRESS_BAR(mainwindow_get_mainwindow()->progressbar),
+#ifdef GENERIC_UMPC
+               /* use a more compact format */
+               const gchar *format = "%d/%d";
+#else
+               const gchar *format = "%d / %d";
+#endif
+               g_snprintf(buf, sizeof(buf), format, done, total);
+               gtk_progress_bar_set_text(progressbar, buf);
+               gtk_progress_bar_set_fraction(progressbar,
                         (gfloat)done / (gfloat)total);
+               if (!gtk_widget_get_visible(GTK_WIDGET(progressbar)))
+                       gtk_widget_show(GTK_WIDGET(progressbar));
        } else if (total == 0) {
-               gtk_progress_bar_set_text
-                       (GTK_PROGRESS_BAR(mainwindow_get_mainwindow()->progressbar), "");
-               gtk_progress_bar_set_fraction
-                       (GTK_PROGRESS_BAR(mainwindow_get_mainwindow()->progressbar), 0.0);
+               gtk_progress_bar_set_text(progressbar, "");
+               gtk_progress_bar_set_fraction(progressbar, 0.0);
+               gtk_widget_hide(GTK_WIDGET(progressbar));
        }
-       
 }