00/a79adadd1b712a9c0fad3993132e7085deaf36

   1 Return-Path: <jrollins@finestructure.net>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id 1B419431FBF\r
   6         for <notmuch@notmuchmail.org>; Wed, 24 Feb 2010 11:01:27 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -4.108\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-4.108 tagged_above=-999 required=5\r
  12         tests=[AWL=-0.109, BAYES_50=0.001, RCVD_IN_DNSWL_MED=-4]\r
  13         autolearn=unavailable\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id liJhqdVadh5Q for <notmuch@notmuchmail.org>;\r
  17         Wed, 24 Feb 2010 11:01:27 -0800 (PST)\r
  18 Received: from brinza.cc.columbia.edu (brinza.cc.columbia.edu [128.59.29.8])\r
  19         by olra.theworths.org (Postfix) with ESMTP id DCA8F431FAE\r
  20         for <notmuch@notmuchmail.org>; Wed, 24 Feb 2010 11:01:26 -0800 (PST)\r
  21 Received: from servo.finestructure.net (geco.phys.columbia.edu\r
  22         [128.59.170.159])\r
  23         (user=jgr2110 author=jrollins@finestructure.net mech=PLAIN bits=0)\r
  24         by brinza.cc.columbia.edu (8.14.3/8.14.3) with ESMTP id o1OJ1LQ9003942\r
  25         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  26         Wed, 24 Feb 2010 14:01:22 -0500 (EST)\r
  27 Received: from jrollins by servo.finestructure.net with local (Exim 4.71)\r
  28         (envelope-from <jrollins@finestructure.net>)\r
  29         id 1NkMU5-0006tx-Pa; Wed, 24 Feb 2010 14:01:21 -0500\r
  30 From: Jameson Rollins <jrollins@finestructure.net>\r
  31 To: Carl Worth <cworth@cworth.org>, racin@free.fr, Sebastian Spaeth\r
  32         <Sebastian@SSpaeth.de>\r
  33 In-Reply-To: <87sk8qwjlt.fsf@yoom.home.cworth.org>\r
  34 References:\r
  35  <1161359196.2247691263984967906.JavaMail.root@zimbra1-e1.priv.proxad.net>\r
  36         <87sk8qwjlt.fsf@yoom.home.cworth.org>\r
  37 Date: Wed, 24 Feb 2010 14:01:18 -0500\r
  38 Message-ID: <87tyt61mrl.fsf@servo.finestructure.net>\r
  39 MIME-Version: 1.0\r
  40 Content-Type: multipart/signed; boundary="=-=-=";\r
  41         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  42 X-No-Spam-Score: Local\r
  43 X-Scanned-By: MIMEDefang 2.68 on 128.59.29.8\r
  44 Cc: notmuch <notmuch@notmuchmail.org>\r
  45 Subject: Re: [notmuch] [PATCH] add notmuch-show-delete keybinding 'd'\r
  46 X-BeenThere: notmuch@notmuchmail.org\r
  47 X-Mailman-Version: 2.1.13\r
  48 Precedence: list\r
  49 List-Id: "Use and development of the notmuch mail system."\r
  50         <notmuch.notmuchmail.org>\r
  51 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  52         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  53 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  54 List-Post: <mailto:notmuch@notmuchmail.org>\r
  55 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  56 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  57         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  58 X-List-Received-Date: Wed, 24 Feb 2010 19:01:27 -0000\r
  59 \r
  60 --=-=-=\r
  61 Content-Transfer-Encoding: quoted-printable\r
  62 \r
  63 On Wed, 24 Feb 2010 10:53:50 -0800, Carl Worth <cworth@cworth.org> wrote:\r
  64 > But this patch does have two good ideas not in the other patch, (both of\r
  65 > which I mentioned in the review):\r
  66 >=20\r
  67 > 1. It adds a keybinding to the notmuch-show mode\r
  68 >=20\r
  69 > 2. It removes the "inbox" and "unread" tags while adding the tag to\r
  70 >    indicate deletion.\r
  71 \r
  72 Hey, Carl.  Why is this last point important?  I've been using my own\r
  73 patchs for handling deleted messages, and all deleting a message or\r
  74 thread does is add the "delete" tag.  Why should it modify any other\r
  75 tags?  A message/thread should be allowed to be both deleted and in the\r
  76 inbox.\r
  77 \r
  78 As for "unread", I think that should be handled by actually reading the\r
  79 message, not by manually applying a state to it.\r
  80 \r
  81 FWIW, below are the functions I've added to my notmuch .el to handle\r
  82 message/thread deleting.\r
  83 \r
  84 jamie.\r
  85 \r
  86 (defun notmuch-search-delete-thread ()\r
  87   "Delete thread (add \"delete\" tag).\r
  88 \r
  89 This function advances the next thread when finished."\r
  90   (interactive)\r
  91   (notmuch-search-add-tag "delete")\r
  92   (forward-line))\r
  93 \r
  94 (define-key notmuch-search-mode-map "d" 'notmuch-search-delete-thread)\r
  95 \r
  96 (defun notmuch-show-delete-message ()\r
  97   "Delete message (add \"delete\" tag).\r
  98 \r
  99 Add the \"delete\" tag to message. Then kill this buffer and\r
 100 show the next thread from the search from which this thread was\r
 101 originally shown."\r
 102   (interactive)\r
 103   (notmuch-show-add-tag "delete")\r
 104   (let ((parent-buffer notmuch-show-parent-buffer))\r
 105     (kill-this-buffer)\r
 106     (if parent-buffer\r
 107         (progn\r
 108           (switch-to-buffer parent-buffer)\r
 109           (forward-line)))))\r
 110 \r
 111 (define-key notmuch-show-mode-map "d" 'notmuch-show-delete-message)\r
 112 \r
 113 --=-=-=\r
 114 Content-Type: application/pgp-signature\r
 115 \r
 116 -----BEGIN PGP SIGNATURE-----\r
 117 Version: GnuPG v1.4.10 (GNU/Linux)\r
 118 \r
 119 iQIcBAEBCAAGBQJLhXd/AAoJEO00zqvie6q8UKYP/RpDAHG9cGVCcNKyHp3C5iFm\r
 120 H8Th94jndZK2CEUDpPM0oqZ9z5xENTdb46OEjsfX35CuFAXAsJ2F0hqsKOtLp7F+\r
 121 HKrCkrJth4+x8qn9jFSPh9U8FJjuG8NgtlzkiR0VuLW6KWgY/JU+ye9tIql6rFNM\r
 122 5PqRBxhVeGB9k+mq6vUXJxWrrbnIalAG3/QCSKLOZJKC8I0fKPsOwq55vgObkRn5\r
 123 kwjw1uFsA3RG8vg2qRl+qmMyGDl3IxoYBRZwx6hrL+o+mMjmHqlh4xCKCQHCWgqN\r
 124 CbK9pTRG0aMbSkJoyQH1FtTnQMC0ZWCnFV00ptdsDAHnSL++rP9R9pj/blnml1fT\r
 125 jLKZC19A8+9HqBOXFpKUq4IRyG7/QZSVE0CLl5iQlplxsswuQAnLbPEpveq8fJQ7\r
 126 q+ei22t4G/AXrVg1ZaNfTgN1OJik1deDsEUSzs67XVe3mYfRrNcWH5WJ8lRqtHmb\r
 127 mRt8wEyC4t/p6Xd2IplfLZIau//ca6oM0r7UahXqBWgRpqEVLmigufMTeSJxSsFq\r
 128 jS3XTd2lUx8WYgiJujKJmmyft1XQZJFxlmRUEi/SLjQOL5L/t570ctOhvmSiD3E/\r
 129 GW3Hjql/Y4Hexaezcmv0uYxf8Oi9H7dV/IdTf62o2mUfiahUR8PcW6+PJRAd9djm\r
 130 vUHmcEMZvdeKFwssyCyA\r
 131 =yD4W\r
 132 -----END PGP SIGNATURE-----\r
 133 --=-=-=--\r