Coverity fixes
[claws.git] / src / summary_search.c
index aa325cebada3a33e14e12e5fbc0de91e5c0cad0a..a085520497af86cc131e34fd2bd348620c580a9d 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2009 Hiroyuki Yamamoto and the Claws Mail team
+ * Copyright (C) 1999-2013 Hiroyuki Yamamoto and the Claws Mail team
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -19,6 +19,7 @@
 
 #ifdef HAVE_CONFIG_H
 #  include "config.h"
+#include "claws-features.h"
 #endif
 
 #include "defs.h"
@@ -27,7 +28,9 @@
 #include <glib/gi18n.h>
 #include <gdk/gdkkeysyms.h>
 #include <gtk/gtk.h>
+#if !GTK_CHECK_VERSION(3, 0, 0)
 #include "gtk/gtksctree.h"
+#endif
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include "prefs_gtk.h"
 #include "manage_window.h"
 #include "alertpanel.h"
+#include "advsearch.h"
 #include "matcher.h"
 #include "matcher_parser.h"
 #include "prefs_matcher.h"
 #include "manual.h"
 #include "prefs_common.h"
+#include "statusbar.h"
 
 static struct SummarySearchWindow {
        GtkWidget *window;
@@ -76,7 +81,11 @@ static struct SummarySearchWindow {
 
        SummaryView *summaryview;
 
-       MatcherList                     *matcher_list;
+       AdvancedSearch  *advsearch;
+       gboolean        is_fast;
+       gboolean        matcher_is_outdated;
+       gboolean        search_in_progress;
+       GHashTable      *matched_msgnums;
 
        gboolean is_searching;
        gboolean from_entry_has_focus;
@@ -86,8 +95,14 @@ static struct SummarySearchWindow {
        gboolean adv_condition_entry_has_focus;
 } search_window;
 
+static gchar* add_history_get(GtkWidget *from, GList **history);
+
 static void summary_search_create      (void);
 
+static gboolean summary_search_verify_match            (MsgInfo *msg);
+static gboolean summary_search_prepare_matcher         ();
+static gboolean summary_search_prereduce_msg_list      ();
+
 static void summary_search_execute     (gboolean        backward,
                                         gboolean        search_all);
 
@@ -130,28 +145,47 @@ static gboolean adv_condition_entry_focus_evt_in(GtkWidget *widget, GdkEventFocu
                                  gpointer data);
 static gboolean adv_condition_entry_focus_evt_out(GtkWidget *widget, GdkEventFocus *event,
                                  gpointer data);
-#ifndef MAEMO
 static gboolean key_pressed            (GtkWidget      *widget,
                                         GdkEventKey    *event,
                                         gpointer        data);
-#endif
 
-#if !GTK_CHECK_VERSION(2,14,0)
-/* Work around http://bugzilla.gnome.org/show_bug.cgi?id=56070 */
 #define GTK_BUTTON_SET_SENSITIVE(widget,sensitive) {                                   \
-       gboolean in_btn = FALSE;                                                        \
-       if (GTK_IS_BUTTON(widget))                                                      \
-               in_btn = GTK_BUTTON(widget)->in_button;                                 \
        gtk_widget_set_sensitive(widget, sensitive);                                    \
-       if (GTK_IS_BUTTON(widget))                                                      \
-               GTK_BUTTON(widget)->in_button = in_btn;                                 \
 }
+
+static gchar* add_history_get(GtkWidget *from, GList **history)
+{
+       gchar *result;
+
+#if !GTK_CHECK_VERSION(2, 24, 0)
+       result = gtk_combo_box_get_active_text(GTK_COMBO_BOX(from));
 #else
-#define GTK_BUTTON_SET_SENSITIVE(widget,sensitive) {                                   \
-       gtk_widget_set_sensitive(widget, sensitive);                                    \
-}
+       result = gtk_combo_box_text_get_active_text(GTK_COMBO_BOX_TEXT(from));
+#endif
+       if (!result)
+               result = gtk_editable_get_chars(GTK_EDITABLE(gtk_bin_get_child(GTK_BIN(from))), 0, -1);
+
+       if (result && result[0] != '\0') {
+               /* add to history */
+#if !GTK_CHECK_VERSION(2, 24, 0)
+               combobox_unset_popdown_strings(GTK_COMBO_BOX(from));
+#else
+               combobox_unset_popdown_strings(GTK_COMBO_BOX_TEXT(from));
+#endif
+               *history = add_history(*history, result);
+#if !GTK_CHECK_VERSION(2, 24, 0)
+               combobox_set_popdown_strings(GTK_COMBO_BOX(from), *history);
+#else
+               combobox_set_popdown_strings(GTK_COMBO_BOX_TEXT(from), *history);
 #endif
 
+               return result;
+       } else {
+               g_free(result);
+               return NULL;
+       }
+}
+
 void summary_search(SummaryView *summaryview)
 {
        if (!search_window.window) {
@@ -220,7 +254,6 @@ static void summary_search_create(void)
        GtkWidget *close_btn;
        GtkWidget *stop_btn;
        gboolean is_searching = FALSE;
-       CLAWS_TIP_DECL();
 
        window = gtkut_window_new(GTK_WINDOW_TOPLEVEL, "summary_search");
        gtk_window_set_title(GTK_WINDOW (window), _("Search messages"));
@@ -228,12 +261,8 @@ static void summary_search_create(void)
        gtk_container_set_border_width(GTK_CONTAINER (window), 8);
        g_signal_connect(G_OBJECT(window), "delete_event",
                         G_CALLBACK(gtk_widget_hide_on_delete), NULL);
-#ifdef MAEMO
-       maemo_connect_key_press_to_mainwindow(GTK_WINDOW(window));
-#else
        g_signal_connect(G_OBJECT(window), "key_press_event",
                         G_CALLBACK(key_pressed), NULL);
-#endif
        MANAGE_WINDOW_SIGNALS_CONNECT(window);
 
        vbox1 = gtk_vbox_new (FALSE, 0);
@@ -264,11 +293,20 @@ static void summary_search_create(void)
        gtk_table_set_row_spacings (GTK_TABLE (table1), 8);
        gtk_table_set_col_spacings (GTK_TABLE (table1), 8);
 
+#if !GTK_CHECK_VERSION(2, 24, 0)
        from_entry = gtk_combo_box_entry_new_text ();
+#else
+       from_entry = gtk_combo_box_text_new_with_entry ();
+#endif
        gtk_combo_box_set_active(GTK_COMBO_BOX(from_entry), -1);
        if (prefs_common.summary_search_from_history)
+#if !GTK_CHECK_VERSION(2, 24, 0)
                combobox_set_popdown_strings(GTK_COMBO_BOX(from_entry),
                                prefs_common.summary_search_from_history);
+#else
+               combobox_set_popdown_strings(GTK_COMBO_BOX_TEXT(from_entry),
+                               prefs_common.summary_search_from_history);
+#endif
        gtk_widget_show (from_entry);
        gtk_table_attach (GTK_TABLE (table1), from_entry, 1, 3, 0, 1,
                          GTK_EXPAND|GTK_FILL, 0, 0, 0);
@@ -279,11 +317,20 @@ static void summary_search_create(void)
        g_signal_connect(G_OBJECT(gtk_bin_get_child(GTK_BIN((from_entry)))),
                         "focus_out_event", G_CALLBACK(from_entry_focus_evt_out), NULL);
 
+#if !GTK_CHECK_VERSION(2, 24, 0)
        to_entry = gtk_combo_box_entry_new_text ();
+#else
+       to_entry = gtk_combo_box_text_new_with_entry ();
+#endif
        gtk_combo_box_set_active(GTK_COMBO_BOX(to_entry), -1);
        if (prefs_common.summary_search_to_history)
+#if !GTK_CHECK_VERSION(2, 24, 0)
                combobox_set_popdown_strings(GTK_COMBO_BOX(to_entry),
                                prefs_common.summary_search_to_history);
+#else
+               combobox_set_popdown_strings(GTK_COMBO_BOX_TEXT(to_entry),
+                               prefs_common.summary_search_to_history);
+#endif
        gtk_widget_show (to_entry);
        gtk_table_attach (GTK_TABLE (table1), to_entry, 1, 3, 1, 2,
                          GTK_EXPAND|GTK_FILL, 0, 0, 0);
@@ -294,11 +341,20 @@ static void summary_search_create(void)
        g_signal_connect(G_OBJECT(gtk_bin_get_child(GTK_BIN((to_entry)))),
                         "focus_out_event", G_CALLBACK(to_entry_focus_evt_out), NULL);
 
+#if !GTK_CHECK_VERSION(2, 24, 0)
        subject_entry = gtk_combo_box_entry_new_text ();
+#else
+       subject_entry = gtk_combo_box_text_new_with_entry ();
+#endif
        gtk_combo_box_set_active(GTK_COMBO_BOX(subject_entry), -1);
        if (prefs_common.summary_search_subject_history)
+#if !GTK_CHECK_VERSION(2, 24, 0)
                combobox_set_popdown_strings(GTK_COMBO_BOX(subject_entry),
                                prefs_common.summary_search_subject_history);
+#else
+               combobox_set_popdown_strings(GTK_COMBO_BOX_TEXT(subject_entry),
+                               prefs_common.summary_search_subject_history);
+#endif
        gtk_widget_show (subject_entry);
        gtk_table_attach (GTK_TABLE (table1), subject_entry, 1, 3, 2, 3,
                          GTK_EXPAND|GTK_FILL, 0, 0, 0);
@@ -309,11 +365,20 @@ static void summary_search_create(void)
        g_signal_connect(G_OBJECT(gtk_bin_get_child(GTK_BIN((subject_entry)))),
                         "focus_out_event", G_CALLBACK(subject_entry_focus_evt_out), NULL);
 
+#if !GTK_CHECK_VERSION(2, 24, 0)
        body_entry = gtk_combo_box_entry_new_text ();
+#else
+       body_entry = gtk_combo_box_text_new_with_entry ();
+#endif
        gtk_combo_box_set_active(GTK_COMBO_BOX(body_entry), -1);
        if (prefs_common.summary_search_body_history)
+#if !GTK_CHECK_VERSION(2, 24, 0)
                combobox_set_popdown_strings(GTK_COMBO_BOX(body_entry),
                                prefs_common.summary_search_body_history);
+#else
+               combobox_set_popdown_strings(GTK_COMBO_BOX_TEXT(body_entry),
+                               prefs_common.summary_search_body_history);
+#endif
        gtk_widget_show (body_entry);
        gtk_table_attach (GTK_TABLE (table1), body_entry, 1, 3, 3, 4,
                          GTK_EXPAND|GTK_FILL, 0, 0, 0);
@@ -324,11 +389,20 @@ static void summary_search_create(void)
        g_signal_connect(G_OBJECT(gtk_bin_get_child(GTK_BIN((body_entry)))),
                         "focus_out_event", G_CALLBACK(body_entry_focus_evt_out), NULL);
 
+#if !GTK_CHECK_VERSION(2, 24, 0)
        adv_condition_entry = gtk_combo_box_entry_new_text ();
+#else
+       adv_condition_entry = gtk_combo_box_text_new_with_entry ();
+#endif
        gtk_combo_box_set_active(GTK_COMBO_BOX(adv_condition_entry), -1);
        if (prefs_common.summary_search_adv_condition_history)
+#if !GTK_CHECK_VERSION(2, 24, 0)
                combobox_set_popdown_strings(GTK_COMBO_BOX(adv_condition_entry),
                                prefs_common.summary_search_adv_condition_history);
+#else
+               combobox_set_popdown_strings(GTK_COMBO_BOX_TEXT(adv_condition_entry),
+                               prefs_common.summary_search_adv_condition_history);
+#endif
        gtk_widget_show (adv_condition_entry);
        gtk_table_attach (GTK_TABLE (table1), adv_condition_entry, 1, 2, 4, 5,
                          GTK_EXPAND|GTK_FILL, 0, 0, 0);
@@ -480,162 +554,191 @@ static void summary_search_create(void)
        search_window.next_btn = next_btn;
        search_window.close_btn = close_btn;
        search_window.stop_btn = stop_btn;
-       search_window.matcher_list = NULL;
+       search_window.advsearch = NULL;
+       search_window.matcher_is_outdated = TRUE;
+       search_window.search_in_progress = FALSE;
+       search_window.matched_msgnums = NULL;
        search_window.is_searching = is_searching;
-#ifdef MAEMO
-       maemo_window_full_screen_if_needed(GTK_WINDOW(search_window.window));
-#endif
 }
 
-static void summary_search_execute(gboolean backward, gboolean search_all)
+static gboolean summary_search_verify_match(MsgInfo *msg)
+{
+       gpointer msgnum = GUINT_TO_POINTER(msg->msgnum);
+
+       if (g_hash_table_lookup(search_window.matched_msgnums, msgnum) != NULL)
+               return TRUE;
+       else
+               return FALSE;
+}
+
+static gboolean summary_search_progress_cb(gpointer data, guint at, guint matched, guint total)
+{
+       if (!search_window.is_searching) {
+               search_window.matcher_is_outdated = TRUE;
+               return FALSE;
+       }
+
+       return summaryview_search_root_progress(search_window.summaryview, at, matched, total);
+}
+
+static gboolean summary_search_prepare_matcher()
 {
-       SummaryView *summaryview = search_window.summaryview;
-       GtkCMCTree *ctree = GTK_CMCTREE(summaryview->ctree);
-       GtkCMCTreeNode *node;
-       MsgInfo *msginfo;
        gboolean adv_search;
        gboolean bool_and = FALSE;
        gboolean case_sens = FALSE;
-       gboolean all_searched = FALSE;
-       gboolean matched = FALSE;
-       gboolean body_matched = FALSE;
+       gchar *matcher_str;
+       gint match_type;
        gchar *from_str = NULL, *to_str = NULL, *subject_str = NULL;
        gchar *body_str = NULL;
-       gchar *adv_condition = NULL;
-       gboolean is_fast = TRUE;
-       gint interval = 1000;
-       gint i = 0;
        GSList *matchers = NULL;
 
-       if (summary_is_locked(summaryview)) {
-               return;
+       if (!search_window.matcher_is_outdated)
+               return TRUE;
+
+       if (search_window.advsearch == NULL) {
+               search_window.advsearch = advsearch_new();
+               advsearch_set_on_error_cb(search_window.advsearch, NULL, NULL); /* TODO */
+               advsearch_set_on_progress_cb(search_window.advsearch, 
+                       summary_search_progress_cb, NULL);
        }
-       summary_lock(summaryview);
 
        adv_search = gtk_toggle_button_get_active
                (GTK_TOGGLE_BUTTON(search_window.adv_search_checkbtn));
 
-       if (search_window.matcher_list != NULL) {
-               matcherlist_free(search_window.matcher_list);
-               search_window.matcher_list = NULL;
-       }
        if (adv_search) {
-               adv_condition = gtk_combo_box_get_active_text(GTK_COMBO_BOX(search_window.adv_condition_entry));
-               if (!adv_condition)
-                       adv_condition = gtk_editable_get_chars(
-                                       GTK_EDITABLE(gtk_bin_get_child(GTK_BIN(search_window.adv_condition_entry))),0,-1);
-               if (adv_condition && adv_condition[0] != '\0') {
-
-                       /* add to history */
-                       combobox_unset_popdown_strings(GTK_COMBO_BOX(search_window.adv_condition_entry));
-                       prefs_common.summary_search_adv_condition_history = add_history(
-                                       prefs_common.summary_search_adv_condition_history, adv_condition);
-                       combobox_set_popdown_strings(GTK_COMBO_BOX(search_window.adv_condition_entry),
-                                       prefs_common.summary_search_adv_condition_history);
-
-                       search_window.matcher_list = matcher_parser_get_cond((gchar*)adv_condition, &is_fast);
-                       if (!is_fast)
-                               interval = 100;
-                       /* TODO: check for condition parsing error and show an error dialog */
-                       g_free(adv_condition);
-               } else {
-                       /* TODO: warn if no search condition? (or make buttons enabled only when
-                               at least one search condition has been set */
-                       summary_unlock(summaryview);
-                       return;
-               }
+               matcher_str = add_history_get(search_window.adv_condition_entry, &prefs_common.summary_search_adv_condition_history);
        } else {
-               bool_and = combobox_get_active_data(
-                               GTK_COMBO_BOX(search_window.bool_optmenu));
-               case_sens = gtk_toggle_button_get_active
-                       (GTK_TOGGLE_BUTTON(search_window.case_checkbtn));
-
-               from_str    = gtk_combo_box_get_active_text(GTK_COMBO_BOX(search_window.from_entry));
-               to_str      = gtk_combo_box_get_active_text(GTK_COMBO_BOX(search_window.to_entry));
-               subject_str = gtk_combo_box_get_active_text(GTK_COMBO_BOX(search_window.subject_entry));
-               body_str    = gtk_combo_box_get_active_text(GTK_COMBO_BOX(search_window.body_entry));
-
-               if (!from_str)
-                       from_str = gtk_editable_get_chars(
-                                       GTK_EDITABLE(gtk_bin_get_child(GTK_BIN(search_window.from_entry))),0,-1);
-               if (!to_str)
-                       to_str = gtk_editable_get_chars(
-                                       GTK_EDITABLE(gtk_bin_get_child(GTK_BIN(search_window.to_entry))),0,-1);
-               if (!subject_str)
-                       subject_str = gtk_editable_get_chars(
-                                       GTK_EDITABLE(gtk_bin_get_child(GTK_BIN(search_window.subject_entry))),0,-1);
-               if (!body_str)
-                       body_str = gtk_editable_get_chars(
-                                       GTK_EDITABLE(gtk_bin_get_child(GTK_BIN(search_window.body_entry))),0,-1);
-
-               if (!from_str || !to_str || !subject_str || !body_str) {
+               MatcherList *matcher_list;
+               bool_and = combobox_get_active_data(GTK_COMBO_BOX(search_window.bool_optmenu));
+               case_sens = gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(search_window.case_checkbtn));
+
+               from_str    = add_history_get(search_window.from_entry, &prefs_common.summary_search_from_history);
+               to_str      = add_history_get(search_window.to_entry, &prefs_common.summary_search_to_history);
+               subject_str = add_history_get(search_window.subject_entry, &prefs_common.summary_search_subject_history);
+               body_str    = add_history_get(search_window.body_entry, &prefs_common.summary_search_body_history);
+
+               if (!from_str && !to_str && !subject_str && !body_str) {
                        /* TODO: warn if no search criteria? (or make buttons enabled only when
                         * at least one search criteria has been set */
-                       summary_unlock(summaryview);
-                       return;
-               }
-               if (    (from_str[0] == '\0') &&
-                               (to_str[0] == '\0') &&
-                               (subject_str[0] == '\0') &&
-                               (body_str[0] == '\0')) {
-                       /* TODO: warn if no search criteria? (or make buttons enabled only when
-                               at least one search criteria has been set */
-                       summary_unlock(summaryview);
-                       return;
+                       return FALSE;
                }
 
-               /* add to history */
-               if (from_str[0] != '\0') {
-                       MatcherProp *prop = matcherprop_new(MATCHCRITERIA_FROM,
-                                               NULL, case_sens ? MATCHTYPE_MATCH:MATCHTYPE_MATCHCASE,
-                                               from_str, 0);
+               match_type = case_sens ? MATCHTYPE_MATCH : MATCHTYPE_MATCHCASE;
+
+               if (from_str) {
+                       MatcherProp *prop = matcherprop_new(MATCHCRITERIA_FROM, NULL, match_type, from_str, 0);
                        matchers = g_slist_append(matchers, prop);
-                       combobox_unset_popdown_strings(GTK_COMBO_BOX(search_window.from_entry));
-                       prefs_common.summary_search_from_history = add_history(
-                                       prefs_common.summary_search_from_history, from_str);
-                       combobox_set_popdown_strings(GTK_COMBO_BOX(search_window.from_entry),
-                                       prefs_common.summary_search_from_history);
                }
-               if (to_str[0] != '\0') {
-                       MatcherProp *prop = matcherprop_new(MATCHCRITERIA_TO,
-                                               NULL, case_sens ? MATCHTYPE_MATCH:MATCHTYPE_MATCHCASE,
-                                               to_str, 0);
+               if (to_str) {
+                       MatcherProp *prop = matcherprop_new(MATCHCRITERIA_TO, NULL, match_type, to_str, 0);
                        matchers = g_slist_append(matchers, prop);
-                       combobox_unset_popdown_strings(GTK_COMBO_BOX(search_window.to_entry));
-                       prefs_common.summary_search_to_history = add_history(
-                                       prefs_common.summary_search_to_history, to_str);
-                       combobox_set_popdown_strings(GTK_COMBO_BOX(search_window.to_entry),
-                                       prefs_common.summary_search_to_history);
                }
-               if (subject_str[0] != '\0') {
-                       MatcherProp *prop = matcherprop_new(MATCHCRITERIA_SUBJECT,
-                                               NULL, case_sens ? MATCHTYPE_MATCH:MATCHTYPE_MATCHCASE,
-                                               subject_str, 0);
+               if (subject_str) {
+                       MatcherProp *prop = matcherprop_new(MATCHCRITERIA_SUBJECT, NULL, match_type, subject_str, 0);
                        matchers = g_slist_append(matchers, prop);
-                       combobox_unset_popdown_strings(GTK_COMBO_BOX(search_window.subject_entry));
-                       prefs_common.summary_search_subject_history = add_history(
-                                       prefs_common.summary_search_subject_history, subject_str);
-                       combobox_set_popdown_strings(GTK_COMBO_BOX(search_window.subject_entry),
-                                       prefs_common.summary_search_subject_history);
                }
-               if (body_str[0] != '\0') {
-                       MatcherProp *prop = matcherprop_new(MATCHCRITERIA_BODY_PART,
-                                               NULL, case_sens ? MATCHTYPE_MATCH:MATCHTYPE_MATCHCASE,
-                                               body_str, 0);
+               if (body_str) {
+                       MatcherProp *prop = matcherprop_new(MATCHCRITERIA_BODY_PART, NULL, match_type, body_str, 0);
                        matchers = g_slist_append(matchers, prop);
-                       combobox_unset_popdown_strings(GTK_COMBO_BOX(search_window.body_entry));
-                       prefs_common.summary_search_body_history = add_history(
-                                       prefs_common.summary_search_body_history, body_str);
-                       combobox_set_popdown_strings(GTK_COMBO_BOX(search_window.body_entry),
-                                       prefs_common.summary_search_body_history);
                }
-               search_window.matcher_list = matcherlist_new(matchers, bool_and);
+               g_free(from_str);
+               g_free(to_str);
+               g_free(subject_str);
+               g_free(body_str);
+
+               matcher_list = matcherlist_new(matchers, bool_and);
+               if (!matcher_list)
+                       return FALSE;
+               matcher_str = matcherlist_to_string(matcher_list);
+               matcherlist_free(matcher_list);
+       }
+       if (!matcher_str)
+               return FALSE;
+
+       advsearch_set(search_window.advsearch, ADVANCED_SEARCH_EXTENDED,
+                     matcher_str);
+
+       debug_print("Advsearch set: %s\n", matcher_str);
+       g_free(matcher_str);
+
+       if (!advsearch_has_proper_predicate(search_window.advsearch))
+               return FALSE;
+
+       search_window.matcher_is_outdated = FALSE;
+
+       return TRUE;
+}
+
+static gboolean summary_search_prereduce_msg_list()
+{
+       MsgInfoList *msglist = NULL;
+       MsgNumberList *msgnums = NULL;
+       MsgNumberList *cur;
+       SummaryView *summaryview = search_window.summaryview;
+       gboolean result;
+       FolderItem *item = summaryview->folder_item;
+       static GdkCursor *watch_cursor = NULL;
+       if (!watch_cursor)
+               watch_cursor = gdk_cursor_new(GDK_WATCH);
+
+       if (search_window.matcher_is_outdated && !summary_search_prepare_matcher()) {
+               return FALSE;
        }
 
+       main_window_cursor_wait(mainwindow_get_mainwindow());
+       gdk_window_set_cursor(gtk_widget_get_window(search_window.window), watch_cursor);
+       statusbar_print_all(_("Searching in %s... \n"),
+               item->path ? item->path : "(null)");
+
+       result = advsearch_search_msgs_in_folders(search_window.advsearch,
+                       &msglist, item, FALSE);
+       statusbar_pop_all();
+       statusbar_progress_all(0, 0, 0);
+       gdk_window_set_cursor(gtk_widget_get_window(search_window.window), NULL);
+       main_window_cursor_normal(mainwindow_get_mainwindow());
+
+       if (!result)
+               return FALSE;
+       msgnums = procmsg_get_number_list_for_msgs(msglist);
+       procmsg_msg_list_free(msglist);
+
+       if (search_window.matched_msgnums == NULL)
+               search_window.matched_msgnums = g_hash_table_new(g_direct_hash, NULL);
+
+       g_hash_table_remove_all(search_window.matched_msgnums);
+
+       for (cur = msgnums; cur != NULL; cur = cur->next) {
+               g_hash_table_insert(search_window.matched_msgnums, cur->data, GINT_TO_POINTER(1));
+       }
+
+       g_slist_free(msgnums);
+
+       return TRUE;
+}
+
+static void summary_search_execute(gboolean backward, gboolean search_all)
+{
+       SummaryView *summaryview = search_window.summaryview;
+       GtkCMCTree *ctree = GTK_CMCTREE(summaryview->ctree);
+       GtkCMCTreeNode *node;
+       MsgInfo *msginfo;
+       gboolean all_searched = FALSE;
+       gboolean matched = FALSE;
+       gint i = 0;
+
+       if (summary_is_locked(summaryview)) {
+               return;
+       }
+       summary_lock(summaryview);
+
        search_window.is_searching = TRUE;
        main_window_cursor_wait(summaryview->mainwin);
        summary_show_stop_button();
 
+       if (search_window.matcher_is_outdated && !summary_search_prereduce_msg_list()) {
+               goto exit;
+       }
+
        if (search_all) {
                summary_freeze(summaryview);
                summary_unselect_all(summaryview);
@@ -657,11 +760,9 @@ static void summary_search_execute(gboolean backward, gboolean search_all)
                }
        } else {
                if (backward) {
-                       node = gtkut_ctree_node_prev
-                               (ctree, summaryview->selected);
+                       node = gtkut_ctree_node_prev(ctree, summaryview->selected);
                } else {
-                       node = gtkut_ctree_node_next
-                               (ctree, summaryview->selected);
+                       node = gtkut_ctree_node_next(ctree, summaryview->selected);
                }
        }
 
@@ -691,11 +792,9 @@ static void summary_search_execute(gboolean backward, gboolean search_all)
                                         GTK_STOCK_NO, "+" GTK_STOCK_YES, NULL);
                        if (G_ALERTALTERNATE == val) {
                                if (backward) {
-                                       node = GTK_CMCTREE_NODE
-                                               (GTK_CMCLIST(ctree)->row_list_end);
+                                       node = GTK_CMCTREE_NODE(GTK_CMCLIST(ctree)->row_list_end);
                                } else {
-                                       node = GTK_CMCTREE_NODE
-                                               (GTK_CMCLIST(ctree)->row_list);
+                                       node = GTK_CMCTREE_NODE(GTK_CMCLIST(ctree)->row_list);
                                }
 
                                all_searched = TRUE;
@@ -707,44 +806,36 @@ static void summary_search_execute(gboolean backward, gboolean search_all)
                }
 
                msginfo = gtk_cmctree_node_get_row_data(ctree, node);
-               body_matched = FALSE;
 
-               matched = matcherlist_match(search_window.matcher_list, msginfo);
+               if (msginfo)
+                       matched = summary_search_verify_match(msginfo);
+               else
+                       matched = FALSE;
 
                if (matched) {
                        if (search_all) {
                                gtk_cmctree_select(ctree, node);
                        } else {
-                               if (messageview_is_visible
-                                       (summaryview->messageview)) {
+                               if (messageview_is_visible(summaryview->messageview)) {
                                        summary_unlock(summaryview);
-                                       summary_select_node
-                                               (summaryview, node, TRUE, TRUE);
+                                       summary_select_node(summaryview, node, TRUE, TRUE);
                                        summary_lock(summaryview);
-                                       if (body_matched) {
-                                               messageview_search_string
-                                                       (summaryview->messageview,
-                                                        body_str, case_sens);
-                                       }
                                } else {
-                                       summary_select_node
-                                               (summaryview, node, FALSE, TRUE);
+                                       summary_select_node(summaryview, node, FALSE, TRUE);
                                }
                                break;
                        }
                }
 
+               if (i % (search_window.is_fast ? 1000 : 100) == 0) {
+                       GTK_EVENTS_FLUSH();
+               }
+
                node = backward ? gtkut_ctree_node_prev(ctree, node)
                                : gtkut_ctree_node_next(ctree, node);
-               if (i % interval == 0)
-                       GTK_EVENTS_FLUSH();
        }
 
-       g_free(from_str);
-       g_free(to_str);
-       g_free(subject_str);
-       g_free(body_str);
-
+exit:
        search_window.is_searching = FALSE;
        summary_hide_stop_button();
        main_window_cursor_normal(summaryview->mainwin);
@@ -769,6 +860,7 @@ static void summary_search_clear(GtkButton *button, gpointer data)
        if (search_window.is_searching) {
                search_window.is_searching = FALSE;
        }
+       search_window.matcher_is_outdated = TRUE;
 }
 
 static void summary_search_prev_clicked(GtkButton *button, gpointer data)
@@ -798,6 +890,7 @@ static void adv_condition_btn_done(MatcherList * matchers)
        }
 
        str = matcherlist_to_string(matchers);
+       search_window.matcher_is_outdated = TRUE;
 
        if (str != NULL) {
                gtk_entry_set_text(GTK_ENTRY(gtk_bin_get_child(GTK_BIN((search_window.adv_condition_entry)))), str);
@@ -819,7 +912,11 @@ static void adv_condition_btn_clicked(GtkButton *button, gpointer data)
 
        /* re-use the current search value if it's a condition expression,
           otherwise ignore it silently */
+#if !GTK_CHECK_VERSION(2, 24, 0)
        cond_str = gtk_combo_box_get_active_text(GTK_COMBO_BOX(search_window.adv_condition_entry));
+#else
+       cond_str = gtk_combo_box_text_get_active_text(GTK_COMBO_BOX_TEXT(search_window.adv_condition_entry));
+#endif
        if (cond_str && *cond_str != '\0') {
                matchers = matcher_parser_get_cond((gchar*)cond_str, NULL);
        }
@@ -835,30 +932,35 @@ static void from_changed(void)
 {
        if (!search_window.from_entry_has_focus)
                gtk_widget_grab_focus(search_window.from_entry);
+       search_window.matcher_is_outdated = TRUE;
 }
 
 static void to_changed(void)
 {
        if (!search_window.to_entry_has_focus)
                gtk_widget_grab_focus(search_window.to_entry);
+       search_window.matcher_is_outdated = TRUE;
 }
 
 static void subject_changed(void)
 {
        if (!search_window.subject_entry_has_focus)
                gtk_widget_grab_focus(search_window.subject_entry);
+       search_window.matcher_is_outdated = TRUE;
 }
 
 static void body_changed(void)
 {
        if (!search_window.body_entry_has_focus)
                gtk_widget_grab_focus(search_window.body_entry);
+       search_window.matcher_is_outdated = TRUE;
 }
 
 static void adv_condition_changed(void)
 {
        if (!search_window.adv_condition_entry_has_focus)
                gtk_widget_grab_focus(search_window.adv_condition_entry);
+       search_window.matcher_is_outdated = TRUE;
 }
 
 static gboolean from_entry_focus_evt_in(GtkWidget *widget, GdkEventFocus *event,
@@ -930,11 +1032,11 @@ static gboolean adv_condition_entry_focus_evt_out(GtkWidget *widget, GdkEventFoc
        search_window.adv_condition_entry_has_focus = FALSE;
        return FALSE;
 }
-#ifndef MAEMO
+
 static gboolean key_pressed(GtkWidget *widget, GdkEventKey *event,
                            gpointer data)
 {
-       if (event && (event->keyval == GDK_Escape)) {
+       if (event && (event->keyval == GDK_KEY_Escape)) {
                /* ESC key will:
                        - stop a running search
                        - close the search window if no search is running
@@ -946,13 +1048,13 @@ static gboolean key_pressed(GtkWidget *widget, GdkEventKey *event,
                }
        }
 
-       if (event && (event->keyval == GDK_Return || event->keyval == GDK_KP_Enter)) {
+       if (event && (event->keyval == GDK_KEY_Return || event->keyval == GDK_KEY_KP_Enter)) {
                if (!search_window.is_searching) {
                        summary_search_execute(FALSE, FALSE);
                }
        }
 
-       if (event && (event->keyval == GDK_Down || event->keyval == GDK_Up)) {
+       if (event && (event->keyval == GDK_KEY_Down || event->keyval == GDK_KEY_Up)) {
                if (search_window.from_entry_has_focus) {
                        combobox_set_value_from_arrow_key(
                                        GTK_COMBO_BOX(search_window.from_entry),
@@ -987,4 +1089,3 @@ static gboolean key_pressed(GtkWidget *widget, GdkEventKey *event,
 
        return FALSE;
 }
-#endif