2011-04-08 [colin] 3.7.8cvs76
[claws.git] / src / action.c
index 81e28fa06ee583005f147b9691341a28b55f53dc..0bd1db715690077e2c4d279b27e2712ac82996f4 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2009 Hiroyuki Yamamoto & The Claws Mail Team
+ * Copyright (C) 1999-2011 Hiroyuki Yamamoto & 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
@@ -89,6 +89,7 @@ struct _ChildInfo
        Children        *children;
        gchar           *cmd;
        pid_t            pid;
+       gint             next_sig;
        gint             chld_in;
        gint             chld_out;
        gint             chld_err;
@@ -947,13 +948,10 @@ static ChildInfo *fork_child(gchar *cmd, const gchar *msg_str,
 
        pid = fork();
        if (pid == 0) { /* Child */
+               int r = 0;
                if (setpgid(0, 0))
                        perror("setpgid");
 
-#ifdef GDK_WINDOWING_X11
-               (void)close(ConnectionNumber(gdk_display));
-#endif /* GDK_WINDOWING_X11 */
-
                gch_pid = fork();
 
                if (gch_pid == 0) {
@@ -965,21 +963,24 @@ static ChildInfo *fork_child(gchar *cmd, const gchar *msg_str,
                                    (ACTION_PIPE_IN |
                                     ACTION_USER_IN |
                                     ACTION_USER_HIDDEN_IN)) {
-                                       (void)close(fileno(stdin));
-                                       (void)dup  (chld_in[0]);
+                                       r |= close(fileno(stdin));
+                                       (void) dup  (chld_in[0]);
                                }
-                               (void)close(chld_in[0]);
-                               (void)close(chld_in[1]);
-
-                               (void)close(fileno(stdout));
-                               (void)dup  (chld_out[1]);
-                               (void)close(chld_out[0]);
-                               (void)close(chld_out[1]);
-
-                               (void)close(fileno(stderr));
-                               (void)dup  (chld_err[1]);
-                               (void)close(chld_err[0]);
-                               (void)close(chld_err[1]);
+                               r |= close(chld_in[0]);
+                               r |= close(chld_in[1]);
+
+                               r |= close(fileno(stdout));
+                               (void) dup  (chld_out[1]);
+                               r |= close(chld_out[0]);
+                               r |= close(chld_out[1]);
+
+                               r |= close(fileno(stderr));
+                               (void) dup  (chld_err[1]);
+                               r |= close(chld_err[0]);
+                               r |= close(chld_err[1]);
+
+                               if (r != 0)
+                                       debug_print("%s(%d)", strerror(errno), errno);
                        }
 
                        cmdline[0] = "sh";
@@ -999,24 +1000,29 @@ static ChildInfo *fork_child(gchar *cmd, const gchar *msg_str,
                        _exit(1);
                } else if (gch_pid < (pid_t) 0) { /* Fork error */
                        if (sync)
-                               (void)write(chld_status[1], "1\n", 2);
+                               r = write(chld_status[1], "1\n", 2);
+                       if (r != 0)
+                               debug_print("%s(%d)", strerror(errno), errno);
                        perror("fork");
                        _exit(1);
                } else { /* Child */
                        if (sync) {
-                               (void)close(chld_in[0]);
-                               (void)close(chld_in[1]);
-                               (void)close(chld_out[0]);
-                               (void)close(chld_out[1]);
-                               (void)close(chld_err[0]);
-                               (void)close(chld_err[1]);
-                               (void)close(chld_status[0]);
+                               r |= close(chld_in[0]);
+                               r |= close(chld_in[1]);
+                               r |= close(chld_out[0]);
+                               r |= close(chld_out[1]);
+                               r |= close(chld_err[0]);
+                               r |= close(chld_err[1]);
+                               r |= close(chld_status[0]);
 
                                debug_print("Child: waiting for grandchild\n");
-                               waitpid(gch_pid, NULL, 0);
+                               r |= waitpid(gch_pid, NULL, 0);
                                debug_print("Child: grandchild ended\n");
-                               (void)write(chld_status[1], "0\n", 2);
-                               (void)close(chld_status[1]);
+                               r |= write(chld_status[1], "0\n", 2);
+                               r |= close(chld_status[1]);
+
+                               if (r != 0)
+                                       debug_print("%s(%d)", strerror(errno), errno);
                        }
                        _exit(0);
                }
@@ -1047,6 +1053,7 @@ static ChildInfo *fork_child(gchar *cmd, const gchar *msg_str,
        child_info->children    = children;
 
        child_info->pid         = pid;
+       child_info->next_sig    = SIGTERM;
        child_info->cmd         = g_strdup(cmd);
        child_info->new_out     = FALSE;
        child_info->output      = g_string_new(NULL);
@@ -1068,17 +1075,20 @@ static ChildInfo *fork_child(gchar *cmd, const gchar *msg_str,
                return child_info;
 
        if ((children->action_type & ACTION_PIPE_IN) && msg_str) {
+               int r;
                ret_str = g_locale_from_utf8(msg_str, strlen(msg_str),
                                             &by_read, &by_written, NULL);
                if (ret_str && by_written) {
-                       (void)write(chld_in[1], ret_str, strlen(ret_str));
+                       r = write(chld_in[1], ret_str, strlen(ret_str));
                        g_free(ret_str);
                } else
-                       (void)write(chld_in[1], msg_str, strlen(msg_str));
+                       r = write(chld_in[1], msg_str, strlen(msg_str));
                if (!(children->action_type &
                      (ACTION_USER_IN | ACTION_USER_HIDDEN_IN)))
-                       (void)close(chld_in[1]);
+                       r = close(chld_in[1]);
                child_info->chld_in = -1; /* No more input */
+               if (r != 0)
+                       debug_print("%s(%d)", strerror(errno), errno);
        }
 
        return child_info;
@@ -1097,8 +1107,9 @@ static void kill_children_cb(GtkWidget *widget, gpointer data)
        for (cur = children->list; cur; cur = cur->next) {
                child_info = (ChildInfo *)(cur->data);
                debug_print("Killing child group id %d\n", child_info->pid);
-               if (child_info->pid && kill(-child_info->pid, SIGTERM) < 0)
+               if (child_info->pid && kill(-child_info->pid, child_info->next_sig) < 0)
                        perror("kill");
+               child_info->next_sig = SIGKILL;
        }
 #endif /* G_OS_UNIX */
 }
@@ -1231,17 +1242,23 @@ static void update_io_dialog(Children *children)
 
        if (children->progress_bar) {
                gchar *text;
+#ifdef GENERIC_UMPC
+               /* use a more compact format */
+               const gchar *format = "%s %d/%d";
+#else
+               const gchar *format = "%s %d / %d";
+#endif
                
                gtk_progress_bar_set_fraction(GTK_PROGRESS_BAR(children->progress_bar),
                                                  (children->initial_nb == 0) ? 0 :
                                              (gdouble) (children->initial_nb - children->nb) /
                                              (gdouble) children->initial_nb);
-               text = g_strdup_printf("%s %d/%d", _("Completed"), 
+               text = g_strdup_printf(format, _("Completed"), 
                                       children->initial_nb - children->nb,
                                       children->initial_nb);
                gtk_progress_bar_set_text(GTK_PROGRESS_BAR(children->progress_bar), text);
                g_free(text);
-       }                                             
+       }
 
        if (!children->nb) {
                gtk_widget_set_sensitive(children->abort_btn, FALSE);
@@ -1387,12 +1404,18 @@ static void create_io_dialog(Children *children)
        }
 
        if (children->initial_nb > 1) {
-               gchar * text;
+               gchar *text;
+#ifdef GENERIC_UMPC
+               /* use a more compact format */
+               const gchar *format = "%s 0/%d\n";
+#else
+               const gchar *format = "%s 0 / %d\n";
+#endif
                
                progress_bar = gtk_progress_bar_new();
                gtk_progress_bar_set_orientation(GTK_PROGRESS_BAR(progress_bar),
                                GTK_PROGRESS_LEFT_TO_RIGHT);
-               text = g_strdup_printf("%s 0/%d\n", _("Completed"), 
+               text = g_strdup_printf(format, _("Completed"), 
                                       children->initial_nb);
                gtk_progress_bar_set_text(GTK_PROGRESS_BAR(progress_bar),
                                          text);
@@ -1499,7 +1522,7 @@ static void catch_input(gpointer data, gint source, GdkInputCondition cond)
        Children *children = (Children *)data;
        ChildInfo *child_info = (ChildInfo *)children->list->data;
        gchar *input, *ret_str;
-       gint c, count, len;
+       gint c, count, len, r;
        gssize by_read = 0, by_written = 0;
 
        debug_print("Sending input to grand child.\n");
@@ -1531,11 +1554,13 @@ static void catch_input(gpointer data, gint source, GdkInputCondition cond)
        } while (c >= 0 && count < len);
 
        if (c >= 0)
-               (void)write(child_info->chld_in, "\n", 2);
+               r = write(child_info->chld_in, "\n", 2);
 
        g_free(input);
 
-       (void)close(child_info->chld_in);
+       r = close(child_info->chld_in);
+       if (r != 0)
+               debug_print("%s(%d)", strerror(errno), errno);
        child_info->chld_in = -1;
        debug_print("Input to grand child sent.\n");
 }