[PATCH v4 08/12] emacs: separate history for operations which accept single and multi...

diff --git a/42/b0121e6c391d5b81df248f7ca6a1988b0595cb b/42/b0121e6c391d5b81df248f7ca6a1988b0595cb
new file mode 100644 (file)
index 0000000..808af87
--- /dev/null
+++ b/42/b0121e6c391d5b81df248f7ca6a1988b0595cb
@@ -0,0 +1,114 @@
+Return-Path: <dmitry.kurochkin@gmail.com>\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 195B6417325\r
+       for <notmuch@notmuchmail.org>; Mon, 30 Jan 2012 20:56:02 -0800 (PST)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.799\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
+       tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
+       FREEMAIL_FROM=0.001, 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 ITdEHPeSX0PS for <notmuch@notmuchmail.org>;\r
+       Mon, 30 Jan 2012 20:56:01 -0800 (PST)\r
+Received: from mail-bk0-f53.google.com (mail-bk0-f53.google.com\r
+       [209.85.214.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
+       (No client certificate requested)\r
+       by olra.theworths.org (Postfix) with ESMTPS id B099241ED91\r
+       for <notmuch@notmuchmail.org>; Mon, 30 Jan 2012 20:56:00 -0800 (PST)\r
+Received: by mail-bk0-f53.google.com with SMTP id 11so2152468bke.26\r
+       for <notmuch@notmuchmail.org>; Mon, 30 Jan 2012 20:56:00 -0800 (PST)\r
+DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma;\r
+       h=from:to:subject:date:message-id:x-mailer:in-reply-to:references;\r
+       bh=6OBNRk2FhWOO5ldIxvA3mkHEDfP1ByFVeBAjEP0ddyg=;\r
+       b=pmOLuwv8td6QrAZhbxynhPLcOw1e7bC3UwHwi4WXTPexKhukF+xBEha8/9xA385aX7\r
+       bKiLQTYKwfo4haY6UCDV9fYbN23J/p/AgpjOZIYNTyBFC2VHhs8an3nXEcwayJu001V3\r
+       xLPXo6pP4Oo5WtEhLhD4SHX/jD7zOIJuA8UlY=\r
+Received: by 10.205.131.13 with SMTP id ho13mr10260406bkc.58.1327985760361;\r
+       Mon, 30 Jan 2012 20:56:00 -0800 (PST)\r
+Received: from localhost ([91.144.186.21])\r
+       by mx.google.com with ESMTPS id o26sm36264418bko.14.2012.01.30.20.55.59\r
+       (version=TLSv1/SSLv3 cipher=OTHER);\r
+       Mon, 30 Jan 2012 20:55:59 -0800 (PST)\r
+From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
+To: notmuch@notmuchmail.org\r
+Subject: [PATCH v4 08/12] emacs: separate history for operations which accept\r
+       single and multiple tags\r
+Date: Tue, 31 Jan 2012 08:54:22 +0400\r
+Message-Id: <1327985666-29191-9-git-send-email-dmitry.kurochkin@gmail.com>\r
+X-Mailer: git-send-email 1.7.9\r
+In-Reply-To: <1327985666-29191-1-git-send-email-dmitry.kurochkin@gmail.com>\r
+References: <1327901644-15799-1-git-send-email-dmitry.kurochkin@gmail.com>\r
+       <1327985666-29191-1-git-send-email-dmitry.kurochkin@gmail.com>\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, 31 Jan 2012 04:56:02 -0000\r
+\r
+Some tag-related operations accept a single tag without prefix\r
+(`notmuch-select-tag-with-completion'), others accept multiple tags\r
+prefixed with '+' or '-' (`notmuch-read-tag-changes').  Before the\r
+change, both functions used a single default minibuffer history.  This\r
+is inconvenient because you have to skip options with incompatible\r
+format when going through the history.  The patch adds separate\r
+history lists for the two functions.  Note that functions that accept\r
+the same input format (e.g. "+", "-", "*") share the history list as\r
+before.\r
+---\r
+ emacs/notmuch.el |   13 +++++++++++--\r
+ 1 files changed, 11 insertions(+), 2 deletions(-)\r
+\r
+diff --git a/emacs/notmuch.el b/emacs/notmuch.el\r
+index 7d06109..0a3bd17 100644\r
+--- a/emacs/notmuch.el\r
++++ b/emacs/notmuch.el\r
+@@ -76,6 +76,14 @@ For example:\r
+ (defvar notmuch-query-history nil\r
+   "Variable to store minibuffer history for notmuch queries")\r
+ \r
++(defvar notmuch-select-tag-history nil\r
++  "Variable to store minibuffer history for\r
++`notmuch-select-tag-with-completion' function.")\r
++\r
++(defvar notmuch-read-tag-changes-history nil\r
++  "Variable to store minibuffer history for\r
++`notmuch-read-tag-changes' function.")\r
++\r
+ (defun notmuch-tag-completions (&optional search-terms)\r
+   (split-string\r
+    (with-output-to-string\r
+@@ -86,7 +94,7 @@ For example:\r
+ \r
+ (defun notmuch-select-tag-with-completion (prompt &rest search-terms)\r
+   (let ((tag-list (notmuch-tag-completions search-terms)))\r
+-    (completing-read prompt tag-list)))\r
++    (completing-read prompt tag-list nil nil nil 'notmuch-select-tag-history)))\r
+ \r
+ (defun notmuch-read-tag-changes (&optional initial-input &rest search-terms)\r
+   (let* ((all-tag-list (notmuch-tag-completions))\r
+@@ -106,7 +114,8 @@ For example:\r
+           (define-key map " " 'self-insert-command)\r
+           map)))\r
+     (delete "" (completing-read-multiple "Tags (+add -drop): "\r
+-              tag-list nil nil initial-input))))\r
++              tag-list nil nil initial-input\r
++              'notmuch-read-tag-changes-history))))\r
+ \r
+ (defun notmuch-update-tags (tags tag-changes)\r
+   "Return a copy of TAGS with additions and removals from TAG-CHANGES.\r
+-- \r
+1.7.9\r
+\r