sort correctly when using simplified subject line
[claws.git] / src / summaryview.c
index 37b255f31884ecdb8f7685280e361b18521413f9..488de6247f312112377498663fefae6f4c3a45c3 100644 (file)
 #include "filtering.h"
 #include "string_match.h"
 
-#define STATUSBAR_PUSH(mainwin, str) \
-{ \
-       gtk_statusbar_push(GTK_STATUSBAR(mainwin->statusbar), \
-                          mainwin->summaryview_cid, str); \
-       gtkut_widget_wait_for_draw(mainwin->hbox_stat); \
-}
-
-#define STATUSBAR_POP(mainwin) \
-{ \
-       gtk_statusbar_pop(GTK_STATUSBAR(mainwin->statusbar), \
-                         mainwin->summaryview_cid); \
-}
-
 #define SUMMARY_COL_MARK_WIDTH         10
 #define SUMMARY_COL_UNREAD_WIDTH       13
 #define SUMMARY_COL_LOCKED_WIDTH       13
@@ -143,9 +130,6 @@ static void summary_free_msginfo_func       (GtkCTree               *ctree,
 static void summary_set_marks_func     (GtkCTree               *ctree,
                                         GtkCTreeNode           *node,
                                         gpointer                data);
-static void summary_write_cache_func   (GtkCTree               *ctree,
-                                        GtkCTreeNode           *node,
-                                        gpointer                data);
 
 static void summary_set_menu_sensitive (SummaryView            *summaryview);
 
@@ -247,7 +231,9 @@ static void summary_unthread_for_exec_func  (GtkCTree       *ctree,
 void summary_simplify_subject(SummaryView *summaryview, gchar * rexp,
                              GSList * mlist);
 
+#if 0
 void summary_processing(SummaryView *summaryview, GSList * mlist);
+#endif
 static void summary_filter_func                (GtkCTree               *ctree,
                                         GtkCTreeNode           *node,
                                         gpointer                data);
@@ -368,6 +354,8 @@ static gint summary_cmp_by_from             (GtkCList               *clist,
 static gint summary_cmp_by_subject     (GtkCList               *clist,
                                         gconstpointer           ptr1,
                                         gconstpointer           ptr2);
+static gint summary_cmp_by_simplified_subject
+       (GtkCList *clist, gconstpointer ptr1, gconstpointer ptr2);
 static gint summary_cmp_by_score       (GtkCList               *clist,
                                         gconstpointer           ptr1,
                                         gconstpointer           ptr2);
@@ -538,6 +526,10 @@ SummaryView *summary_create(void)
        summaryview->popupfactory = popupfactory;
        summaryview->lock_count = 0;
 
+       /* CLAWS: need this to get the SummaryView * from
+        * the CList */
+       gtk_object_set_data(GTK_OBJECT(ctree), "summaryview", (gpointer)summaryview); 
+
        gtk_widget_show_all(vbox);
 
        return summaryview;
@@ -722,30 +714,16 @@ gboolean summary_show(SummaryView *summaryview, FolderItem *item,
                        summary_unlock(summaryview);
                        summary_execute(summaryview);
                        summary_lock(summaryview);
-               } else if (G_ALERTALTERNATE == val)
-                       summary_write_cache(summaryview);
-               else {
+               } else if (G_ALERTALTERNATE == val) {
+                       /* DO NOTHING */
+               else {
                        summary_unlock(summaryview);
                        inc_unlock();
                        return FALSE;
                }
                folder_update_op_count();
-       } else {
-               /* 
-                * CLAWS: summary_show() is responsible for updating the caches. 
-                * after filtering inc.c::inc_finished() forces the update of
-                * the cache by indirectly calling summary_show() (by re-selecting
-                * the currently selected mail folder).  
-                * this collides with the new filtering system that may have set 
-                * any message flag before calling summary_show(). 
-                * we can prevent this cache-write by checking the opened member
-                * of the folderview. if this is NULL, the folderview forced
-                * an update of the summary view.
-                */
-               if (summaryview->folderview->opened) 
-                       summary_write_cache(summaryview);
-       }       
-        
+       }
+       
        summaryview->folderview->opened = selected_node;
 
        gtk_clist_freeze(GTK_CLIST(ctree));
@@ -794,9 +772,9 @@ gboolean summary_show(SummaryView *summaryview, FolderItem *item,
        BY OTHER PROGRAMS TO THE FOLDER
 */
        mlist = folder_item_get_msg_list(item);
-
+#if 0
        summary_processing(summaryview, mlist);
-
+#endif
        for(cur = mlist ; cur != NULL ; cur = g_slist_next(cur)) {
                MsgInfo * msginfo = (MsgInfo *) cur->data;
 
@@ -808,8 +786,6 @@ gboolean summary_show(SummaryView *summaryview, FolderItem *item,
                }
        }
 
-       summaryview->killed_messages = NULL;
-
        if (summaryview->folder_item->hide_read_msgs) {
                GSList *not_killed;
                
@@ -824,8 +800,7 @@ gboolean summary_show(SummaryView *summaryview, FolderItem *item,
                             && !MSG_IS_IGNORE_THREAD(msginfo->flags))
                            not_killed = g_slist_append(not_killed, msginfo);
                        else
-                               summaryview->killed_messages = 
-                                       g_slist_append(summaryview->killed_messages, msginfo);
+                               procmsg_msginfo_free(msginfo);
                }
                g_slist_free(mlist);
                mlist = not_killed;
@@ -833,8 +808,7 @@ gboolean summary_show(SummaryView *summaryview, FolderItem *item,
                summary_set_hide_read_msgs_menu(summaryview, FALSE);
        }
 
-       if ((global_scoring || item->prefs->scoring) &&
-           (item->folder->type == F_NEWS)) {
+       if ((global_scoring || item->prefs->scoring)) {
                GSList *not_killed;
                gint kill_score;
 
@@ -845,12 +819,10 @@ gboolean summary_show(SummaryView *summaryview, FolderItem *item,
                for(cur = mlist ; cur != NULL ; cur = g_slist_next(cur)) {
                        MsgInfo * msginfo = (MsgInfo *) cur->data;
 
-                       if (MSG_IS_NEWS(msginfo->flags) &&
-                           (msginfo->score <= kill_score))
-                               summaryview->killed_messages = g_slist_append(summaryview->killed_messages, msginfo);
+                       if (msginfo->score > kill_score)
+                               not_killed = g_slist_append(not_killed, msginfo);
                        else
-                               not_killed = g_slist_append(not_killed,
-                                                           msginfo);
+                               procmsg_msginfo_free(msginfo);
                }
                g_slist_free(mlist);
                mlist = not_killed;
@@ -864,17 +836,12 @@ gboolean summary_show(SummaryView *summaryview, FolderItem *item,
 
        g_slist_free(mlist);
 
-       summaryview->folder_item->new    = summaryview->newmsgs;
-       summaryview->folder_item->unread = summaryview->unread;
-       summaryview->folder_item->total  = summaryview->messages;
        folderview_update_msg_num(summaryview->folderview,
                                  summaryview->folderview->opened);
 
        if (item->sort_key != SORT_BY_NONE)
                summary_sort(summaryview, item->sort_key, item->sort_type);
 
-       summary_write_cache(summaryview);
-
        gtk_signal_handler_unblock_by_data(GTK_OBJECT(ctree), summaryview);
 
        gtk_clist_thaw(GTK_CLIST(ctree));
@@ -938,19 +905,9 @@ void summary_clear_list(SummaryView *summaryview)
 {
        GtkCList *clist = GTK_CLIST(summaryview->ctree);
        gint optimal_width;
-       GSList * cur;
 
        gtk_clist_freeze(clist);
 
-       for(cur = summaryview->killed_messages ; cur != NULL ; 
-           cur = g_slist_next(cur)) {
-               MsgInfo * msginfo = (MsgInfo *) cur->data;
-
-               procmsg_msginfo_free(msginfo);
-       }
-       g_slist_free(summaryview->killed_messages);
-       summaryview->killed_messages = NULL;
-
        gtk_ctree_pre_recursive(GTK_CTREE(summaryview->ctree),
                                NULL, summary_free_msginfo_func, NULL);
 
@@ -1061,7 +1018,6 @@ static void summary_set_menu_sensitive(SummaryView *summaryview)
                menu_set_sensitive(ifactory, "/Move...", FALSE);
 
        menu_set_sensitive(ifactory, "/Delete", TRUE);
-       menu_set_sensitive(ifactory, "/Select thread", TRUE);
        menu_set_sensitive(ifactory, "/Select all", TRUE);
        menu_set_sensitive(ifactory, "/Copy...", TRUE);
        menu_set_sensitive(ifactory, "/Execute", TRUE);
@@ -1099,6 +1055,7 @@ static void summary_set_menu_sensitive(SummaryView *summaryview)
        else
                menu_set_sensitive(ifactory, "/Re-edit", FALSE);
 
+       menu_set_sensitive(ifactory, "/Select thread", sens);
        menu_set_sensitive(ifactory, "/Save as...", sens);
        menu_set_sensitive(ifactory, "/Print...",   TRUE);
 
@@ -1691,10 +1648,6 @@ static void summary_status_show(SummaryView *summaryview)
                return;
        }
 
-       summaryview->newmsgs    = summaryview->folder_item->new;
-       summaryview->unread     = summaryview->folder_item->unread;
-       summaryview->messages   = summaryview->folder_item->total;
-
        rowlist = GTK_CLIST(summaryview->ctree)->selection;
        for (cur = rowlist; cur != NULL; cur = cur->next) {
                msginfo = gtk_ctree_node_get_row_data
@@ -1909,7 +1862,14 @@ void summary_sort(SummaryView *summaryview,
                cmp_func = (GtkCListCompareFunc)summary_cmp_by_from;
                break;
        case SORT_BY_SUBJECT:
-               cmp_func = (GtkCListCompareFunc)summary_cmp_by_subject;
+               {
+                       PrefsFolderItem *prefs = summaryview->folder_item->prefs;
+
+                       if (prefs == NULL) 
+                               cmp_func = (GtkCListCompareFunc)summary_cmp_by_subject;
+                       else
+                               cmp_func = (GtkCListCompareFunc)summary_cmp_by_simplified_subject;
+               }                               
                break;
        case SORT_BY_SCORE:
                cmp_func = (GtkCListCompareFunc)summary_cmp_by_score;
@@ -1944,7 +1904,7 @@ void summary_sort(SummaryView *summaryview,
        summary_set_column_titles(summaryview);
        summary_set_menu_sensitive(summaryview);
 
-       gtk_ctree_sort_recursive(ctree, NULL);
+       gtk_sctree_sort_recursive(ctree, NULL);
 
        gtk_ctree_node_moveto(ctree, summaryview->selected, -1, 0.5, 0);
 
@@ -2096,120 +2056,6 @@ static void summary_set_ctree_from_list(SummaryView *summaryview,
        }
 }
 
-struct wcachefp
-{
-       FILE *cache_fp;
-       FILE *mark_fp;
-};
-
-gint summary_write_cache(SummaryView *summaryview)
-{
-       struct wcachefp fps;
-       GtkCTree *ctree = GTK_CTREE(summaryview->ctree);
-       gint ver = CACHE_VERSION;
-       gchar *buf;
-       gchar *cachefile, *markfile;
-       GSList *cur;
-       gint filemode = 0;
-       PrefsFolderItem *prefs;
-
-       if (!summaryview->folder_item || !summaryview->folder_item->path)
-               return -1;
-               
-       if (!summaryview->folder_item->cache)
-               return -1;
-       
-       folder_item_write_cache(summaryview->folder_item);
-       return 0;
-
-       if (summaryview->folder_item->folder->update_mark != NULL)
-               summaryview->folder_item->folder->update_mark(summaryview->folder_item->folder, summaryview->folder_item);
-
-       cachefile = folder_item_get_cache_file(summaryview->folder_item);
-       g_return_val_if_fail(cachefile != NULL, -1);
-       if ((fps.cache_fp = fopen(cachefile, "wb")) == NULL) {
-               FILE_OP_ERROR(cachefile, "fopen");
-               g_free(cachefile);
-               return -1;
-       }
-       if (change_file_mode_rw(fps.cache_fp, cachefile) < 0)
-               FILE_OP_ERROR(cachefile, "chmod");
-
-       prefs = summaryview->folder_item->prefs;
-        if (prefs && prefs->enable_folder_chmod && prefs->folder_chmod) {
-               /* for cache file */
-               filemode = prefs->folder_chmod;
-               if (filemode & S_IRGRP) filemode |= S_IWGRP;
-               if (filemode & S_IROTH) filemode |= S_IWOTH;
-#if HAVE_FCHMOD
-               fchmod(fileno(fps.cache_fp), filemode);
-#else
-               chmod(cachefile, filemode);
-#endif
-        }
-
-       g_free(cachefile);
-
-       markfile = folder_item_get_mark_file(summaryview->folder_item);
-       if ((fps.mark_fp = fopen(markfile, "wb")) == NULL) {
-               FILE_OP_ERROR(markfile, "fopen");
-               fclose(fps.cache_fp);
-               g_free(markfile);
-               return -1;
-       }
-       if (change_file_mode_rw(fps.mark_fp, markfile) < 0)
-               FILE_OP_ERROR(markfile, "chmod");
-        if (prefs && prefs->enable_folder_chmod && prefs->folder_chmod) {
-#if HAVE_FCHMOD
-               fchmod(fileno(fps.mark_fp), filemode);
-#else
-               chmod(markfile, filemode);
-#endif
-        }
-
-       g_free(markfile);
-
-       buf = g_strdup_printf(_("Writing summary cache (%s)..."),
-                             summaryview->folder_item->path);
-       debug_print(buf);
-       STATUSBAR_PUSH(summaryview->mainwin, buf);
-       g_free(buf);
-
-       WRITE_CACHE_DATA_INT(ver, fps.cache_fp);
-       ver = MARK_VERSION;
-       WRITE_CACHE_DATA_INT(ver, fps.mark_fp);
-
-       gtk_ctree_pre_recursive(ctree, NULL, summary_write_cache_func, &fps);
-
-       for (cur = summaryview->killed_messages; cur != NULL; cur = cur->next) {
-               MsgInfo *msginfo = (MsgInfo *)cur->data;
-               procmsg_write_cache(msginfo, fps.cache_fp);
-               procmsg_write_flags(msginfo, fps.mark_fp);
-       }
-
-       procmsg_flush_mark_queue(summaryview->folder_item, fps.mark_fp);
-
-       fclose(fps.cache_fp);
-       fclose(fps.mark_fp);
-
-       debug_print(_("done.\n"));
-       STATUSBAR_POP(summaryview->mainwin);
-
-       return 0;
-}
-
-static void summary_write_cache_func(GtkCTree *ctree, GtkCTreeNode *node,
-                                    gpointer data)
-{
-       struct wcachefp *fps = data;
-       MsgInfo *msginfo = gtk_ctree_node_get_row_data(ctree, node);
-
-       if (msginfo == NULL) return;
-
-       procmsg_write_cache(msginfo, fps->cache_fp);
-       procmsg_write_flags(msginfo, fps->mark_fp);
-}
-
 static gchar *summary_complete_address(const gchar *addr)
 {
        gint count;
@@ -2266,16 +2112,22 @@ static void summary_set_header(SummaryView *summaryview, gchar *text[],
            !MSG_IS_NEWS(msginfo->flags)) {
                gchar *addr = NULL;
 
-               if (prefs_common.use_addr_book) {
-                       Xstrdup_a(addr, msginfo->from, return);
-                       extract_address(addr);
+               Xstrdup_a(addr, msginfo->from, return);
+               extract_address(addr);
 
+               if (prefs_common.use_addr_book) {
                        if (account_find_from_address(addr)) {
                                addr = summary_complete_address(msginfo->to);
                                g_free(to);
                                to   = g_strconcat("-->", addr == NULL ? msginfo->to : addr, NULL);
                                text[col_pos[S_COL_FROM]] = to;
                        }
+               } else {
+                       if (cur_account && cur_account->address && !strcmp( addr, cur_account->address)) {
+                               g_free(to);
+                               to = g_strconcat("-->", msginfo->to, NULL);
+                               text[col_pos[S_COL_FROM]] = to;
+                       }
                }
        }
 
@@ -2349,12 +2201,11 @@ static void summary_display_msg_full(SummaryView *summaryview,
        }
        g_free(filename);
 
-/* NOT NEEDED ANYMORE
        if (MSG_IS_NEW(msginfo->flags) && !MSG_IS_IGNORE_THREAD(msginfo->flags))
                summaryview->newmsgs--;
        if (MSG_IS_UNREAD(msginfo->flags) && !MSG_IS_IGNORE_THREAD(msginfo->flags))
                summaryview->unread--;
-*/
+
        procmsg_msginfo_unset_flags(msginfo, MSG_NEW | MSG_UNREAD, 0);
        summary_set_row_marks(summaryview, row);
        gtk_clist_thaw(GTK_CLIST(ctree));
@@ -2731,12 +2582,12 @@ static void summary_mark_row_as_read(SummaryView *summaryview,
        MsgInfo *msginfo;
 
        msginfo = gtk_ctree_node_get_row_data(ctree, row);
-/* NOT NEEDED ANYMORE
+
        if (MSG_IS_NEW(msginfo->flags) && !MSG_IS_IGNORE_THREAD(msginfo->flags))
                summaryview->newmsgs--;
        if (MSG_IS_UNREAD(msginfo->flags) && !MSG_IS_IGNORE_THREAD(msginfo->flags))
                summaryview->unread--;
-*/
+
        procmsg_msginfo_unset_flags(msginfo, MSG_NEW | MSG_UNREAD, 0);
        summary_set_row_marks(summaryview, row);
        debug_print(_("Message %d is marked as read\n"),
@@ -2787,10 +2638,9 @@ static void summary_mark_row_as_unread(SummaryView *summaryview,
                procmsg_msginfo_unset_flags(msginfo, MSG_DELETED, 0);
                summaryview->deleted--;
        }
-/* NOT NEEDED ANYMORE
+
        if (!MSG_IS_UNREAD(msginfo->flags) && !MSG_IS_IGNORE_THREAD(msginfo->flags))
                summaryview->unread++;
-*/
 
        procmsg_msginfo_unset_flags(msginfo, MSG_REPLIED | MSG_FORWARDED, 0);
        procmsg_msginfo_set_flags(msginfo, MSG_UNREAD, 0);
@@ -3368,8 +3218,6 @@ gboolean summary_execute(SummaryView *summaryview)
        summary_update_status(summaryview);
        summary_status_show(summaryview);
 
-       summary_write_cache(summaryview);
-
        gtk_ctree_node_moveto(ctree, summaryview->selected, -1, 0.5, 0);
 
        gtk_clist_thaw(clist);
@@ -3646,7 +3494,7 @@ void summary_unthread(SummaryView *summaryview)
        debug_print(_("Unthreading..."));
        STATUSBAR_PUSH(summaryview->mainwin, _("Unthreading..."));
        main_window_cursor_wait(summaryview->mainwin);
-
+       
        gtk_signal_handler_block_by_func(GTK_OBJECT(ctree),
                                         summary_tree_collapsed, summaryview);
        gtk_clist_freeze(GTK_CLIST(ctree));
@@ -3663,6 +3511,9 @@ void summary_unthread(SummaryView *summaryview)
                }
        }
 
+       /* CLAWS: and sort it */
+       gtk_sctree_sort_recursive(ctree, NULL); 
+
        gtk_clist_thaw(GTK_CLIST(ctree));
        gtk_signal_handler_unblock_by_func(GTK_OBJECT(ctree),
                                           summary_tree_collapsed, summaryview);
@@ -3729,61 +3580,6 @@ static void summary_unthread_for_exec_func(GtkCTree *ctree, GtkCTreeNode *node,
        }
 }
 
-void summary_processing(SummaryView *summaryview, GSList * mlist)
-{
-       GSList * processing_list;
-       FolderItem * folder_item;
-       GSList * cur;
-       gchar * buf;
-
-       folder_item = summaryview->folder_item;
-       if (folder_item == NULL)
-               return;
-
-       processing_list = folder_item->prefs->processing;
-
-       if (processing_list == NULL)
-               return;
-
-       summary_lock(summaryview);
-       
-       buf = g_strdup_printf(_("Processing (%s)..."), folder_item->path);
-       debug_print(buf);
-       STATUSBAR_PUSH(summaryview->mainwin, buf);
-       g_free(buf);
-
-       main_window_cursor_wait(summaryview->mainwin);
-
-       summaryview->folder_table = g_hash_table_new(NULL, NULL);
-
-       for(cur = mlist ; cur != NULL ; cur = cur->next) {
-               MsgInfo * msginfo;
-
-               msginfo = (MsgInfo *) cur->data;
-               filter_message_by_msginfo(processing_list, msginfo,
-                                         summaryview->folder_table);
-       }
-       
-       /* folder_item_scan_foreach(summaryview->folder_table); */
-       folderview_update_item_foreach(summaryview->folder_table);
-       
-       g_hash_table_destroy(summaryview->folder_table);
-       summaryview->folder_table = NULL;
-       
-       if (prefs_common.immediate_exec) {
-               summary_unlock(summaryview);
-               summary_execute(summaryview);
-               summary_lock(summaryview);
-       } else
-               summary_status_show(summaryview);
-
-       debug_print(_("done.\n"));
-       STATUSBAR_POP(summaryview->mainwin);
-       main_window_cursor_normal(summaryview->mainwin);
-
-       summary_unlock(summaryview);
-}
-
 void summary_expand_threads(SummaryView *summaryview)
 {
        GtkCTree *ctree = GTK_CTREE(summaryview->ctree);
@@ -4036,6 +3832,14 @@ void summary_reply(SummaryView *summaryview, ComposeMode mode)
        text = gtkut_editable_get_selection
                (GTK_EDITABLE(summaryview->messageview->textview->text));
 
+       if (!text && summaryview->messageview->type == MVIEW_MIME
+           && summaryview->messageview->mimeview->type == MIMEVIEW_TEXT
+           && summaryview->messageview->mimeview->textview
+           && !summaryview->messageview->mimeview->textview->default_text) {
+               text = gtkut_editable_get_selection (GTK_EDITABLE 
+                        (summaryview->messageview->mimeview->textview->text));   
+       }
+       
        switch (mode) {
        case COMPOSE_REPLY:
                compose_reply(msginfo, prefs_common.reply_with_quote,
@@ -4433,10 +4237,6 @@ void summary_set_column_order(SummaryView *summaryview)
 
        item = summaryview->folder_item;
 
-       summary_lock(summaryview);
-       summary_write_cache(summaryview);
-       summary_unlock(summaryview);
-       
        summary_clear_all(summaryview);
        gtk_widget_destroy(summaryview->ctree);
 
@@ -4912,6 +4712,38 @@ static gint summary_cmp_by_subject(GtkCList *clist,
        return strcasecmp(msginfo1->subject, msginfo2->subject);
 }
 
+static gint summary_cmp_by_simplified_subject
+       (GtkCList *clist, gconstpointer ptr1, gconstpointer ptr2)
+{
+       const PrefsFolderItem *prefs;
+       const gchar *str1, *str2;
+       const GtkCListRow *r1 = (const GtkCListRow *) ptr1;
+       const GtkCListRow *r2 = (const GtkCListRow *) ptr2;
+       const MsgInfo *msginfo1 = r1->data;
+       const MsgInfo *msginfo2 = r2->data;
+       const SummaryView *sv = gtk_object_get_data(GTK_OBJECT(clist), "summaryview");
+       
+       g_return_val_if_fail(sv, -1);
+       g_return_val_if_fail(msginfo1 == NULL || msginfo2 == NULL, -1);
+       
+       str1 = GTK_CELL_TEXT(r1->cell[sv->col_pos[S_COL_SUBJECT]])->text;
+       str2 = GTK_CELL_TEXT(r2->cell[sv->col_pos[S_COL_SUBJECT]])->text;
+
+       if (!str1)
+               return str2 != NULL;
+
+       if (!str2)
+               return -1;
+
+       prefs = msginfo1->folder->prefs;
+       if (!prefs)
+               prefs = msginfo2->folder->prefs;
+       if (!prefs)
+               return -1;
+       
+       return strcasecmp(str1, str2);
+}
+
 static gint summary_cmp_by_label(GtkCList *clist,
                                 gconstpointer ptr1, gconstpointer ptr2)
 {
@@ -5008,12 +4840,12 @@ static void summary_ignore_thread_func(GtkCTree *ctree, GtkCTreeNode *row, gpoin
        MsgInfo *msginfo;
 
        msginfo = gtk_ctree_node_get_row_data(ctree, row);
-/* NOT NEEDED ANYMORE
+
        if (MSG_IS_NEW(msginfo->flags))
                summaryview->newmsgs--;
        if (MSG_IS_UNREAD(msginfo->flags))
                summaryview->unread--;
-*/
+
        procmsg_msginfo_set_flags(msginfo, MSG_IGNORE_THREAD, 0);
 
        summary_set_row_marks(summaryview, row);
@@ -5021,7 +4853,6 @@ static void summary_ignore_thread_func(GtkCTree *ctree, GtkCTreeNode *row, gpoin
            msginfo->msgnum);
 }
 
-
 static void summary_ignore_thread(SummaryView *summaryview)
 {
        GtkCTree *ctree = GTK_CTREE(summaryview->ctree);
@@ -5040,12 +4871,12 @@ static void summary_unignore_thread_func(GtkCTree *ctree, GtkCTreeNode *row, gpo
        MsgInfo *msginfo;
 
        msginfo = gtk_ctree_node_get_row_data(ctree, row);
-/* NOT NEEDED ANYMORE
+
        if (MSG_IS_NEW(msginfo->flags))
                summaryview->newmsgs++;
        if (MSG_IS_UNREAD(msginfo->flags))
                summaryview->unread++;
-*/
+
        procmsg_msginfo_unset_flags(msginfo, MSG_IGNORE_THREAD, 0);
 
        summary_set_row_marks(summaryview, row);
@@ -5065,6 +4896,7 @@ static void summary_unignore_thread(SummaryView *summaryview)
        summary_status_show(summaryview);
 }
 
+#if 0 /* OLD PROCESSING */
 static gboolean processing_apply_func(GNode *node, gpointer data)
 {
        FolderItem *item;
@@ -5125,6 +4957,7 @@ void processing_apply(SummaryView * summaryview)
                                processing_apply_func, summaryview);
        }
 }
+#endif
 
 void summary_toggle_show_read_messages(SummaryView *summaryview)
 {
@@ -5204,8 +5037,6 @@ void summary_reflect_prefs_pixmap_theme(SummaryView *summaryview)
        gtk_widget_show(pixmap);
        summaryview->folder_pixmap = pixmap; 
 
-       summary_write_cache(summaryview);
-
        folderview_unselect(summaryview->folderview);
        folderview_select(summaryview->folderview, summaryview->folder_item);
 }