2006-06-15 [colin] 2.3.0cvs16
[claws.git] / src / compose.c
index aea95b2d214c8b3bb13bc68d4191e3ab9a8f8d6f..f567ea252341e20ed26f6e3048512c67928b2068 100644 (file)
@@ -271,15 +271,12 @@ static gint compose_write_body_to_file            (Compose        *compose,
 static gint compose_remove_reedit_target       (Compose        *compose,
                                                 gboolean        force);
 void compose_remove_draft                      (Compose        *compose);
-static gint compose_queue                      (Compose        *compose,
-                                                gint           *msgnum,
-                                                FolderItem     **item,
-                                                gchar          **msgpath);
 static gint compose_queue_sub                  (Compose        *compose,
                                                 gint           *msgnum,
                                                 FolderItem     **item,
                                                 gchar          **msgpath,
-                                                gboolean       check_subject);
+                                                gboolean       check_subject,
+                                                gboolean       remove_reedit_target);
 static void compose_add_attachments            (Compose        *compose,
                                                 MimeInfo       *parent);
 static gchar *compose_get_header               (Compose        *compose);
@@ -506,8 +503,6 @@ static void compose_check_forwards_go          (Compose *compose);
 static gint compose_defer_auto_save_draft      (Compose        *compose);
 static PrefsAccount *compose_guess_forward_account_from_msginfo        (MsgInfo *msginfo);
 
-static gboolean compose_close  (Compose *compose);
-
 static GtkItemFactoryEntry compose_popup_entries[] =
 {
        {N_("/_Add..."),        NULL, compose_attach_cb, 0, NULL},
@@ -646,6 +641,9 @@ static GtkItemFactoryEntry compose_entries[] =
                                        NULL, compose_check_backwards , 0, NULL},
        {N_("/_Spelling/_Forward to next misspelled word"),
                                        NULL, compose_check_forwards_go, 0, NULL},
+       {N_("/_Spelling/---"),          NULL, NULL, 0, "<Separator>"},
+       {N_("/_Spelling/Options"),
+                                       NULL, NULL, 0, "<Branch>"},
 #endif
        {N_("/_Options"),               NULL, NULL, 0, "<Branch>"},
        {N_("/_Options/Privacy System"),                NULL, NULL,   0, "<Branch>"},
@@ -790,7 +788,18 @@ static gboolean compose_put_existing_to_front(MsgInfo *info)
        return FALSE;
 }
 
-static GdkColor quote_color = 
+static GdkColor quote_color1 = 
+       {(gulong)0, (gushort)0, (gushort)0, (gushort)0};
+static GdkColor quote_color2 = 
+       {(gulong)0, (gushort)0, (gushort)0, (gushort)0};
+static GdkColor quote_color3 = 
+       {(gulong)0, (gushort)0, (gushort)0, (gushort)0};
+
+static GdkColor quote_bgcolor1 = 
+       {(gulong)0, (gushort)0, (gushort)0, (gushort)0};
+static GdkColor quote_bgcolor2 = 
+       {(gulong)0, (gushort)0, (gushort)0, (gushort)0};
+static GdkColor quote_bgcolor3 = 
        {(gulong)0, (gushort)0, (gushort)0, (gushort)0};
 
 static GdkColor signature_color = {
@@ -811,32 +820,92 @@ static void compose_create_tags(GtkTextView *text, Compose *compose)
 {
        GtkTextBuffer *buffer;
        GdkColor black = {(gulong)0, (gushort)0, (gushort)0, (gushort)0};
+       GdkColormap *cmap;
+       GdkColor color[8];
+       gboolean success[8];
+       int i;
 
        buffer = gtk_text_view_get_buffer(text);
-       
+
        if (prefs_common.enable_color) {
                /* grab the quote colors, converting from an int to a GdkColor */
                gtkut_convert_int_to_gdk_color(prefs_common.quote_level1_col,
-                                              &quote_color);
+                                              &quote_color1);
+               gtkut_convert_int_to_gdk_color(prefs_common.quote_level2_col,
+                                              &quote_color2);
+               gtkut_convert_int_to_gdk_color(prefs_common.quote_level3_col,
+                                              &quote_color3);
+               gtkut_convert_int_to_gdk_color(prefs_common.quote_level1_bgcol,
+                                              &quote_bgcolor1);
+               gtkut_convert_int_to_gdk_color(prefs_common.quote_level2_bgcol,
+                                              &quote_bgcolor2);
+               gtkut_convert_int_to_gdk_color(prefs_common.quote_level3_bgcol,
+                                              &quote_bgcolor3);
                gtkut_convert_int_to_gdk_color(prefs_common.signature_col,
                                               &signature_color);
                gtkut_convert_int_to_gdk_color(prefs_common.uri_col,
                                               &uri_color);
        } else {
-               signature_color = quote_color = uri_color = black;
+               signature_color = quote_color1 = quote_color2 = quote_color3 = 
+                       quote_bgcolor1 = quote_bgcolor2 = quote_bgcolor3 = uri_color = black;
+       }
+
+       if (prefs_common.enable_color && prefs_common.enable_bgcolor) {
+               gtk_text_buffer_create_tag(buffer, "quote0",
+                                          "foreground-gdk", &quote_color1,
+                                          "paragraph-background-gdk", &quote_bgcolor1,
+                                          NULL);
+               gtk_text_buffer_create_tag(buffer, "quote1",
+                                          "foreground-gdk", &quote_color2,
+                                          "paragraph-background-gdk", &quote_bgcolor2,
+                                          NULL);
+               gtk_text_buffer_create_tag(buffer, "quote2",
+                                          "foreground-gdk", &quote_color3,
+                                          "paragraph-background-gdk", &quote_bgcolor3,
+                                          NULL);
+       } else {
+               gtk_text_buffer_create_tag(buffer, "quote0",
+                                          "foreground-gdk", &quote_color1,
+                                          NULL);
+               gtk_text_buffer_create_tag(buffer, "quote1",
+                                          "foreground-gdk", &quote_color2,
+                                          NULL);
+               gtk_text_buffer_create_tag(buffer, "quote2",
+                                          "foreground-gdk", &quote_color3,
+                                          NULL);
        }
-
-       gtk_text_buffer_create_tag(buffer, "quote",
-                                  "foreground-gdk", &quote_color,
-                                  NULL);
+       
        gtk_text_buffer_create_tag(buffer, "signature",
                                   "foreground-gdk", &signature_color,
                                   NULL);
        gtk_text_buffer_create_tag(buffer, "link",
-                                        "foreground-gdk", &uri_color,
+                                       "foreground-gdk", &uri_color,
                                         NULL);
        compose->no_wrap_tag = gtk_text_buffer_create_tag(buffer, "no_wrap", NULL);
        compose->no_join_tag = gtk_text_buffer_create_tag(buffer, "no_join", NULL);
+
+       color[0] = quote_color1;
+       color[1] = quote_color2;
+       color[2] = quote_color3;
+       color[3] = quote_bgcolor1;
+       color[4] = quote_bgcolor2;
+       color[5] = quote_bgcolor3;
+       color[6] = signature_color;
+       color[7] = uri_color;
+       cmap = gdk_drawable_get_colormap(compose->window->window);
+       gdk_colormap_alloc_colors(cmap, color, 8, FALSE, TRUE, success);
+
+       for (i = 0; i < 8; i++) {
+               if (success[i] == FALSE) {
+                       GtkStyle *style;
+
+                       g_warning("Compose: color allocation failed.\n");
+                       style = gtk_widget_get_style(GTK_WIDGET(text));
+                       quote_color1 = quote_color2 = quote_color3 = 
+                               quote_bgcolor1 = quote_bgcolor2 = quote_bgcolor3 = 
+                               signature_color = uri_color = black;
+               }
+       }
 }
 
 Compose *compose_new(PrefsAccount *account, const gchar *mailto,
@@ -871,6 +940,7 @@ Compose *compose_generic_new(PrefsAccount *account, const gchar *mailto, FolderI
        g_return_val_if_fail(account != NULL, NULL);
 
        compose = compose_create(account, COMPOSE_NEW, FALSE);
+
        ifactory = gtk_item_factory_from_widget(compose->menubar);
 
        compose->replyinfo = NULL;
@@ -1184,21 +1254,17 @@ static Compose *compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                reply_account = account;
 
        compose = compose_create(account, COMPOSE_REPLY, FALSE);
+
+       compose->updating = TRUE;
+
        ifactory = gtk_item_factory_from_widget(compose->menubar);
 
        menu_set_active(ifactory, "/Options/Remove references", FALSE);
        menu_set_sensitive(ifactory, "/Options/Remove references", TRUE);
 
-       compose->updating = TRUE;
        compose->replyinfo = procmsg_msginfo_get_full_info(msginfo);
        if (!compose->replyinfo)
                compose->replyinfo = procmsg_msginfo_copy(msginfo);
-       compose->updating = FALSE;
-
-       if (compose->deferred_destroy) {
-               compose_destroy(compose);
-               return NULL;
-       }
 
        compose_extract_original_charset(compose);
        
@@ -1268,6 +1334,14 @@ static Compose *compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                
        compose->modified = FALSE;
        compose_set_title(compose);
+
+       compose->updating = FALSE;
+
+       if (compose->deferred_destroy) {
+               compose_destroy(compose);
+               return NULL;
+       }
+
        return compose;
 }
 
@@ -1302,12 +1376,6 @@ Compose *compose_forward(PrefsAccount *account, MsgInfo *msginfo,
        compose->fwdinfo = procmsg_msginfo_get_full_info(msginfo);
        if (!compose->fwdinfo)
                compose->fwdinfo = procmsg_msginfo_copy(msginfo);
-       compose->updating = FALSE;
-
-       if (compose->deferred_destroy) {
-               compose_destroy(compose);
-               return NULL;
-       }
 
        compose_extract_original_charset(compose);
 
@@ -1345,16 +1413,9 @@ Compose *compose_forward(PrefsAccount *account, MsgInfo *msginfo,
                gchar *qmark;
                MsgInfo *full_msginfo;
 
-               compose->updating = TRUE;
                full_msginfo = procmsg_msginfo_get_full_info(msginfo);
                if (!full_msginfo)
                        full_msginfo = procmsg_msginfo_copy(msginfo);
-               compose->updating = FALSE;
-
-               if (compose->deferred_destroy) {
-                       compose_destroy(compose);
-                       return NULL;
-               }
 
                if (prefs_common.fw_quotemark &&
                    *prefs_common.fw_quotemark)
@@ -1401,6 +1462,14 @@ Compose *compose_forward(PrefsAccount *account, MsgInfo *msginfo,
        
        compose->modified = FALSE;
        compose_set_title(compose);
+
+       compose->updating = FALSE;
+
+       if (compose->deferred_destroy) {
+               compose_destroy(compose);
+               return NULL;
+       }
+
         return compose;
 }
 
@@ -1440,20 +1509,16 @@ Compose *compose_forward_multiple(PrefsAccount *account, GSList *msginfo_list)
 
        compose = compose_create(account, COMPOSE_FORWARD, FALSE);
 
+       compose->updating = TRUE;
+
        textview = GTK_TEXT_VIEW(compose->text);
        textbuf = gtk_text_view_get_buffer(textview);
        compose_create_tags(textview, compose);
        
        undo_block(compose->undostruct);
        for (msginfo = msginfo_list; msginfo != NULL; msginfo = msginfo->next) {
-               compose->updating = TRUE;
                msgfile = procmsg_get_message_file_path((MsgInfo *)msginfo->data);
-               compose->updating = FALSE;
-               if (compose->deferred_destroy) {
-                       compose_destroy(compose);
-                       g_free(msgfile);
-                       return NULL;
-               }
+
                if (!is_file_exist(msgfile))
                        g_warning("%s: file not exist\n", msgfile);
                else
@@ -1498,6 +1563,14 @@ Compose *compose_forward_multiple(PrefsAccount *account, GSList *msginfo_list)
        undo_unblock(compose->undostruct);
        compose->modified = FALSE;
        compose_set_title(compose);
+
+       compose->updating = FALSE;
+
+       if (compose->deferred_destroy) {
+               compose_destroy(compose);
+               return NULL;
+       }
+
        return compose;
 }
 
@@ -1548,7 +1621,7 @@ static void compose_colorize_signature(Compose *compose)
                }
 }
 
-void compose_reedit(MsgInfo *msginfo)
+Compose *compose_reedit(MsgInfo *msginfo, gboolean batch)
 {
        Compose *compose = NULL;
        PrefsAccount *account = NULL;
@@ -1562,11 +1635,11 @@ void compose_reedit(MsgInfo *msginfo)
        gboolean use_encryption = FALSE;
        gchar *privacy_system = NULL;
 
-       g_return_if_fail(msginfo != NULL);
-       g_return_if_fail(msginfo->folder != NULL);
+       g_return_val_if_fail(msginfo != NULL, NULL);
+       g_return_val_if_fail(msginfo->folder != NULL, NULL);
 
        if (compose_put_existing_to_front(msginfo)) {
-               return;
+               return NULL;
        }
 
         if (folder_has_parent_of_type(msginfo->folder, F_QUEUE) ||
@@ -1628,9 +1701,12 @@ void compose_reedit(MsgInfo *msginfo)
         if (!account) {
                account = cur_account;
         }
-       g_return_if_fail(account != NULL);
+       g_return_val_if_fail(account != NULL, NULL);
 
-       compose = compose_create(account, COMPOSE_REEDIT, FALSE);
+       compose = compose_create(account, COMPOSE_REEDIT, batch);
+       
+       compose->updating = TRUE;
+       
        if (privacy_system != NULL) {
                compose->privacy_system = privacy_system;
                compose_use_signing(compose, use_signing);
@@ -1640,14 +1716,7 @@ void compose_reedit(MsgInfo *msginfo)
                activate_privacy_system(compose, account, FALSE);
        }
 
-       compose->updating = TRUE;
        compose->targetinfo = procmsg_msginfo_copy(msginfo);
-       compose->updating = FALSE;
-       
-       if (compose->deferred_destroy) {
-               compose_destroy(compose);
-               return;
-       }
 
        compose_extract_original_charset(compose);
 
@@ -1674,7 +1743,9 @@ void compose_reedit(MsgInfo *msginfo)
        }
        
        if (compose_parse_header(compose, msginfo) < 0) {
-               return;
+               compose->updating = FALSE;
+               compose_destroy(compose);
+               return NULL;
        }
        compose_reedit_set_entry(compose, msginfo);
 
@@ -1729,6 +1800,14 @@ void compose_reedit(MsgInfo *msginfo)
        }
        compose->modified = FALSE;
        compose_set_title(compose);
+
+       compose->updating = FALSE;
+
+       if (compose->deferred_destroy) {
+               compose_destroy(compose);
+               return NULL;
+       }
+       return compose;
 }
 
 Compose *compose_redirect(PrefsAccount *account, MsgInfo *msginfo,
@@ -1747,6 +1826,9 @@ Compose *compose_redirect(PrefsAccount *account, MsgInfo *msginfo,
        g_return_val_if_fail(account != NULL, NULL);
 
        compose = compose_create(account, COMPOSE_REDIRECT, batch);
+
+       compose->updating = TRUE;
+
        ifactory = gtk_item_factory_from_widget(compose->menubar);
        compose_create_tags(GTK_TEXT_VIEW(compose->text), compose);
        compose->replyinfo = NULL;
@@ -1756,18 +1838,14 @@ Compose *compose_redirect(PrefsAccount *account, MsgInfo *msginfo,
 
        gtk_widget_grab_focus(compose->header_last->entry);
 
-       compose->updating = TRUE;
        filename = procmsg_get_message_file_path(msginfo);
-       compose->updating = FALSE;
 
-       if (compose->deferred_destroy) {
+       if (filename == NULL) {
+               compose->updating = FALSE;
                compose_destroy(compose);
-               g_free(filename);
-               return NULL;
-       }
 
-       if (filename == NULL)
                return NULL;
+       }
 
        compose->redirect_filename = filename;
        
@@ -1826,7 +1904,14 @@ Compose *compose_redirect(PrefsAccount *account, MsgInfo *msginfo,
 
        compose->modified = FALSE;
        compose_set_title(compose);
-        return compose;
+       compose->updating = FALSE;
+
+       if (compose->deferred_destroy) {
+               compose_destroy(compose);
+               return NULL;
+       }
+       
+       return compose;
 }
 
 GList *compose_get_compose_list(void)
@@ -1878,7 +1963,7 @@ void compose_entry_append(Compose *compose, const gchar *address,
                        tmp[cur-begin]='\0';
                        cur++;
                        begin = cur;
-                       while (*tmp == ' ')
+                       while (*tmp == ' ' || *tmp == '\t')
                                tmp++;
                        compose_add_header_entry(compose, header, tmp);
                        g_free(o_tmp);
@@ -1892,7 +1977,7 @@ void compose_entry_append(Compose *compose, const gchar *address,
                tmp[cur-begin]='\0';
                cur++;
                begin = cur;
-               while (*tmp == ' ')
+               while (*tmp == ' ' || *tmp == '\t')
                        tmp++;
                compose_add_header_entry(compose, header, tmp);
                g_free(o_tmp);          
@@ -2285,7 +2370,7 @@ static gchar *compose_quote_fmt(Compose *compose, MsgInfo *msginfo,
        } else {
                gchar *tmpout = NULL;
                tmpout = conv_codeset_strdup
-                       (buf, conv_get_locale_charset_str(),
+                       (buf, conv_get_locale_charset_str_no_utf8(),
                         CS_INTERNAL);
                if (!tmpout || !g_utf8_validate(tmpout, -1, NULL)) {
                        g_free(tmpout);
@@ -2430,6 +2515,8 @@ static void compose_reply_set_entry(Compose *compose, MsgInfo *msginfo,
                                msginfo->folder->prefs->default_reply_to);
                } else {
                        gchar *tmp1 = NULL;
+                       if (!msginfo->from)
+                               return;
                        Xstrdup_a(tmp1, msginfo->from, return);
                        extract_address(tmp1);
                        if (to_all || to_sender ||
@@ -2568,6 +2655,7 @@ static void compose_reedit_set_entry(Compose *compose, MsgInfo *msginfo)
        g_return_if_fail(msginfo != NULL);
 
        SET_ENTRY(subject_entry, msginfo->subject);
+       SET_ENTRY(from_name, msginfo->from);
        SET_ADDRESS(COMPOSE_TO, msginfo->to);
        SET_ADDRESS(COMPOSE_CC, compose->cc);
        SET_ADDRESS(COMPOSE_BCC, compose->bcc);
@@ -2707,7 +2795,7 @@ static gchar *compose_get_signature_str(Compose *compose)
                        utf8_sig_str = sig_str;
                else {
                        utf8_sig_str = conv_codeset_strdup
-                               (sig_str, conv_get_locale_charset_str(),
+                               (sig_str, conv_get_locale_charset_str_no_utf8(),
                                 CS_INTERNAL);
                        g_free(sig_str);
                }
@@ -2748,7 +2836,7 @@ static ComposeInsertResult compose_insert_file(Compose *compose, const gchar *fi
                                        G_CALLBACK(text_inserted),
                                        compose);
 
-       cur_encoding = conv_get_locale_charset_str();
+       cur_encoding = conv_get_locale_charset_str_no_utf8();
 
        while (fgets(buf, sizeof(buf), fp) != NULL) {
                gchar *str;
@@ -3357,6 +3445,7 @@ static void compose_beautify_paragraph(Compose *compose, GtkTextIter *par_iter,
        gint uri_start = -1, uri_stop = -1;
        gint nouri_start = -1, nouri_stop = -1;
        gint num_blocks = 0;
+       gint quotelevel = -1;
 
        compose->autowrap = FALSE;
 
@@ -3444,9 +3533,18 @@ static void compose_beautify_paragraph(Compose *compose, GtkTextIter *par_iter,
                        debug_print("compose_beautify_paragraph(): quote_str = '%s'\n", quote_str);
                        if (startq_offset == -1) 
                                startq_offset = gtk_text_iter_get_offset(&iter);
+                       quotelevel = get_quote_level(quote_str, prefs_common.quote_chars);
+                       if (quotelevel > 2) {
+                               /* recycle colors */
+                               if (prefs_common.recycle_quote_colors)
+                                       quotelevel %= 3;
+                               else
+                                       quotelevel = 2;
+                       }
                } else {
                        if (startq_offset == -1)
                                noq_offset = gtk_text_iter_get_offset(&iter);
+                       quotelevel = -1;
                }
 
                if (prev_autowrap == FALSE && !force && !wrap_quote) {
@@ -3560,8 +3658,17 @@ colorize:
                        gtk_text_buffer_get_iter_at_offset(
                                buffer, &startquote, startq_offset);
                        endquote = iter;
-                       gtk_text_buffer_apply_tag_by_name(
-                               buffer, "quote", &startquote, &endquote);
+                       switch (quotelevel) {
+                       case 0: gtk_text_buffer_apply_tag_by_name(
+                                       buffer, "quote0", &startquote, &endquote);
+                               break;
+                       case 1: gtk_text_buffer_apply_tag_by_name(
+                                       buffer, "quote1", &startquote, &endquote);
+                               break;
+                       case 2: gtk_text_buffer_apply_tag_by_name(
+                                       buffer, "quote2", &startquote, &endquote);
+                               break;
+                       }
                        startq_offset = -1;
                } else if (noq_offset != -1) {
                        GtkTextIter startnoquote, endnoquote;
@@ -3569,7 +3676,11 @@ colorize:
                                buffer, &startnoquote, noq_offset);
                        endnoquote = iter;
                        gtk_text_buffer_remove_tag_by_name(
-                               buffer, "quote", &startnoquote, &endnoquote);
+                               buffer, "quote0", &startnoquote, &endnoquote);
+                       gtk_text_buffer_remove_tag_by_name(
+                               buffer, "quote1", &startnoquote, &endnoquote);
+                       gtk_text_buffer_remove_tag_by_name(
+                               buffer, "quote2", &startnoquote, &endnoquote);
                        noq_offset = -1;
                }
                
@@ -3672,15 +3783,84 @@ compose_current_mail_account(void)
        return ac;
 }
 
+#define QUOTE_IF_REQUIRED(out, str)                                    \
+{                                                                      \
+       if (*str != '"' && strpbrk(str, ",.[]<>")) {                    \
+               gchar *__tmp;                                           \
+               gint len;                                               \
+                                                                       \
+               len = strlen(str) + 3;                                  \
+               if ((__tmp = alloca(len)) == NULL) {                    \
+                       g_warning("can't allocate memory\n");           \
+                       g_string_free(header, TRUE);                    \
+                       return NULL;                                    \
+               }                                                       \
+               g_snprintf(__tmp, len, "\"%s\"", str);                  \
+               out = __tmp;                                            \
+       } else {                                                        \
+               gchar *__tmp;                                           \
+                                                                       \
+               if ((__tmp = alloca(strlen(str) + 1)) == NULL) {        \
+                       g_warning("can't allocate memory\n");           \
+                       g_string_free(header, TRUE);                    \
+                       return NULL;                                    \
+               } else                                                  \
+                       strcpy(__tmp, str);                             \
+                                                                       \
+               out = __tmp;                                            \
+       }                                                               \
+}
+
+#define QUOTE_IF_REQUIRED_NORMAL(out, str, errret)                     \
+{                                                                      \
+       if (*str != '"' && strpbrk(str, ",.[]<>")) {                    \
+               gchar *__tmp;                                           \
+               gint len;                                               \
+                                                                       \
+               len = strlen(str) + 3;                                  \
+               if ((__tmp = alloca(len)) == NULL) {                    \
+                       g_warning("can't allocate memory\n");           \
+                       errret;                                         \
+               }                                                       \
+               g_snprintf(__tmp, len, "\"%s\"", str);                  \
+               out = __tmp;                                            \
+       } else {                                                        \
+               gchar *__tmp;                                           \
+                                                                       \
+               if ((__tmp = alloca(strlen(str) + 1)) == NULL) {        \
+                       g_warning("can't allocate memory\n");           \
+                       errret;                                         \
+               } else                                                  \
+                       strcpy(__tmp, str);                             \
+                                                                       \
+               out = __tmp;                                            \
+       }                                                               \
+}
+
 static void compose_select_account(Compose *compose, PrefsAccount *account,
                                   gboolean init)
 {
        GtkItemFactory *ifactory;
+       gchar *from = NULL;
 
        g_return_if_fail(account != NULL);
 
        compose->account = account;
 
+       if (account->name && *account->name) {
+               gchar *buf;
+               QUOTE_IF_REQUIRED_NORMAL(buf, account->name, return);
+               from = g_strdup_printf("%s <%s>",
+                                      buf, account->address);
+               gtk_entry_set_text(GTK_ENTRY(compose->from_name), from);
+       } else {
+               from = g_strdup_printf("<%s>",
+                                      account->address);
+               gtk_entry_set_text(GTK_ENTRY(compose->from_name), from);
+       }
+
+       g_free(from);
+
        compose_set_title(compose);
 
        ifactory = gtk_item_factory_from_widget(compose->menubar);
@@ -3766,6 +3946,9 @@ static gboolean compose_check_for_set_recipients(Compose *compose)
                }
                if (!found_other) {
                        AlertValue aval;
+                       if (compose->batch) {
+                               gtk_widget_show_all(compose->window);
+                       }
                        aval = alertpanel(_("Send"),
                                          _("The only recipient is the default CC address. Send anyway?"),
                                          GTK_STOCK_CANCEL, _("+_Send"), NULL);
@@ -3794,6 +3977,9 @@ static gboolean compose_check_for_set_recipients(Compose *compose)
                }
                if (!found_other) {
                        AlertValue aval;
+                       if (compose->batch) {
+                               gtk_widget_show_all(compose->window);
+                       }
                        aval = alertpanel(_("Send"),
                                          _("The only recipient is the default BCC address. Send anyway?"),
                                          GTK_STOCK_CANCEL, _("+_Send"), NULL);
@@ -3809,6 +3995,9 @@ static gboolean compose_check_entries(Compose *compose, gboolean check_subject)
        const gchar *str;
 
        if (compose_check_for_valid_recipient(compose) == FALSE) {
+               if (compose->batch) {
+                       gtk_widget_show_all(compose->window);
+               }
                alertpanel_error(_("Recipient is not specified."));
                return FALSE;
        }
@@ -3836,11 +4025,11 @@ static gboolean compose_check_entries(Compose *compose, gboolean check_subject)
 gint compose_send(Compose *compose)
 {
        gint msgnum;
-       FolderItem *folder;
+       FolderItem *folder = NULL;
        gint val = -1;
        gchar *msgpath = NULL;
        gboolean discard_window = FALSE;
-
+       gchar *errstr = NULL;
        if (prefs_common.send_dialog_mode != SEND_DIALOG_ALWAYS
                        || compose->batch == TRUE)
                discard_window = TRUE;
@@ -3855,7 +4044,7 @@ gint compose_send(Compose *compose)
                goto bail;
        }
 
-       val = compose_queue(compose, &msgnum, &folder, &msgpath);
+       val = compose_queue(compose, &msgnum, &folder, &msgpath, TRUE);
 
        if (val) {
                if (compose->batch) {
@@ -3866,7 +4055,7 @@ gint compose_send(Compose *compose)
                                           "Charset conversion failed."));
                } else if (val == -3) {
                        alertpanel_error(_("Could not queue message for sending:\n\n"
-                                          "Signature failed."));
+                                          "Signature failed: %s"), privacy_get_error());
                } else if (val == -2 && errno != 0) {
                        alertpanel_error(_("Could not queue message for sending:\n\n%s."), strerror(errno));
                } else {
@@ -3884,21 +4073,21 @@ gint compose_send(Compose *compose)
        }
 
        if (msgnum == 0) {
-               if (!discard_window) {
-                       goto bail;
-               }
                alertpanel_error(_("The message was queued but could not be "
                                   "sent.\nUse \"Send queued messages\" from "
                                   "the main window to retry."));
+               if (!discard_window) {
+                       goto bail;
+               }
                return -1;
        }
        
        if (msgpath == NULL) {
                msgpath = folder_item_fetch_msg(folder, msgnum);
-               val = procmsg_send_message_queue(msgpath);
+               val = procmsg_send_message_queue(msgpath, &errstr);
                g_free(msgpath);
        } else {
-               val = procmsg_send_message_queue(msgpath);
+               val = procmsg_send_message_queue(msgpath, &errstr);
                g_unlink(msgpath);
                g_free(msgpath);
        }
@@ -3917,12 +4106,20 @@ gint compose_send(Compose *compose)
                if (!discard_window)
                        compose_close(compose);
        } else {
+               if (errstr) {
+                       gchar *tmp = g_strdup_printf(_("%s\nUse \"Send queued messages\" from "
+                                  "the main window to retry."), errstr);
+                       g_free(errstr);
+                       alertpanel_error_log(tmp);
+                       g_free(tmp);
+               } else {
+                       alertpanel_error_log(_("The message was queued but could not be "
+                                  "sent.\nUse \"Send queued messages\" from "
+                                  "the main window to retry."));
+               }
                if (!discard_window) {
                        goto bail;              
                }
-               alertpanel_error(_("The message was queued but could not be "
-                                  "sent.\nUse \"Send queued messages\" from "
-                                  "the main window to retry."));
                return -1;
        }
 
@@ -4307,8 +4504,7 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
                mimempart->subtype = g_strdup("mixed");
 
                do {
-                       if (boundary)
-                               g_free(boundary);
+                       g_free(boundary);
                        boundary = generate_mime_boundary(NULL);
                } while (strstr(buf, boundary) != NULL);
 
@@ -4426,11 +4622,14 @@ void compose_remove_draft(Compose *compose)
 
 }
 
-static gint compose_queue(Compose *compose, gint *msgnum, FolderItem **item, gchar **msgpath)
+gint compose_queue(Compose *compose, gint *msgnum, FolderItem **item, gchar **msgpath,
+                  gboolean remove_reedit_target)
 {
-       return compose_queue_sub (compose, msgnum, item, msgpath, FALSE);
+       return compose_queue_sub (compose, msgnum, item, msgpath, FALSE, remove_reedit_target);
 }
-static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item, gchar **msgpath, gboolean check_subject)
+static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item, 
+                             gchar **msgpath, gboolean check_subject,
+                             gboolean remove_reedit_target)
 {
        FolderItem *queue;
        gchar *tmp;
@@ -4447,6 +4646,9 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
        
        if (compose_check_entries(compose, check_subject) == FALSE) {
                 lock = FALSE;
+               if (compose->batch) {
+                       gtk_widget_show_all(compose->window);
+               }
                 return -1;
        }
 
@@ -4484,6 +4686,7 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
        if ((fp = g_fopen(tmp, "wb")) == NULL) {
                FILE_OP_ERROR(tmp, "fopen");
                g_free(tmp);
+               lock = FALSE;
                return -2;
        }
 
@@ -4538,13 +4741,20 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
        if (newsac)
                fprintf(fp, "NAID:%d\n", newsac->account_id);
 
+       
        if (compose->privacy_system != NULL) {
                fprintf(fp, "X-Sylpheed-Privacy-System:%s\n", compose->privacy_system);
                fprintf(fp, "X-Sylpheed-Sign:%d\n", compose->use_signing);
                if (compose->use_encryption) {
                        gchar *encdata;
-
-                       encdata = privacy_get_encrypt_data(compose->privacy_system, compose->to_list);
+                       if (mailac && mailac->encrypt_to_self) {
+                               GSList *tmp_list = g_slist_copy(compose->to_list);
+                               tmp_list = g_slist_append(tmp_list, compose->account->address);
+                               encdata = privacy_get_encrypt_data(compose->privacy_system, tmp_list);
+                               g_slist_free(tmp_list);
+                       } else {
+                               encdata = privacy_get_encrypt_data(compose->privacy_system, compose->to_list);
+                       }
                        if (encdata != NULL) {
                                if (strcmp(encdata, "_DONT_ENCRYPT_")) {
                                        fprintf(fp, "X-Sylpheed-Encrypt:%d\n", compose->use_encryption);
@@ -4588,7 +4798,9 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
                fprintf(fp, "FMID:%s\t%d\t%s\n", folderid, compose->fwdinfo->msgnum, compose->fwdinfo->msgid);
                g_free(folderid);
        }
-       fprintf(fp, "\n");
+
+       /* end of headers */
+       fprintf(fp, "X-Sylpheed-End-Special-Headers: 1\n");
 
        if (compose->redirect_filename != NULL) {
                if (compose_redirect_write_to_file(compose, fp) < 0) {
@@ -4613,14 +4825,20 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
                FILE_OP_ERROR(tmp, "fclose");
                g_unlink(tmp);
                g_free(tmp);
+               lock = FALSE;
                return -2;
        }
 
-       queue = account_get_special_folder(compose->account, F_QUEUE);
+       if (item && *item) {
+               queue = *item;
+       } else {
+               queue = account_get_special_folder(compose->account, F_QUEUE);
+       }
        if (!queue) {
                g_warning("can't find queue folder\n");
                g_unlink(tmp);
                g_free(tmp);
+               lock = FALSE;
                return -1;
        }
        folder_item_scan(queue);
@@ -4628,6 +4846,7 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
                g_warning("can't queue the message\n");
                g_unlink(tmp);
                g_free(tmp);
+               lock = FALSE;
                return -1;
        }
        
@@ -4637,7 +4856,7 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
        } else
                *msgpath = tmp;
 
-       if (compose->mode == COMPOSE_REEDIT) {
+       if (compose->mode == COMPOSE_REEDIT && remove_reedit_target) {
                compose_remove_reedit_target(compose, FALSE);
        }
 
@@ -4714,34 +4933,6 @@ static void compose_add_attachments(Compose *compose, MimeInfo *parent)
        } while (gtk_tree_model_iter_next(model, &iter));
 }
 
-#define QUOTE_IF_REQUIRED(out, str)                                    \
-{                                                                      \
-       if (*str != '"' && strpbrk(str, ",.[]<>")) {                    \
-               gchar *__tmp;                                           \
-               gint len;                                               \
-                                                                       \
-               len = strlen(str) + 3;                                  \
-               if ((__tmp = alloca(len)) == NULL) {                    \
-                       g_warning("can't allocate memory\n");           \
-                       g_string_free(header, TRUE);                    \
-                       return NULL;                                    \
-               }                                                       \
-               g_snprintf(__tmp, len, "\"%s\"", str);                  \
-               out = __tmp;                                            \
-       } else {                                                        \
-               gchar *__tmp;                                           \
-                                                                       \
-               if ((__tmp = alloca(strlen(str) + 1)) == NULL) {        \
-                       g_warning("can't allocate memory\n");           \
-                       g_string_free(header, TRUE);                    \
-                       return NULL;                                    \
-               } else                                                  \
-                       strcpy(__tmp, str);                             \
-                                                                       \
-               out = __tmp;                                            \
-       }                                                               \
-}
-
 #define IS_IN_CUSTOM_HEADER(header) \
        (compose->account->add_customhdr && \
         custom_header_find(compose->account->customhdr_list, header) != NULL)
@@ -4810,6 +5001,8 @@ static gchar *compose_get_header(Compose *compose)
        GSList *list;
        gchar *std_headers[] = {"To:", "Cc:", "Bcc:", "Newsgroups:", "Reply-To:", "Followup-To:", NULL};
        GString *header;
+       gchar *from_name = NULL, *from_address = NULL;
+       gchar *tmp;
 
        g_return_val_if_fail(compose->account != NULL, NULL);
        g_return_val_if_fail(compose->account->address != NULL, NULL);
@@ -4823,16 +5016,52 @@ static gchar *compose_get_header(Compose *compose)
        }
 
        /* From */
+       
        if (compose->account->name && *compose->account->name) {
+               gchar *buf;
+               QUOTE_IF_REQUIRED(buf, compose->account->name);
+               tmp = g_strdup_printf("%s <%s>",
+                       buf, compose->account->address);
+       } else {
+               tmp = g_strdup_printf("%s",
+                       compose->account->address);
+       }
+       if (!strcmp(gtk_entry_get_text(GTK_ENTRY(compose->from_name)), tmp)
+       ||  strlen(gtk_entry_get_text(GTK_ENTRY(compose->from_name))) == 0) {
+               /* use default */
+               from_name = compose->account->name ? g_strdup(compose->account->name):NULL;
+               from_address = g_strdup(compose->account->address);
+       } else {
+               gchar *spec = gtk_editable_get_chars(GTK_EDITABLE(compose->from_name), 0, -1);
+               /* extract name and address */
+               if (strstr(spec, " <") && strstr(spec, ">")) {
+                       from_address = g_strdup(strrchr(spec, '<')+1);
+                       *(strrchr(from_address, '>')) = '\0';
+                       from_name = g_strdup(spec);
+                       *(strrchr(from_name, '<')) = '\0';
+               } else {
+                       from_name = NULL;
+                       from_address = g_strdup(spec);
+               }
+               g_free(spec);
+       }
+       g_free(tmp);
+       
+       
+       if (from_name && *from_name) {
                compose_convert_header
-                       (compose, buf, sizeof(buf), compose->account->name,
+                       (compose, buf, sizeof(buf), from_name,
                         strlen("From: "), TRUE);
                QUOTE_IF_REQUIRED(name, buf);
+               
                g_string_append_printf(header, "From: %s <%s>\n",
-                       name, compose->account->address);
+                       name, from_address);
        } else
-               g_string_append_printf(header, "From: %s\n", compose->account->address);
+               g_string_append_printf(header, "From: %s\n", from_address);
        
+       g_free(from_name);
+       g_free(from_address);
+
        /* To */
        compose_add_headerfield_from_headerlist(compose, header, "To", ", ");
 
@@ -5054,7 +5283,8 @@ static void compose_convert_header(Compose *compose, gchar *dest, gint len, gcha
                        test_conv_global_out = conv_codeset_strdup(src, CS_INTERNAL, out_codeset);
                }
 
-               if (!test_conv_global_out && compose->orig_charset) {
+               if (!test_conv_global_out && compose->orig_charset
+               &&  strcmp(compose->orig_charset, CS_US_ASCII)) {
                        out_codeset = compose->orig_charset;
                        debug_print("failure; trying to convert to %s\n", out_codeset);
                        test_conv_reply = conv_codeset_strdup(src, CS_INTERNAL, out_codeset);
@@ -5132,6 +5362,8 @@ static void compose_create_header_entry(Compose *compose)
        /* Entry field */
        entry = gtk_entry_new(); 
        gtk_widget_show(entry);
+       gtk_tooltips_set_tip(compose->tooltips, entry,
+               _("Use <tab> to autocomplete from addressbook"), NULL);
        gtk_table_attach(GTK_TABLE(compose->header_table), entry, 1, 2, compose->header_nextrow, compose->header_nextrow+1, GTK_EXPAND | GTK_FILL, GTK_FILL, 0, 0);
 
         g_signal_connect(G_OBJECT(entry), "key-press-event", 
@@ -5177,8 +5409,6 @@ static void compose_add_header_entry(Compose *compose, gchar *header, gchar *tex
 
 static GtkWidget *compose_create_header(Compose *compose) 
 {
-       GtkWidget *label;
-       GtkWidget *hbox;
        GtkWidget *from_optmenu_hbox;
        GtkWidget *header_scrolledwin;
        GtkWidget *header_table;
@@ -5192,20 +5422,15 @@ static GtkWidget *compose_create_header(Compose *compose)
 
        header_table = gtk_table_new(2, 2, FALSE);
        gtk_widget_show(header_table);
-       gtk_container_set_border_width(GTK_CONTAINER(header_table), 2);
+       gtk_container_set_border_width(GTK_CONTAINER(header_table), BORDER_WIDTH);
        gtk_scrolled_window_add_with_viewport(GTK_SCROLLED_WINDOW(header_scrolledwin), header_table);
        gtk_viewport_set_shadow_type(GTK_VIEWPORT(GTK_BIN(header_scrolledwin)->child), GTK_SHADOW_ETCHED_IN);
        count = 0;
 
        /* option menu for selecting accounts */
-       hbox = gtk_hbox_new(FALSE, 0);
-       label = gtk_label_new(prefs_common.trans_hdr ? _("From:") : "From:");
-       gtk_box_pack_end(GTK_BOX(hbox), label, FALSE, FALSE, 0);
-       gtk_table_attach(GTK_TABLE(header_table), hbox, 0, 1, count, count + 1,
-                        GTK_FILL, 0, 2, 0);
        from_optmenu_hbox = compose_account_option_menu_create(compose);
        gtk_table_attach(GTK_TABLE(header_table), from_optmenu_hbox,
-                                 1, 2, count, count + 1, GTK_EXPAND | GTK_FILL, GTK_SHRINK, 0, 0);
+                                 0, 2, count, count + 1, GTK_EXPAND | GTK_FILL, GTK_SHRINK, 0, 0);
        count++;
 
        compose->header_table = header_table;
@@ -5311,6 +5536,7 @@ static GtkWidget *compose_create_others(Compose *compose)
 
        /* Table for settings */
        table = gtk_table_new(3, 1, FALSE);
+       gtk_container_set_border_width(GTK_CONTAINER(table), BORDER_WIDTH);
        gtk_widget_show(table);
        gtk_table_set_row_spacings(GTK_TABLE(table), VSPACING_NARROW);
        rowcount = 0;
@@ -5457,6 +5683,49 @@ static gboolean text_clicked(GtkWidget *text, GdkEventButton *event,
        return FALSE;
 }
 
+#if USE_ASPELL
+static void compose_spell_menu_changed(void *data)
+{
+       Compose *compose = (Compose *)data;
+       GSList *items;
+       GtkWidget *menuitem;
+       GtkWidget *parent_item;
+       GtkMenu *menu = GTK_MENU(gtk_menu_new());
+       GtkItemFactory *ifactory = gtk_item_factory_from_widget(compose->menubar);
+       GSList *spell_menu;
+
+       if (compose->gtkaspell == NULL)
+               return;
+
+       parent_item = gtk_item_factory_get_item(ifactory, 
+                       "/Spelling/Options");
+
+       /* setting the submenu removes /Spelling/Options from the factory 
+        * so we need to save it */
+
+       if (parent_item == NULL) {
+               parent_item = compose->aspell_options_menu;
+               gtk_menu_item_remove_submenu(GTK_MENU_ITEM(parent_item));
+       } else
+               compose->aspell_options_menu = parent_item;
+
+       spell_menu = gtkaspell_make_config_menu(compose->gtkaspell);
+
+       spell_menu = g_slist_reverse(spell_menu);
+       for (items = spell_menu;
+            items; items = items->next) {
+               menuitem = GTK_WIDGET(GTK_MENU_ITEM(items->data));
+               gtk_menu_shell_prepend(GTK_MENU_SHELL(menu), GTK_WIDGET(menuitem));
+               gtk_widget_show(GTK_WIDGET(menuitem));
+       }
+       g_slist_free(spell_menu);
+
+       gtk_menu_item_set_submenu(GTK_MENU_ITEM(parent_item), GTK_WIDGET(menu));
+       
+}
+#endif
+
+static gboolean compose_force_window_origin = TRUE;
 static Compose *compose_create(PrefsAccount *account, ComposeMode mode,
                                                 gboolean batch)
 {
@@ -5489,9 +5758,6 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode,
 
        gchar *titles[N_ATTACH_COLS];
        guint n_menu_entries;
-       GdkColormap *cmap;
-       GdkColor color[1];
-       gboolean success[1];
        GtkWidget *popupmenu;
        GtkItemFactory *popupfactory;
        GtkItemFactory *ifactory;
@@ -5520,6 +5786,8 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode,
        compose->mutex = g_mutex_new();
        compose->set_cursor_pos = -1;
 
+       compose->tooltips = gtk_tooltips_new();
+
        window = gtk_window_new(GTK_WINDOW_TOPLEVEL);
        gtk_window_set_resizable(GTK_WINDOW(window), TRUE);
        gtk_widget_set_size_request(window, -1, prefs_common.compose_height);
@@ -5537,7 +5805,9 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode,
        }
        gtk_window_set_geometry_hints(GTK_WINDOW(window), NULL,
                                      &geometry, GDK_HINT_MIN_SIZE);
-       gtk_widget_set_uposition(window, prefs_common.compose_x, 
+       
+       if (compose_force_window_origin)
+               gtk_widget_set_uposition(window, prefs_common.compose_x, 
                                 prefs_common.compose_y);
 
        g_signal_connect(G_OBJECT(window), "delete_event",
@@ -5691,17 +5961,6 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode,
                }
        }
 
-       color[0] = quote_color;
-       cmap = gdk_drawable_get_colormap(window->window);
-       gdk_colormap_alloc_colors(cmap, color, 1, FALSE, TRUE, success);
-       if (success[0] == FALSE) {
-               GtkStyle *style;
-
-               g_warning("Compose: color allocation failed.\n");
-               style = gtk_widget_get_style(text);
-               quote_color = style->black;
-       }
-
        n_entries = sizeof(compose_popup_entries) /
                sizeof(compose_popup_entries[0]);
        popupmenu = menu_create_items(compose_popup_entries, n_entries,
@@ -5787,7 +6046,7 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode,
        menu_set_sensitive(ifactory, "/Spelling", FALSE);
        if (mode != COMPOSE_REDIRECT) {
                if (prefs_common.enable_aspell && prefs_common.dictionary &&
-                   strcmp(prefs_common.dictionary, _("None"))) {
+                   strcmp(prefs_common.dictionary, "")) {
                        gtkaspell = gtkaspell_new(prefs_common.aspell_path,
                                                  prefs_common.dictionary,
                                                  conv_get_locale_charset_str(),
@@ -5796,7 +6055,9 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode,
                                                  prefs_common.recheck_when_changing_dict,
                                                  prefs_common.use_alternate,
                                                  GTK_TEXT_VIEW(text),
-                                                 GTK_WINDOW(compose->window));
+                                                 GTK_WINDOW(compose->window),
+                                                 compose_spell_menu_changed,
+                                                 compose);
                        if (!gtkaspell) {
                                alertpanel_error(_("Spell checker could not "
                                                "be started.\n%s"),
@@ -5816,6 +6077,7 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode,
                }
        }
         compose->gtkaspell = gtkaspell;
+       compose_spell_menu_changed(compose);
 #endif
 
        compose_select_account(compose, account, TRUE);
@@ -5880,35 +6142,54 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode,
 static GtkWidget *compose_account_option_menu_create(Compose *compose)
 {
        GList *accounts;
+       GtkWidget *hbox;
        GtkWidget *optmenu;
        GtkWidget *menu;
-       gint num = 0, def_menu = 0;
+       GtkWidget *from_name = NULL;
 
+       gint num = 0, def_menu = 0;
+       
        accounts = account_get_list();
        g_return_val_if_fail(accounts != NULL, NULL);
 
        optmenu = gtk_option_menu_new();
        menu = gtk_menu_new();
 
+       hbox = gtk_hbox_new(FALSE, 6);
+       from_name = gtk_entry_new();
+       
+       g_signal_connect_after(G_OBJECT(from_name), "grab_focus",
+                        G_CALLBACK(compose_grab_focus_cb), compose);
+
        for (; accounts != NULL; accounts = accounts->next, num++) {
                PrefsAccount *ac = (PrefsAccount *)accounts->data;
                GtkWidget *menuitem;
-               gchar *name;
+               gchar *name, *from = NULL;
 
                if (ac == compose->account) def_menu = num;
 
-               if (ac->name)
-                       name = g_markup_printf_escaped("<i>%s</i> : %s &lt;<b>%s</b>&gt;",
-                                              ac->account_name,
-                                              ac->name, ac->address);
-               else
-                       name = g_markup_printf_escaped("<i>%s</i> : &lt;<b>%s</b>&gt;",
-                                              ac->account_name, ac->address);
+               name = g_markup_printf_escaped("From: <i>%s</i>",
+                                      ac->account_name);
+               
+               if (ac == compose->account) {
+                       if (ac->name && *ac->name) {
+                               gchar *buf;
+                               QUOTE_IF_REQUIRED_NORMAL(buf, ac->name, return NULL);
+                               from = g_strdup_printf("%s <%s>",
+                                                      buf, ac->address);
+                               gtk_entry_set_text(GTK_ENTRY(from_name), from);
+                       } else {
+                               from = g_strdup_printf("%s",
+                                                      ac->address);
+                               gtk_entry_set_text(GTK_ENTRY(from_name), from);
+                       }
+               }
                MENUITEM_ADD(menu, menuitem, name, ac->account_id);
                gtk_label_set_use_markup (
                                GTK_LABEL (gtk_bin_get_child (GTK_BIN (menuitem))),
                                TRUE);
                g_free(name);
+               g_free(from);
                g_signal_connect(G_OBJECT(menuitem), "activate",
                                 G_CALLBACK(account_activated),
                                 compose);
@@ -5917,7 +6198,17 @@ static GtkWidget *compose_account_option_menu_create(Compose *compose)
        gtk_option_menu_set_menu(GTK_OPTION_MENU(optmenu), menu);
        gtk_option_menu_set_history(GTK_OPTION_MENU(optmenu), def_menu);
 
-       return optmenu;
+       gtk_box_pack_start(GTK_BOX(hbox), optmenu, FALSE, FALSE, 0);
+       gtk_box_pack_start(GTK_BOX(hbox), from_name, TRUE, TRUE, 0);
+       
+       gtk_tooltips_set_tip(compose->tooltips, optmenu,
+               _("Account to use for this email"), NULL);
+       gtk_tooltips_set_tip(compose->tooltips, from_name,
+               _("Sender address to be used"), NULL);
+
+       compose->from_name = from_name;
+       
+       return hbox;
 }
 
 static void compose_set_priority_cb(gpointer data,
@@ -6360,8 +6651,7 @@ static void compose_destroy(Compose *compose)
        g_free(compose->msgid);
        g_free(compose->boundary);
 
-       if (compose->redirect_filename)
-               g_free(compose->redirect_filename);
+       g_free(compose->redirect_filename);
        if (compose->undostruct)
                undo_destroy(compose->undostruct);
 
@@ -7221,11 +7511,19 @@ static void compose_send_later_cb(gpointer data, guint action,
        Compose *compose = (Compose *)data;
        gint val;
 
-       val = compose_queue_sub(compose, NULL, NULL, NULL, TRUE);
+       val = compose_queue_sub(compose, NULL, NULL, NULL, TRUE, TRUE);
        if (!val) 
                compose_close(compose);
-       else if (val == -2) {
+       else if (val == -1) {
+               alertpanel_error(_("Could not queue message."));
+       } else if (val == -2) {
                alertpanel_error(_("Could not queue message:\n\n%s."), strerror(errno));
+       } else if (val == -3) {
+               alertpanel_error(_("Could not queue message for sending:\n\n"
+                                  "Signature failed: %s"), privacy_get_error());
+       } else if (val == -4) {
+               alertpanel_error(_("Could not queue message for sending:\n\n"
+                                  "Charset conversion failed."));
        }
 }
 
@@ -7278,6 +7576,7 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
        /* Save draft infos */
        fprintf(fp, "X-Sylpheed-Account-Id:%d\n", compose->account->account_id);
        fprintf(fp, "S:%s\n", compose->account->address);
+
        if (gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(compose->savemsg_checkbtn))) {
                gchar *savefolderid;
 
@@ -7293,7 +7592,9 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
                fprintf(fp, "X-Sylpheed-Encrypt:%d\n", compose->use_encryption);
                fprintf(fp, "X-Sylpheed-Privacy-System:%s\n", compose->privacy_system);
        }
-       fprintf(fp, "\n");
+
+       /* end of headers */
+       fprintf(fp, "X-Sylpheed-End-Special-Headers: 1\n");
 
        if (compose_write_to_file(compose, fp, COMPOSE_WRITE_FOR_STORE) < 0) {
                fclose(fp);
@@ -7457,7 +7758,7 @@ static gint compose_delete_cb(GtkWidget *widget, GdkEventAny *event,
        prefs_common.compose_x = x;
        prefs_common.compose_y = y;
 
-       if (compose->sending)
+       if (compose->sending || compose->updating)
                return TRUE;
        compose_close_cb(compose, 0, NULL);
        return TRUE;
@@ -8493,7 +8794,7 @@ static PrefsAccount *compose_guess_forward_account_from_msginfo(MsgInfo *msginfo
        return account;
 }
 
-static gboolean compose_close(Compose *compose)
+gboolean compose_close(Compose *compose)
 {
        gint x, y;
 
@@ -8530,10 +8831,10 @@ static void compose_add_field_list( Compose *compose, GList *listAddress ) {
        }
 }
 
-void compose_reply_from_messageview(MessageView *msgview, GSList *msginfo_list, 
-                                   guint action)
+static void compose_reply_from_messageview_real(MessageView *msgview, GSList *msginfo_list, 
+                                   guint action, gboolean opening_multiple)
 {
-       gchar *body;
+       gchar *body = NULL;
        GSList *new_msglist = NULL;
        MsgInfo *tmp_msginfo = NULL;
        gboolean originally_enc = FALSE;
@@ -8543,7 +8844,7 @@ void compose_reply_from_messageview(MessageView *msgview, GSList *msginfo_list,
 
        g_return_if_fail(msginfo_list != NULL);
 
-       if (g_slist_length(msginfo_list) == 1) {
+       if (g_slist_length(msginfo_list) == 1 && !opening_multiple) {
                MimeInfo *mimeinfo = messageview_get_selected_mime_part(msgview);
                MsgInfo *orig_msginfo = (MsgInfo *)msginfo_list->data;
                
@@ -8560,7 +8861,8 @@ void compose_reply_from_messageview(MessageView *msgview, GSList *msginfo_list,
                }
        }
 
-       body = messageview_get_selection(msgview);
+       if (!opening_multiple)
+               body = messageview_get_selection(msgview);
 
        if (new_msglist) {
                compose = compose_reply_mode((ComposeMode)action, new_msglist, body);
@@ -8576,6 +8878,41 @@ void compose_reply_from_messageview(MessageView *msgview, GSList *msginfo_list,
        g_free(body);
 }
 
+void compose_reply_from_messageview(MessageView *msgview, GSList *msginfo_list, 
+                                   guint action)
+{
+       if ((!prefs_common.forward_as_attachment || action != COMPOSE_FORWARD) 
+       &&  action != COMPOSE_FORWARD_AS_ATTACH && g_slist_length(msginfo_list) > 1) {
+               GSList *cur = msginfo_list;
+               gchar *msg = g_strdup_printf(_("You are about to reply to %d "
+                                              "messages. Opening the windows "
+                                              "could take some time. Do you "
+                                              "want to continue?"), 
+                                              g_slist_length(msginfo_list));
+               if (g_slist_length(msginfo_list) > 9
+               &&  alertpanel(_("Warning"), msg, GTK_STOCK_CANCEL, "+" GTK_STOCK_YES, NULL)
+                   != G_ALERTALTERNATE) {
+                       g_free(msg);
+                       return;
+               }
+               g_free(msg);
+               /* We'll open multiple compose windows */
+               /* let the WM place the next windows */
+               compose_force_window_origin = FALSE;
+               for (; cur; cur = cur->next) {
+                       GSList tmplist;
+                       tmplist.data = cur->data;
+                       tmplist.next = NULL;
+                       compose_reply_from_messageview_real(msgview, &tmplist, action, TRUE);
+               }
+               compose_force_window_origin = TRUE;
+       } else {
+               /* forwarding multiple mails as attachments is done via a
+                * single compose window */
+               compose_reply_from_messageview_real(msgview, msginfo_list, action, FALSE);
+       }
+}
+
 void compose_set_position(Compose *compose, gint pos)
 {
        GtkTextView *text = GTK_TEXT_VIEW(compose->text);