unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
From: Jani Nikula <jani@nikula.org>
To: notmuch@notmuchmail.org
Subject: [PATCH v3 1/8] cli: add --duplicate=N option to notmuch search
Date: Sat, 17 Aug 2013 15:11:26 +0300	[thread overview]
Message-ID: <bbe225e1cc50211258f85a8533a83093337bde3f.1376741447.git.jani@nikula.org> (raw)
In-Reply-To: <cover.1376741447.git.jani@nikula.org>
In-Reply-To: <cover.1376741447.git.jani@nikula.org>

Effective with --output=files, output the Nth filename associated with
each message matching the query (N is 1-based). If N is greater than
the number of files associated with the message, don't print anything.

---

v3: fix commit message, no other changes
---
 notmuch-search.c |   18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/notmuch-search.c b/notmuch-search.c
index a96f07d..d9d39ec 100644
--- a/notmuch-search.c
+++ b/notmuch-search.c
@@ -177,7 +177,8 @@ do_search_messages (sprinter_t *format,
 		    notmuch_query_t *query,
 		    output_t output,
 		    int offset,
-		    int limit)
+		    int limit,
+		    int dupe)
 {
     notmuch_message_t *message;
     notmuch_messages_t *messages;
@@ -206,14 +207,17 @@ do_search_messages (sprinter_t *format,
 	message = notmuch_messages_get (messages);
 
 	if (output == OUTPUT_FILES) {
+	    int j;
 	    filenames = notmuch_message_get_filenames (message);
 
-	    for (;
+	    for (j = 1;
 		 notmuch_filenames_valid (filenames);
-		 notmuch_filenames_move_to_next (filenames))
+		 notmuch_filenames_move_to_next (filenames), j++)
 	    {
-		format->string (format, notmuch_filenames_get (filenames));
-		format->separator (format);
+		if (dupe < 0 || dupe == j) {
+		    format->string (format, notmuch_filenames_get (filenames));
+		    format->separator (format);
+		}
 	    }
 	    
 	    notmuch_filenames_destroy( filenames );
@@ -296,6 +300,7 @@ notmuch_search_command (notmuch_config_t *config, int argc, char *argv[])
     int offset = 0;
     int limit = -1; /* unlimited */
     notmuch_exclude_t exclude = NOTMUCH_EXCLUDE_TRUE;
+    int dupe = -1;
     unsigned int i;
 
     enum {
@@ -332,6 +337,7 @@ notmuch_search_command (notmuch_config_t *config, int argc, char *argv[])
                                   { 0, 0 } } },
 	{ NOTMUCH_OPT_INT, &offset, "offset", 'O', 0 },
 	{ NOTMUCH_OPT_INT, &limit, "limit", 'L', 0  },
+	{ NOTMUCH_OPT_INT, &dupe, "duplicate", 'D', 0  },
 	{ 0, 0, 0, 0, 0 }
     };
 
@@ -414,7 +420,7 @@ notmuch_search_command (notmuch_config_t *config, int argc, char *argv[])
 	break;
     case OUTPUT_MESSAGES:
     case OUTPUT_FILES:
-	ret = do_search_messages (format, query, output, offset, limit);
+	ret = do_search_messages (format, query, output, offset, limit, dupe);
 	break;
     case OUTPUT_TAGS:
 	ret = do_search_tags (notmuch, format, query);
-- 
1.7.10.4

  reply	other threads:[~2013-08-17 12:11 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-17 12:11 [PATCH v3 0/8] cli: better tooling support for duplicate messages Jani Nikula
2013-08-17 12:11 ` Jani Nikula [this message]
2013-08-17 12:11 ` [PATCH v3 2/8] test: test notmuch search --duplicate=N Jani Nikula
2013-08-17 12:11 ` [PATCH v3 3/8] man: document " Jani Nikula
2013-08-17 12:11 ` [PATCH v3 4/8] cli: add --output=files option to notmuch count Jani Nikula
2013-08-17 12:11 ` [PATCH v3 5/8] test: test notmuch count --output=files Jani Nikula
2013-08-17 12:11 ` [PATCH v3 6/8] man: document " Jani Nikula
2013-08-17 12:11 ` [PATCH v3 7/8] NEWS: cli options for improved duplicate message handling Jani Nikula
2013-08-17 13:36   ` Tomi Ollila
     [not found]     ` <cover.1376830237.git.jani@nikula.org>
2013-08-18 12:52       ` [PATCH v4 " Jani Nikula
2013-08-17 12:11 ` [PATCH v3 8/8] man: clarify folder: prefix regarding duplicate files Jani Nikula
2013-08-21 19:53 ` [PATCH v3 0/8] cli: better tooling support for duplicate messages Tomi Ollila
2013-08-24  7:50 ` Mark Walters
2013-08-24  9:51 ` David Bremner

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=bbe225e1cc50211258f85a8533a83093337bde3f.1376741447.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).