0.8.8claws63
[claws.git] / src / compose.c
index 18db192614d49f3d3c62c97ad09a0f1b98a319e7..6a670b443a902e3b377cfdb342389dfb44233eb9 100644 (file)
@@ -164,11 +164,6 @@ Compose *compose_generic_new                       (PrefsAccount   *account,
 
 static Compose *compose_create                 (PrefsAccount   *account,
                                                 ComposeMode     mode);
-static void compose_toolbar_create             (Compose        *compose,
-                                                GtkWidget      *container);
-static void compose_toolbar_update              (Compose        *compose);
-static void compose_toolbar_set_sensitive       (Compose        *compose,
-                                                gboolean        sensitive);
 
 static GtkWidget *compose_account_option_menu_create
                                                (Compose        *compose);
@@ -296,33 +291,8 @@ static void compose_update_priority_menu_item(Compose * compose);
 static gboolean compose_edit_size_alloc (GtkEditable   *widget,
                                         GtkAllocation  *allocation,
                                         GtkSHRuler     *shruler);
-
-static void toolbar_send_cb            (GtkWidget      *widget,
-                                        gpointer        data);
-static void toolbar_send_later_cb      (GtkWidget      *widget,
-                                        gpointer        data);
-static void toolbar_draft_cb           (GtkWidget      *widget,
-                                        gpointer        data);
-static void toolbar_insert_cb          (GtkWidget      *widget,
-                                        gpointer        data);
-static void toolbar_attach_cb          (GtkWidget      *widget,
-                                        gpointer        data);
-static void toolbar_sig_cb             (GtkWidget      *widget,
-                                        gpointer        data);
-static void toolbar_ext_editor_cb      (GtkWidget      *widget,
-                                        gpointer        data);
-static void toolbar_linewrap_cb                (GtkWidget      *widget,
-                                        gpointer        data);
-static void toolbar_address_cb         (GtkWidget      *widget,
-                                        gpointer        data);
-static void toolbar_actions_execute_cb  (GtkWidget      *widget,
-                                        gpointer        data);
-static void toolbar_compose_buttons_cb  (GtkWidget      *widget, 
-
-                                        ToolbarItem    *t_item);
 static void account_activated          (GtkMenuItem    *menuitem,
                                         gpointer        data);
-
 static void attach_selected            (GtkCList       *clist,
                                         gint            row,
                                         gint            column,
@@ -837,44 +807,15 @@ static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                                  const gchar *body)
 {
        Compose *compose;
-       PrefsAccount *account;
+       PrefsAccount *account = NULL;
        PrefsAccount *reply_account;
        GtkSText *text;
 
        g_return_if_fail(msginfo != NULL);
        g_return_if_fail(msginfo->folder != NULL);
 
-       account = NULL;
-       /* select the account set in folderitem's property (if enabled) */
-       if (msginfo->folder->prefs && msginfo->folder->prefs->enable_default_account)
-               account = account_find_from_id(msginfo->folder->prefs->default_account);
+       account = account_get_reply_account(msginfo, prefs_common.reply_account_autosel);
        
-       /* select the account for the whole folder (IMAP / NNTP) */
-       if (!account)
-               /* FIXME: this is not right, because folder may be nested. we should
-                * ascend the tree until we find a parent with proper account 
-                * information */
-               account = msginfo->folder->folder->account;
-
-       /* select account by to: and cc: header if enabled */
-       if (prefs_common.reply_account_autosel) {
-               if (!account && msginfo->to) {
-                       gchar *to;
-                       Xstrdup_a(to, msginfo->to, return);
-                       extract_address(to);
-                       account = account_find_from_address(to);
-               }
-               if (!account) {
-                       gchar cc[BUFFSIZE];
-                       if (!get_header_from_msginfo(msginfo, cc, sizeof(cc), "CC:")) { /* Found a CC header */
-                               extract_address(cc);
-                               account = account_find_from_address(cc);
-                       }        
-               }
-       }
-
-       /* select current account */
-       if (!account) account = cur_account;
        g_return_if_fail(account != NULL);
 
        if (ignore_replyto && account->protocol == A_NNTP &&
@@ -949,8 +890,6 @@ static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                compose_exec_ext_editor(compose);
 }
 
-static void set_toolbar_style(Compose *compose);
-
 #define INSERT_FW_HEADER(var, hdr) \
 if (msginfo->var && *msginfo->var) { \
        gtk_stext_insert(text, NULL, NULL, NULL, hdr, -1); \
@@ -1017,7 +956,7 @@ Compose *compose_forward(PrefsAccount *account, MsgInfo *msginfo,
 
                        msgfile = procmsg_get_message_file_path(msginfo);
                        if (!is_file_exist(msgfile))
-                               g_warning(_("%s: file not exist\n"), msgfile);
+                               g_warning("%s: file not exist\n", msgfile);
                        else
                                compose_attach_append(compose, msgfile, msgfile,
                                                      "message/rfc822");
@@ -1118,7 +1057,7 @@ Compose *compose_forward_multiple(PrefsAccount *account, GSList *msginfo_list)
        for (msginfo = msginfo_list; msginfo != NULL; msginfo = msginfo->next) {
                msgfile = procmsg_get_message_file_path((MsgInfo *)msginfo->data);
                if (!is_file_exist(msgfile))
-                       g_warning(_("%s: file not exist\n"), msgfile);
+                       g_warning("%s: file not exist\n", msgfile);
                else
                        compose_attach_append(compose, msgfile, msgfile,
                                "message/rfc822");
@@ -1164,6 +1103,11 @@ void compose_reedit(MsgInfo *msginfo)
 
                /* Select Account from queue headers */
                if (!get_header_from_msginfo(msginfo, queueheader_buf, 
+                                            sizeof(queueheader_buf), "X-Sylpheed-Account-Id:")) {
+                       id = atoi(&queueheader_buf[22]);
+                       account = account_find_from_id(id);
+               }
+               if (!account && !get_header_from_msginfo(msginfo, queueheader_buf, 
                                             sizeof(queueheader_buf), "NAID:")) {
                        id = atoi(&queueheader_buf[5]);
                        account = account_find_from_id(id);
@@ -1214,7 +1158,7 @@ void compose_reedit(MsgInfo *msginfo)
        gtk_stext_freeze(text);
 
        if ((fp = procmime_get_first_text_content(msginfo)) == NULL)
-               g_warning(_("Can't get text part\n"));
+               g_warning("Can't get text part\n");
        else {
                while (fgets(buf, sizeof(buf), fp) != NULL) {
                        strcrchomp(buf);
@@ -1257,6 +1201,8 @@ Compose *compose_redirect(PrefsAccount *account, MsgInfo *msginfo)
                return NULL;
 
        compose->redirect_filename = filename;
+       
+       compose_attach_parts(compose, msginfo);
 
        if (msginfo->subject)
                gtk_entry_set_text(GTK_ENTRY(compose->subject_entry),
@@ -1277,20 +1223,21 @@ Compose *compose_redirect(PrefsAccount *account, MsgInfo *msginfo)
        menu_set_sensitive(ifactory, "/File/Insert file", FALSE);
        menu_set_sensitive(ifactory, "/File/Attach file", FALSE);
        menu_set_sensitive(ifactory, "/File/Insert signature", FALSE);
-       menu_set_sensitive(ifactory, "/Edit/Paste", FALSE);
-       menu_set_sensitive(ifactory, "/Edit/Wrap current paragraph", FALSE);
-       menu_set_sensitive(ifactory, "/Edit/Wrap all long lines", FALSE);
-       menu_set_sensitive(ifactory, "/Edit/Edit with external editor", FALSE);
-       menu_set_sensitive(ifactory, "/Message/Attach", FALSE);
+       menu_set_sensitive(ifactory, "/Edit", FALSE);
+       menu_set_sensitive(ifactory, "/Message/Save to draft folder", FALSE);
+       menu_set_sensitive(ifactory, "/Message/Save and keep editing", FALSE);
 #if USE_GPGME
        menu_set_sensitive(ifactory, "/Message/Sign", FALSE);
        menu_set_sensitive(ifactory, "/Message/Encrypt", FALSE);
        menu_set_sensitive(ifactory, "/Message/Mode/MIME", FALSE);
        menu_set_sensitive(ifactory, "/Message/Mode/Inline", FALSE);
 #endif
+       menu_set_sensitive(ifactory, "/Message/Priority", FALSE);
        menu_set_sensitive(ifactory, "/Message/Request Return Receipt", FALSE);
-       menu_set_sensitive(ifactory, "/Tools/Template", FALSE);
+       menu_set_sensitive(ifactory, "/Tools/Show ruler", FALSE);
+       menu_set_sensitive(ifactory, "/Tools/Actions", FALSE);
        
+       gtk_widget_set_sensitive(compose->toolbar->draft_btn, FALSE);
        gtk_widget_set_sensitive(compose->toolbar->insert_btn, FALSE);
        gtk_widget_set_sensitive(compose->toolbar->attach_btn, FALSE);
        gtk_widget_set_sensitive(compose->toolbar->sig_btn, FALSE);
@@ -1375,6 +1322,46 @@ void compose_entry_select (Compose *compose, const gchar *mailto)
        }
 }
 
+void compose_toolbar_cb(gint action, gpointer data)
+{
+       ToolbarItem *toolbar_item = (ToolbarItem*)data;
+       Compose *compose = (Compose*)toolbar_item->parent;
+       
+       g_return_if_fail(compose != NULL);
+
+       switch(action) {
+       case A_SEND:
+               compose_send_cb(compose, 0, NULL);
+               break;
+       case A_SENDL:
+               compose_send_later_cb(compose, 0, NULL);
+               break;
+       case A_DRAFT:
+               compose_draft_cb(compose, 0, NULL);
+               break;
+       case A_INSERT:
+               compose_insert_file_cb(compose, 0, NULL);
+               break;
+       case A_ATTACH:
+               compose_attach_cb(compose, 0, NULL);
+               break;
+       case A_SIG:
+               compose_insert_sig(compose);
+               break;
+       case A_EXTEDITOR:
+               compose_ext_editor_cb(compose, 0, NULL);
+               break;
+       case A_LINEWRAP:
+               compose_wrap_line(compose);
+               break;
+       case A_ADDRBOOK:
+               compose_address_cb(compose, 0, NULL);
+               break;
+       default:
+               break;
+       }
+}
+
 static void compose_entries_set(Compose *compose, const gchar *mailto)
 {
        gchar *to = NULL;
@@ -1652,7 +1639,8 @@ static void compose_reply_set_entry(Compose *compose, MsgInfo *msginfo,
        g_return_if_fail(msginfo != NULL);
 
        if (compose->account->protocol != A_NNTP || followup_and_reply_to) {
-               if (!compose->replyto && to_ml && compose->ml_post)
+               if (!compose->replyto && to_ml && compose->ml_post
+                   && !(msginfo->folder && msginfo->folder->prefs->enable_default_reply_to))
                        compose_entry_append(compose,
                                           compose->ml_post,
                                           COMPOSE_TO);
@@ -1783,27 +1771,6 @@ static void compose_reedit_set_entry(Compose *compose, MsgInfo *msginfo)
 #endif
        compose_show_first_last_header(compose, TRUE);
 
-#if 0 /* NEW COMPOSE GUI */
-       if (compose->bcc) {
-               GtkItemFactory *ifactory;
-               GtkWidget *menuitem;
-
-               ifactory = gtk_item_factory_from_widget(compose->menubar);
-               menuitem = gtk_item_factory_get_item(ifactory, "/View/Bcc");
-               gtk_check_menu_item_set_active
-                       (GTK_CHECK_MENU_ITEM(menuitem), TRUE);
-       }
-       if (compose->replyto) {
-               GtkItemFactory *ifactory;
-               GtkWidget *menuitem;
-
-               ifactory = gtk_item_factory_from_widget(compose->menubar);
-               menuitem = gtk_item_factory_get_item
-                       (ifactory, "/View/Reply to");
-               gtk_check_menu_item_set_active
-                       (GTK_CHECK_MENU_ITEM(menuitem), TRUE);
-       }
-#endif
 }
 
 #undef SET_ENTRY
@@ -1923,11 +1890,11 @@ static void compose_attach_append(Compose *compose, const gchar *file,
        gint row;
 
        if (!is_file_exist(file)) {
-               g_warning(_("File %s doesn't exist\n"), file);
+               g_warning("File %s doesn't exist\n", file);
                return;
        }
        if ((size = get_file_size(file)) < 0) {
-               g_warning(_("Can't get file size of %s\n"), file);
+               g_warning("Can't get file size of %s\n", file);
                return;
        }
        if (size == 0) {
@@ -2039,7 +2006,7 @@ static void compose_attach_parts(Compose *compose, MsgInfo *msginfo)
                }
                outfile = procmime_get_tmp_file_name(child);
                if (procmime_get_part(outfile, infile, child) < 0)
-                       g_warning(_("Can't get the part of multipart message."));
+                       g_warning("Can't get the part of multipart message.");
                else if (compose->mode != COMPOSE_REEDIT || strcmp(child->content_type, "application/pgp-signature"))
                        compose_attach_append
                                (compose, outfile,
@@ -2447,7 +2414,7 @@ static void compose_wrap_line_all_full(Compose *compose, gboolean autowrap)
 
                        /* skip delete if it is continuous URL */
                        if (do_delete && (line_pos - p_pos <= i_len) &&
-                           gtkut_stext_is_uri_string(text, line_pos, tlen))
+                           gtk_stext_is_uri_string(text, line_pos, tlen))
                                do_delete = FALSE;
 
 #ifdef WRAP_DEBUG
@@ -2466,7 +2433,7 @@ static void compose_wrap_line_all_full(Compose *compose, gboolean autowrap)
                                   indent string, delete them */
                                if (i_len) {
                                        guint ilen;
-                                       ilen =  gtkut_stext_str_compare_n
+                                       ilen =  gtk_stext_str_compare_n
                                                (text, cur_pos, p_pos, i_len,
                                                 tlen);
                                        if (ilen) {
@@ -2529,7 +2496,7 @@ static void compose_wrap_line_all_full(Compose *compose, gboolean autowrap)
 #endif
                        /* force wrapping if it is one long word but not URL */
                        if (line_pos - p_pos <= i_len)
-                               if (!gtkut_stext_is_uri_string
+                               if (!gtk_stext_is_uri_string
                                    (text, line_pos, tlen))
                                        line_pos = cur_pos - 1;
 #ifdef WRAP_DEBUG
@@ -2541,7 +2508,7 @@ static void compose_wrap_line_all_full(Compose *compose, gboolean autowrap)
                        /* if next character is space delete it */
                        if (clen == 1 && isspace(*cbuf)) {
                                if (p_pos + i_len != line_pos ||
-                                   !gtkut_stext_is_uri_string
+                                   !gtk_stext_is_uri_string
                                        (text, line_pos, tlen)) {
                                        STEXT_FREEZE();
                                        /* workaround for correct cursor
@@ -2563,7 +2530,7 @@ static void compose_wrap_line_all_full(Compose *compose, gboolean autowrap)
 
                        /* if it is URL at beginning of line don't wrap */
                        if (p_pos + i_len == line_pos &&
-                           gtkut_stext_is_uri_string(text, line_pos, tlen)) {
+                           gtk_stext_is_uri_string(text, line_pos, tlen)) {
 #ifdef WRAP_DEBUG
                                g_print("found URL at ");
                                dump_text(text, line_pos, tlen, 1);
@@ -2596,7 +2563,7 @@ static void compose_wrap_line_all_full(Compose *compose, gboolean autowrap)
                        /* should we insert quotation ? */
                        if (linewrap_quote && i_len) {
                                /* only if line is not already quoted  */
-                               if (!gtkut_stext_str_compare
+                               if (!gtk_stext_str_compare
                                        (text, line_pos, tlen, qfmt)) {
                                        guint ins_len;
 
@@ -2726,30 +2693,23 @@ static void compose_select_account(Compose *compose, PrefsAccount *account)
                gtk_widget_set_sensitive(menuitem, FALSE);
        }
 
-       if (account->set_autocc && account->auto_cc &&
-           compose->mode != COMPOSE_REEDIT) {
-               gtk_entry_set_text
-                       (GTK_ENTRY(compose->cc_entry), account->auto_cc);
-               menuitem = gtk_item_factory_get_item(ifactory, "/View/Cc");
-               gtk_check_menu_item_set_active
-                       (GTK_CHECK_MENU_ITEM(menuitem), TRUE);
+       if (account->set_autocc) {
+               compose_entry_show(compose, COMPOSE_ENTRY_CC);
+               if (account->auto_cc && compose->mode != COMPOSE_REEDIT)
+                       compose_entry_set(compose, account->auto_cc,
+                                         COMPOSE_ENTRY_CC);
        }
        if (account->set_autobcc) {
-               menuitem = gtk_item_factory_get_item(ifactory, "/View/Bcc");
-               gtk_check_menu_item_set_active
-                       (GTK_CHECK_MENU_ITEM(menuitem), TRUE);
+               compose_entry_show(compose, COMPOSE_ENTRY_BCC);
                if (account->auto_bcc && compose->mode != COMPOSE_REEDIT)
-                       gtk_entry_set_text(GTK_ENTRY(compose->bcc_entry),
-                                          account->auto_bcc);
+                       compose_entry_set(compose, account->auto_bcc,
+                                         COMPOSE_ENTRY_BCC);
        }
        if (account->set_autoreplyto) {
-               menuitem = gtk_item_factory_get_item(ifactory,
-                                                    "/View/Reply to");
-               gtk_check_menu_item_set_active
-                       (GTK_CHECK_MENU_ITEM(menuitem), TRUE);
+               compose_entry_show(compose, COMPOSE_ENTRY_REPLY_TO);
                if (account->auto_replyto && compose->mode != COMPOSE_REEDIT)
-                       gtk_entry_set_text(GTK_ENTRY(compose->reply_entry),
-                                          account->auto_replyto);
+                       compose_entry_set(compose, account->auto_replyto,
+                                         COMPOSE_ENTRY_REPLY_TO);
        }
 
        menuitem = gtk_item_factory_get_item(ifactory, "/View/Ruler");
@@ -2856,7 +2816,6 @@ gint compose_send(Compose *compose)
        g_free(msgpath);
 
        folder_item_remove_msg(folder, msgnum);
-       folder_update_item(folder, TRUE);
 
        return val;
 }
@@ -2892,7 +2851,7 @@ gint compose_send(Compose *compose)
        }
 
        if (!compose->to_list && !compose->newsgroup_list) {
-               g_warning(_("can't get recipient list."));
+               g_warning("can't get recipient list.");
                unlink(tmp);
                lock = FALSE;
                return -1;
@@ -2924,7 +2883,6 @@ gint compose_send(Compose *compose)
                ac = compose->account;
 
                ok = send_message(tmp, ac, compose->to_list);
-               statusbar_pop_all();
        }
 
        if (ok == 0 && compose->newsgroup_list) {
@@ -2958,9 +2916,6 @@ gint compose_send(Compose *compose)
        } else {
                if (compose->mode == COMPOSE_REEDIT) {
                        compose_remove_reedit_target(compose);
-                       if (compose->targetinfo)
-                               folder_update_item
-                                       (compose->targetinfo->folder, TRUE);
                }
                /* save message to outbox */
                if (prefs_common.savemsg) {
@@ -2971,8 +2926,6 @@ gint compose_send(Compose *compose)
                        if (procmsg_save_to_outbox(outbox, tmp, FALSE) < 0)
                                alertpanel_error
                                        (_("Can't save the message to Sent."));
-                       else
-                               folder_update_item(outbox, TRUE);
                }
        }
 
@@ -3106,7 +3059,7 @@ static gint compose_redirect_write_to_file(Compose *compose, const gchar *file)
        /* chmod for security */
        if (change_file_mode_rw(fdest, file) < 0) {
                FILE_OP_ERROR(file, "chmod");
-               g_warning(_("can't change file mode\n"));
+               g_warning("can't change file mode\n");
        }
 
        while (procheader_get_unfolded_line(buf, sizeof(buf), fp)) {
@@ -3259,7 +3212,7 @@ static gint compose_write_to_file(Compose *compose, const gchar *file,
        /* chmod for security */
        if (change_file_mode_rw(fp, file) < 0) {
                FILE_OP_ERROR(file, "chmod");
-               g_warning(_("can't change file mode\n"));
+               g_warning("can't change file mode\n");
        }
 
        /* get all composed text */
@@ -3358,7 +3311,7 @@ static gint compose_write_to_file(Compose *compose, const gchar *file,
        /* write headers */
        if (compose_write_headers
                (compose, fp, out_codeset, encoding, is_draft) < 0) {
-               g_warning(_("can't write headers\n"));
+               g_warning("can't write headers\n");
                fclose(fp);
                /* unlink(file); */
                g_free(buf);
@@ -3459,7 +3412,7 @@ static gint compose_write_body_to_file(Compose *compose, const gchar *file)
        /* chmod for security */
        if (change_file_mode_rw(fp, file) < 0) {
                FILE_OP_ERROR(file, "chmod");
-               g_warning(_("can't change file mode\n"));
+               g_warning("can't change file mode\n");
        }
 
        chars = gtk_editable_get_chars(GTK_EDITABLE(compose->text), 0, -1);
@@ -3498,7 +3451,7 @@ static gint compose_remove_reedit_target(Compose *compose)
        if (procmsg_msg_exist(msginfo) &&
            (item->stype == F_DRAFT || item->stype == F_QUEUE)) {
                if (folder_item_remove_msg(item, msginfo->msgnum) < 0) {
-                       g_warning(_("can't remove the old message\n"));
+                       g_warning("can't remove the old message\n");
                        return -1;
                }
        }
@@ -3514,7 +3467,6 @@ void compose_remove_draft(Compose *compose)
 
        if (procmsg_msg_exist(msginfo)) {
                folder_item_remove_msg(drafts, msginfo->msgnum);
-               folder_update_item(drafts, TRUE);
        }
 
 }
@@ -3545,7 +3497,7 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
        }
 
        if (!compose->to_list && !compose->newsgroup_list) {
-               g_warning(_("can't get recipient list."));
+               g_warning("can't get recipient list.");
                lock = FALSE;
                 return -1;
         }
@@ -3613,7 +3565,7 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
        }
        if (change_file_mode_rw(fp, tmp) < 0) {
                FILE_OP_ERROR(tmp, "chmod");
-               g_warning(_("can't change file mode\n"));
+               g_warning("can't change file mode\n");
        }
 
        /* queueing variables */
@@ -3705,14 +3657,14 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
 
        queue = account_get_special_folder(compose->account, F_QUEUE);
        if (!queue) {
-               g_warning(_("can't find queue folder\n"));
+               g_warning("can't find queue folder\n");
                unlink(tmp);
                g_free(tmp);
                return -1;
        }
        folder_item_scan(queue);
        if ((num = folder_item_add_msg(queue, tmp, TRUE)) < 0) {
-               g_warning(_("can't queue the message\n"));
+               g_warning("can't queue the message\n");
                unlink(tmp);
                g_free(tmp);
                return -1;
@@ -3724,14 +3676,8 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
 
        if (compose->mode == COMPOSE_REEDIT) {
                compose_remove_reedit_target(compose);
-               if (compose->targetinfo &&
-                   compose->targetinfo->folder != queue)
-                       folder_update_item
-                               (compose->targetinfo->folder, TRUE);
        }
 
-       folder_update_item(queue, TRUE);
-
        if ((msgnum != NULL) && (item != NULL)) {
                *msgnum = num;
                *item = queue;
@@ -3755,7 +3701,7 @@ static void compose_write_attach(Compose *compose, FILE *fp)
                gchar inbuf[B64_LINE_SIZE], outbuf[B64_BUFFSIZE];
 
                if ((attach_fp = fopen(ainfo->file, "rb")) == NULL) {
-                       g_warning(_("Can't open file %s\n"), ainfo->file);
+                       g_warning("Can't open file %s\n", ainfo->file);
                        continue;
                }
 
@@ -3765,8 +3711,8 @@ static void compose_write_attach(Compose *compose, FILE *fp)
                        fprintf(fp, "Content-Type: %s\n", ainfo->content_type);
                        fprintf(fp, "Content-Disposition: inline\n");
                } else {
-                       conv_encode_header(filename, sizeof(filename),
-                                          ainfo->name, 12);
+                       compose_convert_header(filename, sizeof(filename),
+                                              ainfo->name, 12);
                        fprintf(fp, "Content-Type: %s;\n"
                                    " name=\"%s\"\n",
                                ainfo->content_type, filename);
@@ -3921,6 +3867,7 @@ static gint compose_write_headers(Compose *compose, FILE *fp,
 
        /* Save draft infos */
        if (is_draft) {
+               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;
@@ -4214,14 +4161,9 @@ static void compose_convert_header(gchar *dest, gint len, gchar *src,
 
        if (len < 1) return;
 
-       remove_return(src);
+       g_strchomp(src);
 
-       if (is_ascii_str(src)) {
-               strncpy2(dest, src, len);
-               dest[len - 1] = '\0';
-               return;
-       } else
-               conv_encode_header(dest, len, src, header_len);
+       conv_encode_header(dest, len, src, header_len);
 }
 
 static void compose_generate_msgid(Compose *compose, gchar *buf, gint len)
@@ -4262,7 +4204,7 @@ static void compose_create_header_entry(Compose *compose)
        GtkWidget *combo;
        GtkWidget *entry;
        GList *combo_list = NULL;
-       gchar **string, *header;
+       gchar **string, *header = NULL;
        ComposeHeaderEntry *headerentry;
        gboolean standard_header = FALSE;
 
@@ -4272,8 +4214,8 @@ static void compose_create_header_entry(Compose *compose)
        combo = gtk_combo_new();
        string = headers; 
        while(*string != NULL) {
-           combo_list = g_list_append(combo_list, (prefs_common.trans_hdr ? gettext(*string) : *string));
-           string++;
+               combo_list = g_list_append(combo_list, (prefs_common.trans_hdr ? gettext(*string) : *string));
+               string++;
        }
        gtk_combo_set_popdown_strings(GTK_COMBO(combo), combo_list);
        g_list_free(combo_list);
@@ -4301,7 +4243,8 @@ static void compose_create_header_entry(Compose *compose)
                                break;
                }                                                                   
        }
-       gtk_entry_set_text(GTK_ENTRY(GTK_COMBO(combo)->entry), header);
+       if (header)
+               gtk_entry_set_text(GTK_ENTRY(GTK_COMBO(combo)->entry), header);
 
        /* Entry field */
        entry = gtk_entry_new(); 
@@ -4685,7 +4628,8 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
        handlebox = gtk_handle_box_new();
        gtk_box_pack_start(GTK_BOX(vbox), handlebox, FALSE, FALSE, 0);
 
-       compose_toolbar_create(compose, handlebox);
+       compose->toolbar = toolbar_create(TOOLBAR_COMPOSE, handlebox,
+                                         (gpointer)compose);
 
        vbox2 = gtk_vbox_new(FALSE, 2);
        gtk_box_pack_start(GTK_BOX(vbox), vbox2, TRUE, TRUE, 0);
@@ -4945,34 +4889,35 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
        compose->redirect_filename = NULL;
        compose->undostruct = undostruct;
 #if USE_ASPELL
-       
        menu_set_sensitive(ifactory, "/Spelling", FALSE);
-        if (prefs_common.enable_aspell && prefs_common.dictionary &&
-           strcmp(prefs_common.dictionary, _("None"))) {
-               gtkaspell = gtkaspell_new((const gchar*)prefs_common.dictionary,
-                                         conv_get_current_charset_str(),
-                                         prefs_common.misspelled_col,
-                                         prefs_common.check_while_typing,
-                                         prefs_common.use_alternate,
-                                         GTK_STEXT(text));
-               if (!gtkaspell) {
-                       alertpanel_error(_("Spell checker could not be started.\n%s"), gtkaspellcheckers->error_message);
-                       gtkaspell_checkers_reset_error();
-               } else {
+       if (mode != COMPOSE_REDIRECT) {
+               if (prefs_common.enable_aspell && prefs_common.dictionary &&
+                   strcmp(prefs_common.dictionary, _("None"))) {
+                       gtkaspell = gtkaspell_new((const gchar*)prefs_common.dictionary,
+                                                 conv_get_current_charset_str(),
+                                                 prefs_common.misspelled_col,
+                                                 prefs_common.check_while_typing,
+                                                 prefs_common.use_alternate,
+                                                 GTK_STEXT(text));
+                       if (!gtkaspell) {
+                               alertpanel_error(_("Spell checker could not be started.\n%s"), gtkaspellcheckers->error_message);
+                               gtkaspell_checkers_reset_error();
+                       } else {
 
-                       GtkWidget *menuitem;
+                               GtkWidget *menuitem;
 
-                       if (!gtkaspell_set_sug_mode(gtkaspell, prefs_common.aspell_sugmode)) {
-                               debug_print("Aspell: could not set suggestion mode %s\n",
-                                   gtkaspellcheckers->error_message);
-                               gtkaspell_checkers_reset_error();
-                       }
+                               if (!gtkaspell_set_sug_mode(gtkaspell, prefs_common.aspell_sugmode)) {
+                                       debug_print("Aspell: could not set suggestion mode %s\n",
+                                       gtkaspellcheckers->error_message);
+                                       gtkaspell_checkers_reset_error();
+                               }
 
-                       menuitem = gtk_item_factory_get_item(ifactory, "/Spelling/Spelling Configuration");
-                       gtkaspell_populate_submenu(gtkaspell, menuitem);
-                       menu_set_sensitive(ifactory, "/Spelling", TRUE);
-                       }
-        }
+                               menuitem = gtk_item_factory_get_item(ifactory, "/Spelling/Spelling Configuration");
+                               gtkaspell_populate_submenu(gtkaspell, menuitem);
+                               menu_set_sensitive(ifactory, "/Spelling", TRUE);
+                               }
+               }
+       }
 #endif
 
        compose_select_account(compose, account);
@@ -4998,7 +4943,14 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
 
        addressbook_set_target_compose(compose);
        update_compose_actions_menu(ifactory, "/Tools/Actions", compose);
-       compose_set_template_menu(compose);
+       
+       if (mode != COMPOSE_REDIRECT)
+               compose_set_template_menu(compose);
+       else {
+               GtkWidget *menuitem;
+               menuitem = gtk_item_factory_get_item(ifactory, "/Tools/Template");
+               menu_set_sensitive(ifactory, "/Tools/Template", FALSE);
+       }
 
        compose_list = g_list_append(compose_list, compose);
 
@@ -5012,234 +4964,12 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
 #if USE_GPGME
        activate_gnupg_mode(compose, account);
 #endif 
-
-       set_toolbar_style(compose);
-
+       toolbar_set_style(compose->toolbar->toolbar, compose->handlebox, prefs_common.toolbar_style);
        gtk_widget_show(window);
        
        return compose;
 }
 
-static void toolbar_compose_buttons_cb(GtkWidget   *widget, 
-                                      ToolbarItem *t_item)
-{
-       struct {
-               gint   index;
-               void (*func)(GtkWidget *widget, gpointer data);
-       } compose_action[] = {
-               { A_SEND,        toolbar_send_cb            },
-               { A_SENDL,       toolbar_send_later_cb      },
-               { A_DRAFT,       toolbar_draft_cb           },
-               { A_INSERT,      toolbar_insert_cb          },
-               { A_ATTACH,      toolbar_attach_cb          },
-               { A_SIG,         toolbar_sig_cb             },
-               { A_EXTEDITOR,   toolbar_ext_editor_cb      },
-               { A_LINEWRAP,    toolbar_linewrap_cb        },
-               { A_ADDRBOOK,    toolbar_address_cb         },
-               { A_SYL_ACTIONS, toolbar_actions_execute_cb }};
-
-
-       gint num_items = sizeof(compose_action)/sizeof(compose_action[0]);
-       gint i;
-       debug_print("_buttons: index: %i \n", t_item->index);
-       for (i = 0; i < num_items; i++) {
-               
-               if (compose_action[i].index == t_item->index) {
-                       Compose *compose = (Compose*)t_item->parent;
-                       debug_print("_buttons: compose: %p toolbar: %p\n", compose, compose->toolbar);                  compose_action[i].func(widget, compose);
-                       break;
-               }
-       }
-}
-
-static void compose_toolbar_update(Compose *compose)
-{
-       gtk_container_remove(GTK_CONTAINER(compose->handlebox), 
-                            GTK_WIDGET(compose->toolbar->toolbar));
-       
-       compose->toolbar->toolbar       = NULL;
-       compose->toolbar->send_btn      = NULL;
-       compose->toolbar->sendl_btn     = NULL;
-       compose->toolbar->draft_btn     = NULL;
-       compose->toolbar->insert_btn    = NULL;
-       compose->toolbar->attach_btn    = NULL;
-       compose->toolbar->sig_btn       = NULL; 
-       compose->toolbar->exteditor_btn = NULL; 
-       compose->toolbar->linewrap_btn  = NULL; 
-       compose->toolbar->addrbook_btn  = NULL; 
-
-       TOOLBAR_DESTROY_ITEMS(compose->toolbar->t_item_list);   
-       TOOLBAR_DESTROY_ACTIONS(compose->toolbar->t_action_list);
-       compose_toolbar_create(compose, compose->handlebox);    
-}
-
-static void compose_toolbar_create(Compose   *compose, 
-                                  GtkWidget *container)
-{
-       ToolbarItem *toolbar_item;
-
-       GtkWidget *toolbar;
-       GtkWidget *icon_wid  = NULL;
-       GtkWidget *item;
-       GtkTooltips *toolbar_tips;
-       ToolbarSylpheedActions *t_action_item;
-       GSList *cur;
-       GSList *toolbar_list;
-       GList *elem;
-       toolbar_tips = gtk_tooltips_new();
-       
-       toolbar_read_config_file(TOOLBAR_COMPOSE);
-       toolbar_list = toolbar_get_list(TOOLBAR_COMPOSE);
-
-       compose->toolbar = g_new0(ComposeToolbar, 1); 
-       
-       for (elem = compose_list; elem != NULL; elem = elem->next) {
-               Compose *c = (Compose*)elem->data;
-               debug_print("toolbar_create: compose: %p toolbar: %p\n", 
-                           c, c->toolbar);
-       }
-       
-       toolbar = gtk_toolbar_new(GTK_ORIENTATION_HORIZONTAL,
-                                 GTK_TOOLBAR_BOTH);
-       gtk_container_add(GTK_CONTAINER(container), toolbar);
-       gtk_container_set_border_width(GTK_CONTAINER(container), 2);
-       gtk_toolbar_set_button_relief(GTK_TOOLBAR(toolbar), GTK_RELIEF_NONE);
-       gtk_toolbar_set_space_style(GTK_TOOLBAR(toolbar),
-                                   GTK_TOOLBAR_SPACE_LINE);
-       
-       for (cur = toolbar_list; cur != NULL; cur = cur->next) {
-
-               if (g_strcasecmp(((ToolbarItem*)cur->data)->file, SEPARATOR) == 0) {
-                       gtk_toolbar_append_space(GTK_TOOLBAR(toolbar));
-                       continue;
-               }
-               
-               toolbar_item = g_new0(ToolbarItem, 1); 
-               toolbar_item->file = g_strdup(((ToolbarItem*)cur->data)->file);
-               toolbar_item->text = g_strdup(((ToolbarItem*)cur->data)->text);
-               toolbar_item->index = ((ToolbarItem*)cur->data)->index;
-
-               toolbar_item->parent = (gpointer)compose;
-
-               /* collect toolbar items in list to keep track */
-               compose->toolbar->t_item_list = g_slist_append(compose->toolbar->t_item_list, 
-                                                             toolbar_item);
-
-               icon_wid = stock_pixmap_widget(container, stock_pixmap_get_icon(toolbar_item->file));
-               item  = gtk_toolbar_append_item(GTK_TOOLBAR(toolbar),
-                                               toolbar_item->text,
-                                               (""),
-                                               (""),
-                                               icon_wid, toolbar_compose_buttons_cb, 
-                                               toolbar_item);
-               
-               switch (toolbar_item->index) {
-               case A_SEND:
-                       compose->toolbar->send_btn = item;
-                       gtk_tooltips_set_tip(GTK_TOOLTIPS(toolbar_tips), 
-                                            compose->toolbar->send_btn, 
-                                            _("Send Message"), NULL);
-                       break;
-               case A_SENDL:
-                       compose->toolbar->sendl_btn = item;
-                       gtk_tooltips_set_tip(GTK_TOOLTIPS(toolbar_tips), 
-                                            compose->toolbar->sendl_btn,
-                                            _("Put into queue folder and send later"), NULL);
-                       break;
-               case A_DRAFT:
-                       compose->toolbar->draft_btn = item; 
-                       gtk_tooltips_set_tip(GTK_TOOLTIPS(toolbar_tips), 
-                                            compose->toolbar->draft_btn,
-                                            _("Save to draft folder"), NULL);
-                       break;
-               case A_INSERT:
-                       compose->toolbar->insert_btn = item; 
-                       gtk_tooltips_set_tip(GTK_TOOLTIPS(toolbar_tips), 
-                                            compose->toolbar->insert_btn,
-                                            _("Insert file"), NULL);
-                       break;
-               case A_ATTACH:
-                       compose->toolbar->attach_btn = item;
-                       gtk_tooltips_set_tip(GTK_TOOLTIPS(toolbar_tips), 
-                                            compose->toolbar->attach_btn,
-                                            _("Attach file"), NULL);
-                       break;
-               case A_SIG:
-                       compose->toolbar->sig_btn = item;
-                       gtk_tooltips_set_tip(GTK_TOOLTIPS(toolbar_tips), 
-                                            compose->toolbar->sig_btn,
-                                            _("Insert signature"), NULL);
-                       break;
-               case A_EXTEDITOR:
-                       compose->toolbar->exteditor_btn = item;
-                       gtk_tooltips_set_tip(GTK_TOOLTIPS(toolbar_tips), 
-                                            compose->toolbar->exteditor_btn,
-                                            _("Edit with external editor"), NULL);
-                       break;
-               case A_LINEWRAP:
-                       compose->toolbar->linewrap_btn = item;
-                       gtk_tooltips_set_tip(GTK_TOOLTIPS(toolbar_tips), 
-                                            compose->toolbar->linewrap_btn,
-                                            _("Wrap all long lines"), NULL);
-                       break;
-               case A_ADDRBOOK:
-                       compose->toolbar->addrbook_btn = item;
-                       gtk_tooltips_set_tip(GTK_TOOLTIPS(toolbar_tips), 
-                                            compose->toolbar->addrbook_btn,
-                                            _("Address book"), NULL);
-                       break;
-               case A_SYL_ACTIONS:
-                       t_action_item = g_new0(ToolbarSylpheedActions, 1);
-                       t_action_item->widget = item;
-                       t_action_item->name   = g_strdup(toolbar_item->text);
-
-                       compose->toolbar->t_action_list = 
-                               g_slist_append(compose->toolbar->t_action_list,
-                                              t_action_item);
-
-                       gtk_tooltips_set_tip(GTK_TOOLTIPS(toolbar_tips), 
-                                            item,
-                                            t_action_item->name, NULL);
-
-                       gtk_widget_show(item);
-                       break;
-               default:
-                       break;
-               }
-       }
-
-       compose->toolbar->toolbar = toolbar;
-
-       gtk_widget_show_all(toolbar);
-}
-
-static void compose_toolbar_set_sensitive (Compose * compose, gboolean sensitive)
-{
-       GSList *items = compose->toolbar->t_action_list;
-       if (compose->toolbar->send_btn)
-               gtk_widget_set_sensitive(compose->toolbar->send_btn, sensitive);
-       if (compose->toolbar->sendl_btn)
-               gtk_widget_set_sensitive(compose->toolbar->sendl_btn, sensitive);
-       if (compose->toolbar->draft_btn )
-               gtk_widget_set_sensitive(compose->toolbar->draft_btn , sensitive);
-       if (compose->toolbar->insert_btn )
-               gtk_widget_set_sensitive(compose->toolbar->insert_btn , sensitive);
-       if (compose->toolbar->attach_btn)
-               gtk_widget_set_sensitive(compose->toolbar->attach_btn, sensitive);
-       if (compose->toolbar->sig_btn)
-               gtk_widget_set_sensitive(compose->toolbar->sig_btn, sensitive);
-       if (compose->toolbar->exteditor_btn)
-               gtk_widget_set_sensitive(compose->toolbar->exteditor_btn, sensitive);
-       if (compose->toolbar->linewrap_btn)
-               gtk_widget_set_sensitive(compose->toolbar->linewrap_btn, sensitive);
-       if (compose->toolbar->addrbook_btn)
-               gtk_widget_set_sensitive(compose->toolbar->addrbook_btn, sensitive);
-       for (; items != NULL; items = g_slist_next(items)) {
-               ToolbarSylpheedActions *item = (ToolbarSylpheedActions *)items->data;
-               gtk_widget_set_sensitive(item->widget, sensitive);
-       }
-}
-
 static GtkWidget *compose_account_option_menu_create(Compose *compose)
 {
        GList *accounts;
@@ -5270,7 +5000,7 @@ static GtkWidget *compose_account_option_menu_create(Compose *compose)
                else
                        name = g_strdup_printf("%s: %s",
                                               ac->account_name, ac->address);
-               MENUITEM_ADD(menu, menuitem, name, ac);
+               MENUITEM_ADD(menu, menuitem, name, ac->account_id);
                g_free(name);
                gtk_signal_connect(GTK_OBJECT(menuitem), "activate",
                                   GTK_SIGNAL_FUNC(account_activated),
@@ -5402,8 +5132,7 @@ void compose_reflect_prefs_pixmap_theme(void)
 
        for (cur = compose_list; cur != NULL; cur = cur->next) {
                compose = (Compose *)cur->data;
-               compose_toolbar_update(compose);
-               set_toolbar_style(compose);
+               toolbar_update(TOOLBAR_COMPOSE, compose);
        }
 }
 
@@ -5514,8 +5243,7 @@ static void compose_destroy(Compose *compose)
 
        gtk_widget_destroy(compose->paned);
 
-       TOOLBAR_DESTROY_ITEMS(compose->toolbar->t_item_list);
-       TOOLBAR_DESTROY_ACTIONS(compose->toolbar->t_action_list);
+       toolbar_destroy(compose->toolbar);
        g_free(compose->toolbar);
        g_free(compose);
 
@@ -5926,7 +5654,7 @@ static gint compose_exec_ext_editor_real(const gchar *file)
                g_snprintf(buf, sizeof(buf), prefs_common.ext_editor_cmd, file);
        } else {
                if (prefs_common.ext_editor_cmd)
-                       g_warning(_("External editor command line is invalid: `%s'\n"),
+                       g_warning("External editor command line is invalid: `%s'\n",
                                  prefs_common.ext_editor_cmd);
                g_snprintf(buf, sizeof(buf), def_cmd, file);
        }
@@ -5965,8 +5693,8 @@ static gboolean compose_ext_editor_kill(Compose *compose)
                        if (kill(pgid, SIGTERM) < 0) perror("kill");
                        waitpid(compose->exteditor_pid, NULL, 0);
 
-                       g_warning(_("Terminated process group id: %d"), -pgid);
-                       g_warning(_("Temporary file: %s"),
+                       g_warning("Terminated process group id: %d", -pgid);
+                       g_warning("Temporary file: %s",
                                  compose->exteditor_file);
 
                        compose_set_ext_editor_sensitive(compose, TRUE);
@@ -6023,13 +5751,13 @@ static void compose_input_cb(gpointer data, gint source,
                if (unlink(compose->exteditor_file) < 0)
                        FILE_OP_ERROR(compose->exteditor_file, "unlink");
        } else if (buf[0] == '1') {     /* failed */
-               g_warning(_("Couldn't exec external editor\n"));
+               g_warning("Couldn't exec external editor\n");
                if (unlink(compose->exteditor_file) < 0)
                        FILE_OP_ERROR(compose->exteditor_file, "unlink");
        } else if (buf[0] == '2') {
-               g_warning(_("Couldn't write to file\n"));
+               g_warning("Couldn't write to file\n");
        } else if (buf[0] == '3') {
-               g_warning(_("Pipe read failed\n"));
+               g_warning("Pipe read failed\n");
        }
 
        close(source);
@@ -6179,72 +5907,14 @@ static gboolean compose_edit_size_alloc(GtkEditable *widget,
        return TRUE;
 }
 
-static void toolbar_send_cb(GtkWidget *widget, gpointer data)
-{
-       compose_send_cb(data, 0, NULL);
-}
-
-static void toolbar_send_later_cb(GtkWidget *widget, gpointer data)
-{
-       compose_send_later_cb(data, 0, NULL);
-}
-
-static void toolbar_draft_cb(GtkWidget *widget, gpointer data)
-{
-       compose_draft_cb(data, 0, NULL);
-}
-
-static void toolbar_insert_cb(GtkWidget *widget, gpointer data)
-{
-       compose_insert_file_cb(data, 0, NULL);
-}
-
-static void toolbar_attach_cb(GtkWidget *widget, gpointer data)
-{
-       compose_attach_cb(data, 0, NULL);
-}
-
-static void toolbar_sig_cb(GtkWidget *widget, gpointer data)
-{
-       Compose *compose = (Compose *)data;
-
-       compose_insert_sig(compose);
-}
-
-static void toolbar_ext_editor_cb(GtkWidget *widget, gpointer data)
-{
-       Compose *compose = (Compose *)data;
-
-       compose_exec_ext_editor(compose);
-}
-
-static void toolbar_linewrap_cb(GtkWidget *widget, gpointer data)
-{
-       Compose *compose = (Compose *)data;
-
-       compose_wrap_line(compose);
-}
-
-static void toolbar_address_cb(GtkWidget *widget, gpointer data)
-{
-       compose_address_cb(data, 0, NULL);
-}
-
-static void toolbar_actions_execute_cb(GtkWidget *widget,
-                                      gpointer   data)
-{
-       Compose *compose = (Compose*)data;
-
-       toolbar_action_execute(widget, compose->toolbar->t_action_list, data, TOOLBAR_COMPOSE);
-}
-
 static void account_activated(GtkMenuItem *menuitem, gpointer data)
 {
        Compose *compose = (Compose *)data;
 
        PrefsAccount *ac;
 
-       ac = (PrefsAccount *)gtk_object_get_user_data(GTK_OBJECT(menuitem));
+       ac = account_find_from_id(
+               GPOINTER_TO_INT(gtk_object_get_user_data(GTK_OBJECT(menuitem))));
        g_return_if_fail(ac != NULL);
 
        if (ac != compose->account)
@@ -6302,9 +5972,10 @@ static void attach_key_pressed(GtkWidget *widget, GdkEventKey *event,
 static void compose_allow_user_actions (Compose *compose, gboolean allow)
 {
        GtkItemFactory *ifactory = gtk_item_factory_from_widget(compose->menubar);
-       compose_toolbar_set_sensitive(compose, allow);
+       toolbar_comp_set_sensitive(compose, allow);
        menu_set_sensitive(ifactory, "/File", allow);
        menu_set_sensitive(ifactory, "/Edit", allow);
+       menu_set_sensitive(ifactory, "/Spelling", allow);
        menu_set_sensitive(ifactory, "/Message", allow);
        menu_set_sensitive(ifactory, "/Tools", allow);
        menu_set_sensitive(ifactory, "/Help", allow);
@@ -6381,17 +6052,14 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
 
        if (compose->mode == COMPOSE_REEDIT) {
                compose_remove_reedit_target(compose);
-               if (compose->targetinfo &&
-                   compose->targetinfo->folder != draft)
-                       folder_update_item(compose->targetinfo->folder,
-                                              TRUE);
        }
 
-       newmsginfo = folder_item_fetch_msginfo(draft, msgnum);
-       procmsg_msginfo_unset_flags(newmsginfo, ~0, ~0);
-       MSG_SET_TMP_FLAGS(newmsginfo->flags, MSG_DRAFT);
-       folder_update_item(draft, TRUE);
-       procmsg_msginfo_free(newmsginfo);
+       newmsginfo = folder_item_get_msginfo(draft, msgnum);
+       if (newmsginfo) {
+               procmsg_msginfo_unset_flags(newmsginfo, ~0, ~0);
+               procmsg_msginfo_set_flags(newmsginfo, 0, MSG_DRAFT);
+               procmsg_msginfo_free(newmsginfo);
+       }
        
        lock = FALSE;
 
@@ -7104,8 +6772,11 @@ static gboolean compose_send_control_enter(Compose *compose)
        ifactory = gtk_item_factory_from_widget(compose->menubar);
        send_menu = gtk_item_factory_get_widget(ifactory, "/Message/Send");
        list = gtk_accel_group_entries_from_object(GTK_OBJECT(send_menu));
+       if (!list)
+               return FALSE;
+
        accel = (GtkAccelEntry *)list->data;
-       if (accel->accelerator_key == kev->keyval &&
+       if (accel && accel->accelerator_key == kev->keyval &&
            (accel->accelerator_mods & ~ignored_mods) ==
            (kev->state & ~ignored_mods)) {
                compose_send_cb(compose, 0, NULL);
@@ -7153,28 +6824,3 @@ static void compose_check_forwards_go(Compose *compose)
 }
 #endif
 
-static void set_toolbar_style(Compose *compose)
-{
-       switch (prefs_common.toolbar_style) {
-       case TOOLBAR_NONE:
-               gtk_widget_hide(compose->handlebox);
-               break;
-       case TOOLBAR_ICON:
-               gtk_toolbar_set_style(GTK_TOOLBAR(compose->toolbar->toolbar),
-                                     GTK_TOOLBAR_ICONS);
-               break;
-       case TOOLBAR_TEXT:
-               gtk_toolbar_set_style(GTK_TOOLBAR(compose->toolbar->toolbar),
-                                     GTK_TOOLBAR_TEXT);
-               break;
-       case TOOLBAR_BOTH:
-               gtk_toolbar_set_style(GTK_TOOLBAR(compose->toolbar->toolbar),
-                                     GTK_TOOLBAR_BOTH);
-               break;
-       }
-       
-       if (prefs_common.toolbar_style != TOOLBAR_NONE) {
-               gtk_widget_show(compose->handlebox);
-               gtk_widget_queue_resize(compose->handlebox);
-       }
-}