From: Jani Nikula <jani@nikula.org>
To: notmuch@notmuchmail.org
Subject: [PATCH v4 2/6] cli: add options --first and --maxitems to notmuch search
Date: Sun, 6 Nov 2011 23:47:11 +0200 [thread overview]
Message-ID: <45e0f0f47250b20b4c3372910fea4d2fd5144e20.1320615322.git.jani@nikula.org> (raw)
In-Reply-To: <cover.1320615322.git.jani@nikula.org>
In-Reply-To: <cover.1320615322.git.jani@nikula.org>
Add options --first=[-]N and --maxitems=M to notmuch search to determine
the first result and maximum number of results to display.
Option --maxitems=M limits the maximum number of results to display to M.
Option --first=[-]N skips the first N results; with the leading '-' skip
until the Nth result from the end (showing a total of N results if within
bounds of the total number of results and not limited with --maxitems).
Note that --first with a negative N for thread or summary output requires
counting the number of matching threads in advance.
Signed-off-by: Jani Nikula <jani@nikula.org>
---
NEWS | 5 ++++
notmuch-search.c | 70 ++++++++++++++++++++++++++++++++++++++++++++---------
notmuch.1 | 21 ++++++++++++++-
3 files changed, 82 insertions(+), 14 deletions(-)
diff --git a/NEWS b/NEWS
index b44b11e..bfdba7b 100644
--- a/NEWS
+++ b/NEWS
@@ -23,6 +23,11 @@ Add search terms to "notmuch dump"
search/show/tag. The output file argument of dump is deprecated in
favour of using stdout.
+Add "notmuch search" --first and --maxitems options
+
+ The search command now takes options --first=[-]N and --maxitems=N to limit
+ the number of results shown.
+
Notmuch 0.9 (2011-10-01)
========================
diff --git a/notmuch-search.c b/notmuch-search.c
index 6f04d9a..c62a594 100644
--- a/notmuch-search.c
+++ b/notmuch-search.c
@@ -194,13 +194,22 @@ static int
do_search_threads (const search_format_t *format,
notmuch_query_t *query,
notmuch_sort_t sort,
- output_t output)
+ output_t output,
+ int first,
+ int maxitems)
{
notmuch_thread_t *thread;
notmuch_threads_t *threads;
notmuch_tags_t *tags;
time_t date;
int first_thread = 1;
+ int i;
+
+ if (first < 0) {
+ first += notmuch_query_count_threads (query);
+ if (first < 0)
+ first = 0;
+ }
threads = notmuch_query_search_threads (query);
if (threads == NULL)
@@ -208,17 +217,23 @@ do_search_threads (const search_format_t *format,
fputs (format->results_start, stdout);
- for (;
- notmuch_threads_valid (threads);
- notmuch_threads_move_to_next (threads))
+ for (i = 0;
+ notmuch_threads_valid (threads) &&
+ (maxitems < 0 || i < first + maxitems);
+ notmuch_threads_move_to_next (threads), i++)
{
int first_tag = 1;
+ thread = notmuch_threads_get (threads);
+
+ if (i < first) {
+ notmuch_thread_destroy (thread);
+ continue;
+ }
+
if (! first_thread)
fputs (format->item_sep, stdout);
- thread = notmuch_threads_get (threads);
-
if (output == OUTPUT_THREADS) {
format->item_id (thread, "thread:",
notmuch_thread_get_thread_id (thread));
@@ -271,12 +286,21 @@ do_search_threads (const search_format_t *format,
static int
do_search_messages (const search_format_t *format,
notmuch_query_t *query,
- output_t output)
+ output_t output,
+ int first,
+ int maxitems)
{
notmuch_message_t *message;
notmuch_messages_t *messages;
notmuch_filenames_t *filenames;
int first_message = 1;
+ int i;
+
+ if (first < 0) {
+ first += notmuch_query_count_messages (query);
+ if (first < 0)
+ first = 0;
+ }
messages = notmuch_query_search_messages (query);
if (messages == NULL)
@@ -284,10 +308,14 @@ do_search_messages (const search_format_t *format,
fputs (format->results_start, stdout);
- for (;
- notmuch_messages_valid (messages);
- notmuch_messages_move_to_next (messages))
+ for (i = 0;
+ notmuch_messages_valid (messages) &&
+ (maxitems < 0 || i < first + maxitems);
+ notmuch_messages_move_to_next (messages), i++)
{
+ if (i < first)
+ continue;
+
message = notmuch_messages_get (messages);
if (output == OUTPUT_FILES) {
@@ -394,6 +422,8 @@ notmuch_search_command (void *ctx, int argc, char *argv[])
const search_format_t *format = &format_text;
int i, ret;
output_t output = OUTPUT_SUMMARY;
+ int maxitems = -1; /* unlimited */
+ int first = 0;
argc--; argv++; /* skip subcommand argument */
@@ -412,6 +442,22 @@ notmuch_search_command (void *ctx, int argc, char *argv[])
fprintf (stderr, "Invalid value for --sort: %s\n", opt);
return 1;
}
+ } else if (STRNCMP_LITERAL (argv[i], "--first=") == 0) {
+ char *p;
+ opt = argv[i] + sizeof ("--first=") - 1;
+ first = strtol (opt, &p, 10);
+ if (*opt == '\0' || p == opt || *p != '\0') {
+ fprintf (stderr, "Invalid value for --first: %s\n", opt);
+ return 1;
+ }
+ } else if (STRNCMP_LITERAL (argv[i], "--maxitems=") == 0) {
+ char *p;
+ opt = argv[i] + sizeof ("--maxitems=") - 1;
+ maxitems = strtoul (opt, &p, 10);
+ if (*opt == '\0' || p == opt || *p != '\0') {
+ fprintf (stderr, "Invalid value for --maxitems: %s\n", opt);
+ return 1;
+ }
} else if (STRNCMP_LITERAL (argv[i], "--format=") == 0) {
opt = argv[i] + sizeof ("--format=") - 1;
if (strcmp (opt, "text") == 0) {
@@ -478,11 +524,11 @@ notmuch_search_command (void *ctx, int argc, char *argv[])
default:
case OUTPUT_SUMMARY:
case OUTPUT_THREADS:
- ret = do_search_threads (format, query, sort, output);
+ ret = do_search_threads (format, query, sort, output, first, maxitems);
break;
case OUTPUT_MESSAGES:
case OUTPUT_FILES:
- ret = do_search_messages (format, query, output);
+ ret = do_search_messages (format, query, output, first, maxitems);
break;
case OUTPUT_TAGS:
ret = do_search_tags (notmuch, format, query);
diff --git a/notmuch.1 b/notmuch.1
index bba479e..c97334c 100644
--- a/notmuch.1
+++ b/notmuch.1
@@ -214,11 +214,28 @@ when sorting by
.B newest\-first
the threads will be sorted by the newest message in each thread.
-.RE
-.RS 4
By default, results will be displayed in reverse chronological order,
(that is, the newest results will be displayed first).
+.RE
+
+.RS 4
+.TP 4
+.BR \-\-first=[\-]N
+
+This option can be used to skip the display of first N results. With the
+leading '\-' skip until the Nth result from the end, showing a total of N
+results if not otherwise bounded by total number of matching results or
+\-\-maxitems.
+.RE
+
+.RS 4
+.TP 4
+.BR \-\-maxitems=N
+This option can be used to limit the number of results to display to N.
+.RE
+
+.RS 4
See the
.B "SEARCH SYNTAX"
section below for details of the supported syntax for <search-terms>.
--
1.7.5.4
next prev parent reply other threads:[~2011-11-06 21:47 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-06 21:47 [PATCH v4 0/6] lib/cli: limit number of messages in search results Jani Nikula
2011-11-06 21:47 ` [PATCH v4 1/6] lib: add function to get the number of threads matching a search Jani Nikula
2011-11-06 21:47 ` Jani Nikula [this message]
2011-11-13 16:52 ` [PATCH v4 2/6] cli: add options --first and --maxitems to notmuch search Austin Clements
2011-11-06 21:47 ` [PATCH v4 3/6] cli: drop unused code from notmuch count Jani Nikula
2011-11-06 21:47 ` [PATCH v4 4/6] cli: add support for --output parameter in " Jani Nikula
2011-11-13 17:01 ` Austin Clements
2011-11-06 21:47 ` [PATCH v4 5/6] test: add tests for " Jani Nikula
2011-11-06 21:47 ` [PATCH v4 6/6] test: add tests for notmuch search --first and --maxitems Jani Nikula
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://notmuchmail.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=45e0f0f47250b20b4c3372910fea4d2fd5144e20.1320615322.git.jani@nikula.org \
--to=jani@nikula.org \
--cc=notmuch@notmuchmail.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://yhetil.org/notmuch.git/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).