[PATCH 4/8] emacs: Use interactive specifications for tag changes in search
authorAustin Clements <amdragon@MIT.EDU>
Tue, 22 Oct 2013 19:50:53 +0000 (15:50 +2000)
committerW. Trevor King <wking@tremily.us>
Fri, 7 Nov 2014 17:57:31 +0000 (09:57 -0800)
f3/d889eee690a354a82e3ed8baa5e9db32433065 [new file with mode: 0644]

diff --git a/f3/d889eee690a354a82e3ed8baa5e9db32433065 b/f3/d889eee690a354a82e3ed8baa5e9db32433065
new file mode 100644 (file)
index 0000000..268b13c
--- /dev/null
@@ -0,0 +1,201 @@
+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 5C177431FD7\r
+       for <notmuch@notmuchmail.org>; Tue, 22 Oct 2013 12:51:48 -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 CAZ2u8zGdxJ3 for <notmuch@notmuchmail.org>;\r
+       Tue, 22 Oct 2013 12:51:42 -0700 (PDT)\r
+Received: from dmz-mailsec-scanner-6.mit.edu (dmz-mailsec-scanner-6.mit.edu\r
+       [18.7.68.35])\r
+       by olra.theworths.org (Postfix) with ESMTP id 5D110429E32\r
+       for <notmuch@notmuchmail.org>; Tue, 22 Oct 2013 12:51:17 -0700 (PDT)\r
+X-AuditID: 12074423-b7fc98e0000009a2-56-5266d7339018\r
+Received: from mailhub-auth-3.mit.edu ( [18.9.21.43])\r
+       by dmz-mailsec-scanner-6.mit.edu (Symantec Messaging Gateway) with SMTP\r
+       id 28.6A.02466.337D6625; 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-3.mit.edu (8.13.8/8.9.2) with ESMTP id r9MJp39r030254; \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 r9MJp17R005196\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 1VYhym-0006yk-PV; Tue, 22 Oct 2013 15:51:00 -0400\r
+From: Austin Clements <amdragon@MIT.EDU>\r
+To: notmuch@notmuchmail.org\r
+Subject: [PATCH 4/8] emacs: Use interactive specifications for tag changes in\r
+       search\r
+Date: Tue, 22 Oct 2013 15:50:53 -0400\r
+Message-Id: <1382471457-26056-5-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+NgFtrEIsWRmVeSWpSXmKPExsUixCmqrWt8PS3IYMdeHovrN2cyOzB6PFt1\r
+       izmAMYrLJiU1J7MstUjfLoEr48TsP0wFr7UqFi9qY2tgvKvcxcjJISFgIrHu6k4WCFtM4sK9\r
+       9WxdjFwcQgL7GCX2b1vABOFsZJSYefY+K4RzjEniYs9/dghnLqPEyu9vmED62QQ0JLbtX84I\r
+       YosISEvsvDsbqIODg1lATeJPlwpIWFggROJ0wyQ2EJtFQFWi+eJCZhCbV8BBorfvPBvEGUoS\r
+       C09tYwWxOQUcJabv+8cGMkYIqObUHY4JjPwLGBlWMcqm5Fbp5iZm5hSnJusWJyfm5aUW6Zrp\r
+       5WaW6KWmlG5iBAUNu4vyDsY/B5UOMQpwMCrx8D5oSQsSYk0sK67MPcQoycGkJMobcRUoxJeU\r
+       n1KZkVicEV9UmpNafIhRgoNZSYS32Rcox5uSWFmVWpQPk5LmYFES573FYR8kJJCeWJKanZpa\r
+       kFoEk5Xh4FCS4BW6BtQoWJSanlqRlplTgpBm4uAEGc4DNPwYyGLe4oLE3OLMdIj8KUZFKXFe\r
+       YZBmAZBERmkeXC8sql8xigO9IszLCVLFA0wIcN2vgAYzAQ2esgRscEkiQkoKGN36UvHzpvCw\r
+       a5e6OMbp9UhWLnoStD+rlCVQSMdy/dPgyTZZIRbS6zb43j68vNgz0ObO6p1xC87xc0o6FTLu\r
+       atdxZO7mqDng+pJdjf/ArgTbuCjmXwqZge0Kzh/sN733W7G02vyEseTbqZptfAU6RXWqrXl6\r
+       2z/k5kzYpetxT+1UsdIegx9KLMUZiYZazEXFiQDnOcw0xQIAAA==\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:48 -0000\r
+\r
+This is similar to the previous commit, but applies to search.\r
+\r
+Search is somewhat more complicated because its tagging operations can\r
+also apply to a region.  Hence, this lifts interactive prompting into\r
+a helper function.  This also takes advantage of the new ability to\r
+provide a prompt to distinguish tagging a single thread from tagging a\r
+region of threads.\r
+---\r
+ emacs/notmuch.el | 70 +++++++++++++++++++++++++++++++++++---------------------\r
+ 1 file changed, 44 insertions(+), 26 deletions(-)\r
+\r
+diff --git a/emacs/notmuch.el b/emacs/notmuch.el\r
+index e0511c8..5492e1b 100644\r
+--- a/emacs/notmuch.el\r
++++ b/emacs/notmuch.el\r
+@@ -560,43 +560,59 @@ will be signaled."\r
+       (setq output (append output (notmuch-search-get-tags pos)))))\r
+     output))\r
\r
+-(defun notmuch-search-tag-region (beg end &optional tag-changes)\r
+-  "Change tags for threads in the given region."\r
++(defun notmuch-search-interactive-region ()\r
++  "Return the bounds of the current interactive region.\r
++\r
++This returns (BEG END), where BEG and END are the bounds of the\r
++region if the region is active, or both `point' otherwise."\r
++  (if (region-active-p)\r
++      (list (region-beginning) (region-end))\r
++    (list (point) (point))))\r
++\r
++(defun notmuch-search-interactive-tag-changes (&optional initial-input)\r
++  "Prompt for tag changes for the current thread or region.\r
++\r
++Returns (TAG-CHANGES REGION-BEGIN REGION-END)."\r
++  (let* ((region (notmuch-search-interactive-region))\r
++       (beg (first region)) (end (second region))\r
++       (prompt (if (= beg end) "Tag thread" "Tag region")))\r
++    (cons (notmuch-read-tag-changes\r
++         (notmuch-search-get-tags-region beg end) prompt initial-input)\r
++        region)))\r
++\r
++(defun notmuch-search-tag (tag-changes &optional beg end)\r
++  "Change tags for the currently selected thread or region.\r
++\r
++See `notmuch-tag' for information on the format of TAG-CHANGES.\r
++If BEG or END are nil, applies to the thread at point."\r
++  (interactive (notmuch-search-interactive-tag-changes))\r
++  (unless (and beg end) (setq beg (point) end (point)))\r
+   (let ((search-string (notmuch-search-find-thread-id-region-search beg end)))\r
+-    (setq tag-changes (notmuch-tag search-string tag-changes))\r
++    (notmuch-tag search-string tag-changes)\r
+     (notmuch-search-foreach-result beg end\r
+       (lambda (pos)\r
+       (notmuch-search-set-tags\r
+        (notmuch-update-tags (notmuch-search-get-tags pos) tag-changes)\r
+        pos)))))\r
\r
+-(defun notmuch-search-tag (&optional tag-changes)\r
+-  "Change tags for the currently selected thread or region.\r
+-\r
+-See `notmuch-tag' for information on the format of TAG-CHANGES."\r
+-  (interactive)\r
+-  (let* ((beg (if (region-active-p) (region-beginning) (point)))\r
+-       (end (if (region-active-p) (region-end) (point))))\r
+-    (notmuch-search-tag-region beg end tag-changes)))\r
+-\r
+-(defun notmuch-search-add-tag ()\r
+-  "Change tags for the current thread (defaulting to add).\r
++(defun notmuch-search-add-tag (tag-changes &optional beg end)\r
++  "Change tags for the current thread or region (defaulting to add).\r
\r
+ Same as `notmuch-search-tag' but sets initial input to '+'."\r
+-  (interactive)\r
+-  (notmuch-search-tag "+"))\r
++  (interactive (notmuch-search-interactive-tag-changes "+"))\r
++  (notmuch-search-tag tag-changes beg end))\r
\r
+-(defun notmuch-search-remove-tag ()\r
+-  "Change tags for the current thread (defaulting to remove).\r
++(defun notmuch-search-remove-tag (tag-changes &optional beg end)\r
++  "Change tags for the current thread or region (defaulting to remove).\r
\r
+ Same as `notmuch-search-tag' but sets initial input to '-'."\r
+-  (interactive)\r
+-  (notmuch-search-tag "-"))\r
++  (interactive (notmuch-search-interactive-tag-changes "-"))\r
++  (notmuch-search-tag tag-changes beg end))\r
\r
+ (put 'notmuch-search-archive-thread 'notmuch-prefix-doc\r
+      "Un-archive the currently selected thread.")\r
+-(defun notmuch-search-archive-thread (&optional unarchive)\r
+-  "Archive the currently selected thread.\r
++(defun notmuch-search-archive-thread (&optional unarchive beg end)\r
++  "Archive the currently selected thread or region.\r
\r
+ Archive each message in the currently selected thread by applying\r
+ the tag changes in `notmuch-archive-tags' to each (remove the\r
+@@ -605,10 +621,10 @@ messages will be \"unarchived\" (i.e. the tag changes in\r
+ `notmuch-archive-tags' will be reversed).\r
\r
+ This function advances the next thread when finished."\r
+-  (interactive "P")\r
++  (interactive (cons current-prefix-arg (notmuch-search-interactive-region)))\r
+   (when notmuch-archive-tags\r
+     (notmuch-search-tag\r
+-     (notmuch-tag-change-list notmuch-archive-tags unarchive)))\r
++     (notmuch-tag-change-list notmuch-archive-tags unarchive) beg end))\r
+   (notmuch-search-next-thread))\r
\r
+ (defun notmuch-search-update-result (result &optional pos)\r
+@@ -832,11 +848,13 @@ non-authors is found, assume that all of the authors match."\r
+       (notmuch-sexp-parse-partial-list 'notmuch-search-show-result\r
+                                        results-buf)))))\r
\r
+-(defun notmuch-search-tag-all (&optional tag-changes)\r
++(defun notmuch-search-tag-all (tag-changes)\r
+   "Add/remove tags from all messages in current search buffer.\r
\r
+ See `notmuch-tag' for information on the format of TAG-CHANGES."\r
+-  (interactive)\r
++  (interactive\r
++   (list (notmuch-read-tag-changes\r
++        (notmuch-search-get-tags-region (point-min) (point-max)) "Tag all")))\r
+   (notmuch-tag notmuch-search-query-string tag-changes))\r
\r
+ (defun notmuch-search-buffer-title (query)\r
+-- \r
+1.8.4.rc3\r
+\r