Re: [PATCH] emacs: wash: make word-wrap bound message width

[notmuch-archives.git] / df / 3cb2e43f9debd94e2c4e72e7fb0e2196c999a8

Return-Path: <cworth@cworth.org>\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 262D04196F0\r
        for <notmuch@notmuchmail.org>; Wed,  7 Apr 2010 14:10:40 -0700 (PDT)\r
X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
X-Spam-Flag: NO\r
X-Spam-Score: -2.89\r
X-Spam-Level: \r
X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5\r
        tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01]\r
        autolearn=ham\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 0ngserUuMYMX; Wed,  7 Apr 2010 14:10:39 -0700 (PDT)\r
Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
        by olra.theworths.org (Postfix) with ESMTP id 0FA64431FC1;\r
        Wed,  7 Apr 2010 14:10:39 -0700 (PDT)\r
Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
        id B0E5525400C; Wed,  7 Apr 2010 14:10:38 -0700 (PDT)\r
From: Carl Worth <cworth@cworth.org>\r
To: Jesse Rosenthal <jrosenthal@jhu.edu>, notmuch@notmuchmail.org\r
Subject: Re: [notmuch] [PATCH v2] notmuch.el: add functionality in notmuch\r
        search mode to add or remove tags by region\r
In-Reply-To: <87sk90ragj.fsf@jhu.edu>\r
References: <m1636w7au5.fsf@watt.gilman.jhu.edu> <87sk90ragj.fsf@jhu.edu>\r
Date: Wed, 07 Apr 2010 14:10:38 -0700\r
Message-ID: <87aatfq88h.fsf@yoom.home.cworth.org>\r
MIME-Version: 1.0\r
Content-Type: multipart/signed; boundary="=-=-=";\r
        micalg=pgp-sha1; protocol="application/pgp-signature"\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: Wed, 07 Apr 2010 21:10:40 -0000\r
\r
--=-=-=\r
Content-Transfer-Encoding: quoted-printable\r
\r
On Tue, 16 Feb 2010 19:07:40 -0500, Jesse Rosenthal <jrosenthal@jhu.edu> wr=\r
ote:\r
> This patch adds `-region' versions of the `notmuch-search-' commands to f=\r
ind\r
> properties. It also splits up  `notmuch-add/remove-tags' into both a\r
> `-thread' and a `-region' version. (This makes us modify\r
> `notmuch-search-archive-thread' to use the\r
> `notmuch-search-remove-tag-thread' function, instead of\r
> `notmuch-search-remove-tag', for consistency.) The add/remove-tag command\r
> called by pressing `+' or `-' will then choose accordingly, based on whet=\r
her\r
> region is active.\r
>=20\r
> This version fixes a couple of errors in the first version, which led to\r
> incorrect marking of some tags in the search view (though the actual\r
> tagging was still correct). It's also based on current master.\r
>=20\r
> I'm not sure any more if region selection is actually the correct way to\r
> do this, or if a mutt-style message-marking method would be better. But\r
> I didn't want a buggy incorrect version out there.\r
\r
I think this feature is very useful, and that the region is definitely\r
an appropriate way to implement it, (doing region-based operations is\r
very natural for emacs users). Mutt-style marking could be implemented\r
as well, but that would be separate I think.\r
\r
I tested this patch a bit and added one small cleanup to the\r
documentation (see below).\r
\r
I also don't like the duplication of code in\r
notmuch-search-add-tag-thread and notmuch-search-add-tag-region, (and\r
the same in the remove case). Fortunately, I think this easy to avoid by\r
simply making notmuch-search-add-tag-thread call:\r
\r
       (notmuch-search-add-tag-region tag (point) (point))\r
\r
and the same in the remove case.\r
\r
But I haven't pushed this patch yet for a flaw in the case of selecting\r
beyond the last thread, (such as selecting to the line that includes the\r
"End of search results). If I try an operation on that line, it will act\r
like it works, (it displays the new tags by all threads), but then a\r
refresh makes them all disappear again. Presumably the "notmuch tag"\r
command is failing, this isn't being noticed, and the code marches on to\r
update the representation of the tags.\r
\r
And presumably the notmuch tag is failing because no thread ID is found\r
for the last line, so the mapconcat call is sticking an extra " or "\r
onto the end of the search string. And Xapian doesn't like that:\r
\r
        $ notmuch search tag:inbox or\r
        A Xapian exception occurred performing query: Syntax: <expression> OR <exp=\r
ression>\r
        Query string was: tag:inbox or\r
\r
Things behave even worse if I make the region be the entire buffer,\r
(including the last blank line). Then the commands hang. I got nervous\r
that this was then adding "or or" and trying to add/remove a tag to\r
every message containing the word "or". But I haven't looked closely.\r
\r
If we can fix this bug, then I'd like to apply this patch. I'd even like\r
to fix the "*" binding to be implemented in terms of something like\r
these functions so that we could get visual updates of the tag state\r
From=20that command. (But that will take some reworking of the interface\r
as the current implementation can't support multiple tags added/removed\r
as * current expects).\r
\r
I'd appreciate any help fixing up the patch.\r
\r
=2DCarl\r
\r
From=20fec5622add1a4e9f305c16e96143439ee22a5c58 Mon Sep 17 00:00:00 2001\r
From: Carl Worth <cworth@cworth.org>\r
Date: Wed, 7 Apr 2010 13:15:27 -0700\r
Subject: [PATCH] emacs: Correct the documentation for notmuch-search-add-ta=\r
g (and -remove-tag)\r
\r
These commands act on all messages in the thread, not simply those\r
that match the search. (There are use case for both behaviors, but the\r
documentation must match the behavior that's actually implemented).\r
=2D--\r
 emacs/notmuch.el |    9 ++++-----\r
 1 files changed, 4 insertions(+), 5 deletions(-)\r
\r
diff --git a/emacs/notmuch.el b/emacs/notmuch.el\r
index 64f3e3d..517c53a 100644\r
=2D-- a/emacs/notmuch.el\r
+++ b/emacs/notmuch.el\r
@@ -532,12 +532,11 @@ and will also appear in a buffer named \"*Notmuch err=\r
ors*\"."\r
          (notmuch-search-set-tags (delete tag (notmuch-search-get-tags)))\r
          (forward-line))))))\r
=20\r
=2D\r
 (defun notmuch-search-add-tag (tag)\r
   "Add a tag to the currently selected thread or region.\r
=20\r
=2DThe tag is added to messages in the currently selected thread or\r
=2Dregion which match the current search terms."\r
+The tag is added to all messages in the currently selected thread\r
+or threads in the current region."\r
   (interactive\r
    (list (notmuch-select-tag-with-completion "Tag to add: ")))\r
   (save-excursion\r
@@ -550,8 +549,8 @@ region which match the current search terms."\r
 (defun notmuch-search-remove-tag (tag)\r
   "Remove a tag from the currently selected thread or region.\r
=20\r
=2DThe tag is removed from all messages in the currently selected thread\r
=2Dor region which match the current search terms."\r
+The tag is removed from all messages in the currently selected\r
+thread or threads in the current region."\r
   (interactive\r
    (list (notmuch-select-tag-with-completion\r
          "Tag to remove: "\r
=2D-=20\r
1.7.0\r
\r
\r
--=-=-=\r
Content-Type: application/pgp-signature\r
\r
-----BEGIN PGP SIGNATURE-----\r
Version: GnuPG v1.4.10 (GNU/Linux)\r
\r
iD8DBQFLvPTO6JDdNq8qSWgRAsxHAKCpnJDPWvi6vZhr6L0qctDiAcoryACbBbTJ\r
CIsitjtUX2yLdude6PH796M=\r
=25Nm\r
-----END PGP SIGNATURE-----\r
--=-=-=--\r