From: Mark Walters Date: Thu, 2 Feb 2012 17:43:34 +0000 (+0000) Subject: [PATCH v4 06/11] lib: Add the exclude flag to notmuch_query_search_threads X-Git-Url: http://git.tremily.us/gitweb.cgi?a=commitdiff_plain;h=e102cdc43c8409e38ae0799ff07dc0d990586f6c;p=notmuch-archives.git [PATCH v4 06/11] lib: Add the exclude flag to notmuch_query_search_threads --- diff --git a/f1/a8d9a010bf179193a90fd5bf429fcd8b92cac7 b/f1/a8d9a010bf179193a90fd5bf429fcd8b92cac7 new file mode 100644 index 000000000..a90dded4c --- /dev/null +++ b/f1/a8d9a010bf179193a90fd5bf429fcd8b92cac7 @@ -0,0 +1,178 @@ +Return-Path: +X-Original-To: notmuch@notmuchmail.org +Delivered-To: notmuch@notmuchmail.org +Received: from localhost (localhost [127.0.0.1]) + by olra.theworths.org (Postfix) with ESMTP id 3CCB1429E36 + for ; Thu, 2 Feb 2012 09:43:09 -0800 (PST) +X-Virus-Scanned: Debian amavisd-new at olra.theworths.org +X-Spam-Flag: NO +X-Spam-Score: 0.201 +X-Spam-Level: +X-Spam-Status: No, score=0.201 tagged_above=-999 required=5 + tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, + FREEMAIL_ENVFROM_END_DIGIT=1, FREEMAIL_FROM=0.001, + RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled +Received: from olra.theworths.org ([127.0.0.1]) + by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024) + with ESMTP id QhXMmpuqQ-19 for ; + Thu, 2 Feb 2012 09:43:08 -0800 (PST) +Received: from mail-we0-f181.google.com (mail-we0-f181.google.com + [74.125.82.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) + (No client certificate requested) + by olra.theworths.org (Postfix) with ESMTPS id 6F821431FBD + for ; Thu, 2 Feb 2012 09:43:07 -0800 (PST) +Received: by mail-we0-f181.google.com with SMTP id b10so2324200wer.26 + for ; Thu, 02 Feb 2012 09:43:07 -0800 (PST) +DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; + h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references; + bh=eAUX78p88IX+cG4hVJxkEo2EBDwpTAL60rWPZ7/6HBQ=; + b=mmTSMU4Bp0T0IO812Ul4GzEwjPS2z+2PmMTXUQk/0pSRDljR/xDIwqTU//ufqXFMgv + na6ijTX2froXmJ9ZDgW9rhScSxzflsl1CJACpiG2UPamwse4AaMAf1tprENKURZG8d4b + DHK+RYu1vkTPhmKI+4LVLu0ek0/aVJvwaxsCw= +Received: by 10.216.138.102 with SMTP id z80mr4277872wei.38.1328204587111; + Thu, 02 Feb 2012 09:43:07 -0800 (PST) +Received: from localhost (94-192-233-223.zone6.bethere.co.uk. + [94.192.233.223]) by mx.google.com with ESMTPS id + hc10sm1119723wib.8.2012.02.02.09.43.05 (version=TLSv1/SSLv3 cipher=OTHER); + Thu, 02 Feb 2012 09:43:06 -0800 (PST) +From: Mark Walters +To: notmuch@notmuchmail.org, + amdragon@MIT.EDU +Subject: [PATCH v4 06/11] lib: Add the exclude flag to + notmuch_query_search_threads +Date: Thu, 2 Feb 2012 17:43:34 +0000 +Message-Id: <1328204619-25046-6-git-send-email-markwalters1009@gmail.com> +X-Mailer: git-send-email 1.7.2.3 +In-Reply-To: <874nv9rv79.fsf@qmul.ac.uk> +References: <874nv9rv79.fsf@qmul.ac.uk> +X-BeenThere: notmuch@notmuchmail.org +X-Mailman-Version: 2.1.13 +Precedence: list +List-Id: "Use and development of the notmuch mail system." + +List-Unsubscribe: , + +List-Archive: +List-Post: +List-Help: +List-Subscribe: , + +X-List-Received-Date: Thu, 02 Feb 2012 17:43:10 -0000 + +Add the NOTMUCH_MESSAGE_FLAG_EXCLUDED flag to +notmuch_query_search_threads. Implemented by inspecting the tags +directly in _notmuch_thread_create/_thread_add_message rather than as +a Xapian query for speed reasons. +--- + lib/notmuch-private.h | 7 +++++-- + lib/query.cc | 1 + + lib/thread.cc | 18 +++++++++++++++--- + 3 files changed, 21 insertions(+), 5 deletions(-) + +diff --git a/lib/notmuch-private.h b/lib/notmuch-private.h +index e791bb0..ea836f7 100644 +--- a/lib/notmuch-private.h ++++ b/lib/notmuch-private.h +@@ -148,6 +148,8 @@ typedef enum _notmuch_private_status { + + typedef struct _notmuch_doc_id_set notmuch_doc_id_set_t; + ++typedef struct _notmuch_string_list notmuch_string_list_t; ++ + /* database.cc */ + + /* Lookup a prefix value by name. +@@ -216,6 +218,7 @@ _notmuch_thread_create (void *ctx, + notmuch_database_t *notmuch, + unsigned int seed_doc_id, + notmuch_doc_id_set_t *match_set, ++ notmuch_string_list_t *excluded_terms, + notmuch_sort_t sort); + + /* message.cc */ +@@ -459,11 +462,11 @@ typedef struct _notmuch_string_node { + struct _notmuch_string_node *next; + } notmuch_string_node_t; + +-typedef struct visible _notmuch_string_list { ++struct visible _notmuch_string_list { + int length; + notmuch_string_node_t *head; + notmuch_string_node_t **tail; +-} notmuch_string_list_t; ++}; + + notmuch_string_list_t * + _notmuch_string_list_create (const void *ctx); +diff --git a/lib/query.cc b/lib/query.cc +index 90a71a1..e1c3977 100644 +--- a/lib/query.cc ++++ b/lib/query.cc +@@ -472,6 +472,7 @@ notmuch_threads_get (notmuch_threads_t *threads) + threads->query->notmuch, + doc_id, + &threads->match_set, ++ threads->query->exclude_terms, + threads->query->sort); + } + +diff --git a/lib/thread.cc b/lib/thread.cc +index 0435ee6..e976d64 100644 +--- a/lib/thread.cc ++++ b/lib/thread.cc +@@ -214,7 +214,8 @@ _thread_cleanup_author (notmuch_thread_t *thread, + */ + static void + _thread_add_message (notmuch_thread_t *thread, +- notmuch_message_t *message) ++ notmuch_message_t *message, ++ notmuch_string_list_t *exclude_terms) + { + notmuch_tags_t *tags; + const char *tag; +@@ -262,6 +263,15 @@ _thread_add_message (notmuch_thread_t *thread, + notmuch_tags_move_to_next (tags)) + { + tag = notmuch_tags_get (tags); ++ /* Mark excluded messages. */ ++ for (notmuch_string_node_t *term = exclude_terms->head; term; ++ term = term->next) { ++ /* We ignore initial 'K'. */ ++ if (strcmp(tag, (term->string + 1)) == 0) { ++ notmuch_message_set_flag (message, NOTMUCH_MESSAGE_FLAG_EXCLUDED, TRUE); ++ break; ++ } ++ } + g_hash_table_insert (thread->tags, xstrdup (tag), NULL); + } + } +@@ -321,7 +331,8 @@ _thread_add_matched_message (notmuch_thread_t *thread, + _thread_set_subject_from_message (thread, message); + } + +- thread->matched_messages++; ++ if (!notmuch_message_get_flag (message, NOTMUCH_MESSAGE_FLAG_EXCLUDED)) ++ thread->matched_messages++; + + if (g_hash_table_lookup_extended (thread->message_hash, + notmuch_message_get_message_id (message), NULL, +@@ -392,6 +403,7 @@ _notmuch_thread_create (void *ctx, + notmuch_database_t *notmuch, + unsigned int seed_doc_id, + notmuch_doc_id_set_t *match_set, ++ notmuch_string_list_t *exclude_terms, + notmuch_sort_t sort) + { + notmuch_thread_t *thread; +@@ -467,7 +479,7 @@ _notmuch_thread_create (void *ctx, + if (doc_id == seed_doc_id) + message = seed_message; + +- _thread_add_message (thread, message); ++ _thread_add_message (thread, message, exclude_terms); + + if ( _notmuch_doc_id_set_contains (match_set, doc_id)) { + _notmuch_doc_id_set_remove (match_set, doc_id); +-- +1.7.2.3 +