[PATCH 7/8] pick: Use interactive specifications for tag changes

diff --git a/22/7bb8b81eb110b756d324f850e57811ef8ba0d1 b/22/7bb8b81eb110b756d324f850e57811ef8ba0d1
new file mode 100644 (file)
index 0000000..023093e
--- /dev/null
+++ b/22/7bb8b81eb110b756d324f850e57811ef8ba0d1
@@ -0,0 +1,147 @@
+Return-Path: <amdragon@mit.edu>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id B5F3C429E2F\r
+       for <notmuch@notmuchmail.org>; Tue, 22 Oct 2013 12:51:42 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.7\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
+       tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id ZprCSvwsIaYP for <notmuch@notmuchmail.org>;\r
+       Tue, 22 Oct 2013 12:51:37 -0700 (PDT)\r
+Received: from dmz-mailsec-scanner-4.mit.edu (dmz-mailsec-scanner-4.mit.edu\r
+       [18.9.25.15])\r
+       by olra.theworths.org (Postfix) with ESMTP id 7264F429E30\r
+       for <notmuch@notmuchmail.org>; Tue, 22 Oct 2013 12:51:15 -0700 (PDT)\r
+X-AuditID: 1209190f-b7fa08e0000009c6-28-5266d732aae7\r
+Received: from mailhub-auth-1.mit.edu ( [18.9.21.35])\r
+       by dmz-mailsec-scanner-4.mit.edu (Symantec Messaging Gateway) with SMTP\r
+       id C3.16.02502.237D6625; Tue, 22 Oct 2013 15:51:15 -0400 (EDT)\r
+Received: from outgoing.mit.edu (outgoing-auth-1.mit.edu [18.9.28.11])\r
+       by mailhub-auth-1.mit.edu (8.13.8/8.9.2) with ESMTP id r9MJp3lR015066; \r
+       Tue, 22 Oct 2013 15:51:04 -0400\r
+Received: from drake.dyndns.org (31-33-57.wireless.csail.mit.edu\r
+       [128.31.33.57]) (authenticated bits=0)\r
+       (User authenticated as amdragon@ATHENA.MIT.EDU)\r
+       by outgoing.mit.edu (8.13.8/8.12.4) with ESMTP id r9MJp11l005202\r
+       (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
+       Tue, 22 Oct 2013 15:51:03 -0400\r
+Received: from amthrax by drake.dyndns.org with local (Exim 4.77)\r
+       (envelope-from <amdragon@mit.edu>)\r
+       id 1VYhyn-0006ys-5X; Tue, 22 Oct 2013 15:51:01 -0400\r
+From: Austin Clements <amdragon@MIT.EDU>\r
+To: notmuch@notmuchmail.org\r
+Subject: [PATCH 7/8] pick: Use interactive specifications for tag changes\r
+Date: Tue, 22 Oct 2013 15:50:56 -0400\r
+Message-Id: <1382471457-26056-8-git-send-email-amdragon@mit.edu>\r
+X-Mailer: git-send-email 1.8.4.rc3\r
+In-Reply-To: <1382471457-26056-1-git-send-email-amdragon@mit.edu>\r
+References: <1382471457-26056-1-git-send-email-amdragon@mit.edu>\r
+X-Brightmail-Tracker:\r
+ H4sIAAAAAAAAA+NgFtrEIsWRmVeSWpSXmKPExsUixCmqrGt8PS3I4NkNRovrN2cyOzB6PFt1\r
+       izmAMYrLJiU1J7MstUjfLoErY2PHAuaCS1IVX1buYm1gfCTWxcjJISFgIvH72Gk2CFtM4sK9\r
+       9UA2F4eQwD5GiXdfTjFDOBsZJX70vIJyjjFJXH3WAeXMZZSY8ukhM0g/m4CGxLb9yxlBbBEB\r
+       aYmdd2ezdjFycDALqEn86VIBCQsLeEq0/T7GAmKzCKhKHO48DFbOK+Ag8X3fUmaIM5QkFp7a\r
+       xgpicwo4Skzf948NZIwQUM2pOxwTGPkXMDKsYpRNya3SzU3MzClOTdYtTk7My0st0jXRy80s\r
+       0UtNKd3ECA4aSf4djN8OKh1iFOBgVOLhfdCSFiTEmlhWXJl7iFGSg0lJlDfiKlCILyk/pTIj\r
+       sTgjvqg0J7X4EKMEB7OSCG+zL1CONyWxsiq1KB8mJc3BoiTOe5PDPkhIID2xJDU7NbUgtQgm\r
+       K8PBoSTBG3gNqFGwKDU9tSItM6cEIc3EwQkynAdo+DGQxbzFBYm5xZnpEPlTjIpS4rwRIM0C\r
+       IImM0jy4XlhUv2IUB3pFmLcKpIoHmBDgul8BDWYCGjxlCdjgkkSElFQDY6fS1l97zhSLH6ma\r
+       ++J67b1rMV+7WNSmcDbafTt2WXu+xLSDvQkfrb0YUm/cPtuqNeGyuMpPv4cbE5LVRHmOpWuu\r
+       UokIZC1+mzX15cvQGTecVrvoH1zadbc8yTYqSLFY2MN2eeVFT7V9fWm/377hWWRq9riJUc/g\r
+       3ppvl4MYcnp613h761emKbEUZyQaajEXFScCAKNpzxXFAgAA\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Tue, 22 Oct 2013 19:51:43 -0000\r
+\r
+---\r
+ contrib/notmuch-pick/notmuch-pick.el | 36 +++++++++++++++++++++---------------\r
+ 1 file changed, 21 insertions(+), 15 deletions(-)\r
+\r
+diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el\r
+index 8f504ed..13682ea 100644\r
+--- a/contrib/notmuch-pick/notmuch-pick.el\r
++++ b/contrib/notmuch-pick/notmuch-pick.el\r
+@@ -37,7 +37,7 @@\r
+ (declare-function notmuch-show-strip-re "notmuch-show" (subject))\r
+ (declare-function notmuch-show-spaces-n "notmuch-show" (n))\r
+ (declare-function notmuch-read-query "notmuch" (prompt))\r
+-(declare-function notmuch-read-tag-changes "notmuch" (&optional initial-input &rest search-terms))\r
++(declare-function notmuch-read-tag-changes "notmuch" (current-tags &optional prompt initial-input))\r
+ (declare-function notmuch-update-tags "notmuch" (current-tags tag-changes))\r
+ (declare-function notmuch-hello-trim "notmuch-hello" (search))\r
+ (declare-function notmuch-search-find-thread-id "notmuch" ())\r
+@@ -372,21 +372,24 @@ Does NOT change the database."\r
+       (notmuch-pick-set-tags new-tags)\r
+       (notmuch-pick-refresh-result))))\r
+ \r
+-(defun notmuch-pick-tag (&optional tag-changes)\r
++(defun notmuch-pick-tag (tag-changes)\r
+   "Change tags for the current message"\r
+-  (interactive)\r
+-  (setq tag-changes (notmuch-tag (notmuch-pick-get-message-id) tag-changes))\r
++  (interactive\r
++   (list (notmuch-read-tag-changes (notmuch-pick-get-tags) "Tag message")))\r
++  (notmuch-tag (notmuch-pick-get-message-id) tag-changes)\r
+   (notmuch-pick-tag-update-display tag-changes))\r
+ \r
+-(defun notmuch-pick-add-tag ()\r
++(defun notmuch-pick-add-tag (tag-changes)\r
+   "Same as `notmuch-pick-tag' but sets initial input to '+'."\r
+-  (interactive)\r
+-  (notmuch-pick-tag "+"))\r
++  (interactive\r
++   (list (notmuch-read-tag-changes (notmuch-pick-get-tags) "Tag message" "+")))\r
++  (notmuch-pick-tag tag-changes))\r
+ \r
+-(defun notmuch-pick-remove-tag ()\r
++(defun notmuch-pick-remove-tag (tag-changes)\r
+   "Same as `notmuch-pick-tag' but sets initial input to '-'."\r
+-  (interactive)\r
+-  (notmuch-pick-tag "-"))\r
++  (interactive\r
++   (list (notmuch-read-tag-changes (notmuch-pick-get-tags) "Tag message" "-")))\r
++  (notmuch-pick-tag tag-changes))\r
+ \r
+ ;; The next two functions close the message window before searching or\r
+ ;; picking but they do so after the user has entered the query (in\r
+@@ -626,13 +629,16 @@ message will be \"unarchived\", i.e. the tag changes in\r
+            (notmuch-pick-thread-mapcar 'notmuch-pick-get-message-id)\r
+            " or "))\r
+ \r
+-(defun notmuch-pick-tag-thread (&optional tag-changes)\r
++(defun notmuch-pick-tag-thread (tag-changes)\r
+   "Tag all messages in the current thread"\r
+-  (interactive)\r
++  (interactive\r
++   (let ((tags (apply #'append (notmuch-pick-thread-mapcar\r
++                              (lambda () (notmuch-pick-get-tags))))))\r
++     (list (notmuch-read-tag-changes tags "Tag thread"))))\r
+   (when (notmuch-pick-get-message-properties)\r
+-    (let ((tag-changes (notmuch-tag (notmuch-pick-get-messages-ids-thread-search) tag-changes)))\r
+-      (notmuch-pick-thread-mapcar\r
+-       (lambda () (notmuch-pick-tag-update-display tag-changes))))))\r
++    (notmuch-tag (notmuch-pick-get-messages-ids-thread-search) tag-changes)\r
++    (notmuch-pick-thread-mapcar\r
++     (lambda () (notmuch-pick-tag-update-display tag-changes)))))\r
+ \r
+ (defun notmuch-pick-archive-thread (&optional unarchive)\r
+   "Archive each message in thread.\r
+-- \r
+1.8.4.rc3\r
+\r