Add the actual NOTMUCH_MESSAGE_FLAG_EXCLUDED flag. --- lib/notmuch-private.h | 1 + lib/notmuch.h | 3 ++- lib/query.cc | 11 +++++++---- lib/thread.cc | 14 ++++++++++---- 4 files changed, 20 insertions(+), 9 deletions(-)
diff --git a/lib/notmuch-private.h b/lib/notmuch-private.h index e791bb0..cb3eca6 100644 --- a/lib/notmuch-private.h +++ b/lib/notmuch-private.h @@ -216,6 +216,7 @@ _notmuch_thread_create (void *ctx, notmuch_database_t *notmuch, unsigned int seed_doc_id, notmuch_doc_id_set_t *match_set, + notmuch_doc_id_set_t *excluded_doc_ids, notmuch_sort_t sort); /* message.cc */ diff --git a/lib/notmuch.h b/lib/notmuch.h index 7929fe7..cf0d45d 100644 --- a/lib/notmuch.h +++ b/lib/notmuch.h @@ -895,7 +895,8 @@ notmuch_message_get_filenames (notmuch_message_t *message); /* Message flags */ typedef enum _notmuch_message_flag { - NOTMUCH_MESSAGE_FLAG_MATCH + NOTMUCH_MESSAGE_FLAG_MATCH, + NOTMUCH_MESSAGE_FLAG_EXCLUDED } notmuch_message_flag_t; /* Get a value of a flag for the email corresponding to 'message'. */ diff --git a/lib/query.cc b/lib/query.cc index 92fa834..69e32bd 100644 --- a/lib/query.cc +++ b/lib/query.cc @@ -55,6 +55,7 @@ struct visible _notmuch_threads { /* The set of matched docid's that have not been assigned to a * thread. Initially, this contains every docid in doc_ids. */ notmuch_doc_id_set_t match_set; + notmuch_doc_id_set_t *excluded_doc_ids; }; static notmuch_bool_t @@ -302,6 +303,9 @@ _notmuch_mset_messages_get (notmuch_messages_t *messages) INTERNAL_ERROR ("a messages iterator contains a non-existent document ID.\n"); } + if (_notmuch_doc_id_set_contains (messages->excluded_doc_ids, doc_id)) + notmuch_message_set_flag (message, NOTMUCH_MESSAGE_FLAG_EXCLUDED, TRUE); + return message; } @@ -314,10 +318,6 @@ _notmuch_mset_messages_move_to_next (notmuch_messages_t *messages) mset_messages->iterator++; - while ((mset_messages->iterator != mset_messages->iterator_end) && - (_notmuch_doc_id_set_contains (messages->excluded_doc_ids, - *mset_messages->iterator))) - mset_messages->iterator++; } static notmuch_bool_t @@ -403,6 +403,8 @@ notmuch_query_search_threads (notmuch_query_t *query) notmuch_messages_move_to_next (messages); } threads->doc_id_pos = 0; + /* the excluded messages are in query context so this should be ok */ + threads->excluded_doc_ids = messages->excluded_doc_ids; talloc_free (messages); @@ -452,6 +454,7 @@ notmuch_threads_get (notmuch_threads_t *threads) threads->query->notmuch, doc_id, &threads->match_set, + threads->excluded_doc_ids, threads->query->sort); } diff --git a/lib/thread.cc b/lib/thread.cc index 0435ee6..6ea2a44 100644 --- a/lib/thread.cc +++ b/lib/thread.cc @@ -302,7 +302,8 @@ _thread_set_subject_from_message (notmuch_thread_t *thread, static void _thread_add_matched_message (notmuch_thread_t *thread, notmuch_message_t *message, - notmuch_sort_t sort) + notmuch_sort_t sort, + notmuch_bool_t excluded) { time_t date; notmuch_message_t *hashed_message; @@ -321,7 +322,8 @@ _thread_add_matched_message (notmuch_thread_t *thread, _thread_set_subject_from_message (thread, message); } - thread->matched_messages++; + if (!excluded) + thread->matched_messages++; if (g_hash_table_lookup_extended (thread->message_hash, notmuch_message_get_message_id (message), NULL, @@ -392,6 +394,7 @@ _notmuch_thread_create (void *ctx, notmuch_database_t *notmuch, unsigned int seed_doc_id, notmuch_doc_id_set_t *match_set, + notmuch_doc_id_set_t *excluded_doc_ids, notmuch_sort_t sort) { notmuch_thread_t *thread; @@ -456,7 +459,9 @@ _notmuch_thread_create (void *ctx, * oldest or newest subject is desired. */ notmuch_query_set_sort (thread_id_query, NOTMUCH_SORT_OLDEST_FIRST); - for (messages = notmuch_query_search_messages (thread_id_query); + messages = notmuch_query_search_messages (thread_id_query); + messages->excluded_doc_ids = excluded_doc_ids; + for (; notmuch_messages_valid (messages); notmuch_messages_move_to_next (messages)) { @@ -471,7 +476,8 @@ _notmuch_thread_create (void *ctx, if ( _notmuch_doc_id_set_contains (match_set, doc_id)) { _notmuch_doc_id_set_remove (match_set, doc_id); - _thread_add_matched_message (thread, message, sort); + _thread_add_matched_message (thread, message, sort, + _notmuch_doc_id_set_contains (excluded_doc_ids, doc_id)); } _notmuch_message_close (message); -- 1.7.2.3