notmuch show: limit display to only matching messages

This patch changes the default behaviour of notmuch show to display only
messages that match the search expression.  However, --entire-thread
option is provided to display all messages in threads that matched the
search expression.

It is deemed that will be more useful for human users on the command line.
Scripts can be modified to include the --entire-thread option so that they
can display all messages once more.

Example:

$ notmuch search subject:git AND thread:23d99d0f364f93e90e15df8b42eddb5b
thread:23d99d0f364f93e90e15df8b42eddb5b      July 31 [4/12] Johan Herland; [RFCv2 00/12] Foreign VCS helper program for CVS repositories (inbox unread)

Note that in this thread 4 out of 12 messages matched.  The default show
behaviour is to show only those messages that match:

$ notmuch show subject:git AND thread:23d99d0f364f93e90e15df8b42eddb5b | grep 'message{' | wc -l
4

With the --entire-thread option the output will contain all dozen
messages:

$ notmuch show --entire-thread subject:git AND thread:23d99d0f364f93e90e15df8b42eddb5b | grep 'message{' | wc -l
12

Signed-off-by: Bart Trojanowski <bart@jukie.net>
This commit is contained in:
Bart Trojanowski 2009-11-27 21:49:39 -05:00 committed by Carl Worth
parent ea2d9a2cbf
commit d50c67d53b
2 changed files with 46 additions and 12 deletions

View file

@ -212,6 +212,24 @@ notmuch_show_command (void *ctx, unused (int argc), unused (char *argv[]))
notmuch_thread_t *thread; notmuch_thread_t *thread;
notmuch_messages_t *messages; notmuch_messages_t *messages;
char *query_string; char *query_string;
int entire_thread = 0;
int i;
for (i = 0; i < argc && argv[i][0] == '-'; i++) {
if (strcmp (argv[i], "--") == 0) {
i++;
break;
}
if (strcmp(argv[i], "--entire-thread") == 0) {
entire_thread = 1;
} else {
fprintf (stderr, "Unrecognized option: %s\n", argv[i]);
return 1;
}
}
argc -= i;
argv += i;
config = notmuch_config_open (ctx, NULL, NULL); config = notmuch_config_open (ctx, NULL, NULL);
if (config == NULL) if (config == NULL)
@ -239,21 +257,29 @@ notmuch_show_command (void *ctx, unused (int argc), unused (char *argv[]))
return 1; return 1;
} }
for (threads = notmuch_query_search_threads (query); if (!entire_thread) {
notmuch_threads_has_more (threads); messages = notmuch_query_search_messages (query);
notmuch_threads_advance (threads))
{
thread = notmuch_threads_get (threads);
messages = notmuch_thread_get_toplevel_messages (thread);
if (messages == NULL) if (messages == NULL)
INTERNAL_ERROR ("Thread %s has no toplevel messages.\n", INTERNAL_ERROR ("No messages.\n");
notmuch_thread_get_thread_id (thread));
show_messages (ctx, messages, 0); show_messages (ctx, messages, 0);
notmuch_thread_destroy (thread); } else {
for (threads = notmuch_query_search_threads (query);
notmuch_threads_has_more (threads);
notmuch_threads_advance (threads))
{
thread = notmuch_threads_get (threads);
messages = notmuch_thread_get_toplevel_messages (thread);
if (messages == NULL)
INTERNAL_ERROR ("Thread %s has no toplevel messages.\n",
notmuch_thread_get_thread_id (thread));
show_messages (ctx, messages, 0);
notmuch_thread_destroy (thread);
}
} }
notmuch_query_destroy (query); notmuch_query_destroy (query);

View file

@ -177,6 +177,14 @@ command_t commands[] = {
"\t\t(all replies to a particular message appear immediately\n" "\t\t(all replies to a particular message appear immediately\n"
"\t\tafter that message in date order).\n" "\t\tafter that message in date order).\n"
"\n" "\n"
"\t\tSupported options for show include:\n"
"\n"
"\t\t--entire-thread\n"
"\n"
"\t\t\tBy default only messages strictly matching the\n"
"\t\t\tsearch expression will be displayed. With this\n"
"\t\t\toption all messages in matching threads are shown.\n"
"\n"
"\t\tThe output format is plain-text, with all text-content\n" "\t\tThe output format is plain-text, with all text-content\n"
"\t\tMIME parts decoded. Various components in the output,\n" "\t\tMIME parts decoded. Various components in the output,\n"
"\t\t('message', 'header', 'body', 'attachment', and MIME 'part')\n" "\t\t('message', 'header', 'body', 'attachment', and MIME 'part')\n"