06/6e2ddda4b66f6cd1f871e4afabf3a37600a3dd

   1 Return-Path: <m.walters@qmul.ac.uk>\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 AE2EF431FBC\r
   6         for <notmuch@notmuchmail.org>; Sun, 30 Jun 2013 01:46:17 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 1.401\r
  10 X-Spam-Level: *\r
  11 X-Spam-Status: No, score=1.401 tagged_above=-999 required=5\r
  12         tests=[DKIM_ADSP_CUSTOM_MED=0.001, FREEMAIL_FROM=0.001,\r
  13         FREEMAIL_REPLY=2.499, NML_ADSP_CUSTOM_MED=1.2, RCVD_IN_DNSWL_MED=-2.3]\r
  14         autolearn=disabled\r
  15 Received: from olra.theworths.org ([127.0.0.1])\r
  16         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  17         with ESMTP id kvqxr7xx6pws for <notmuch@notmuchmail.org>;\r
  18         Sun, 30 Jun 2013 01:46:10 -0700 (PDT)\r
  19 Received: from mail2.qmul.ac.uk (mail2.qmul.ac.uk [138.37.6.6])\r
  20         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  21         (No client certificate requested)\r
  22         by olra.theworths.org (Postfix) with ESMTPS id EE0E9431FB6\r
  23         for <notmuch@notmuchmail.org>; Sun, 30 Jun 2013 01:46:09 -0700 (PDT)\r
  24 Received: from smtp.qmul.ac.uk ([138.37.6.40])\r
  25         by mail2.qmul.ac.uk with esmtp (Exim 4.71)\r
  26         (envelope-from <m.walters@qmul.ac.uk>)\r
  27         id 1UtDGq-0003Ix-AQ; Sun, 30 Jun 2013 09:46:08 +0100\r
  28 Received: from 93-97-24-31.zone5.bethere.co.uk ([93.97.24.31] helo=localhost)\r
  29         by smtp.qmul.ac.uk with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.71)\r
  30         (envelope-from <m.walters@qmul.ac.uk>)\r
  31         id 1UtDGq-00053s-2c; Sun, 30 Jun 2013 09:46:08 +0100\r
  32 From: Mark Walters <markwalters1009@gmail.com>\r
  33 To: notmuch@notmuchmail.org\r
  34 Subject: Re: [PATCH 0/2] Contrib: Pick: Remove horrible hack\r
  35 In-Reply-To: <87sj00xapn.fsf@qmul.ac.uk>\r
  36 References: <1369551008-30697-1-git-send-email-markwalters1009@gmail.com>\r
  37         <87sj00xapn.fsf@qmul.ac.uk>\r
  38 User-Agent: Notmuch/0.15.2+192~g8222af3 (http://notmuchmail.org) Emacs/23.4.1\r
  39         (x86_64-pc-linux-gnu)\r
  40 Date: Sun, 30 Jun 2013 09:46:07 +0100\r
  41 Message-ID: <87ppv4x9yo.fsf@qmul.ac.uk>\r
  42 MIME-Version: 1.0\r
  43 Content-Type: text/plain; charset=us-ascii\r
  44 X-Sender-Host-Address: 93.97.24.31\r
  45 X-QM-SPAM-Info: Sender has good ham record.  :)\r
  46 X-QM-Body-MD5: c41d29687ee397f107f411006a432bd6 (of first 20000 bytes)\r
  47 X-SpamAssassin-Score: 0.6\r
  48 X-SpamAssassin-SpamBar: /\r
  49 X-SpamAssassin-Report: The QM spam filters have analysed this message to\r
  50         determine if it is\r
  51         spam. We require at least 5.0 points to mark a message as spam.\r
  52         This message scored 0.6 points. Summary of the scoring: \r
  53         * 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\r
  54         provider *      (markwalters1009[at]gmail.com)\r
  55         *  1.0 FREEMAIL_REPLY From and body contain different freemails\r
  56         * -0.4 AWL AWL: From: address is in the auto white-list\r
  57 X-QM-Scan-Virus: ClamAV says the message is clean\r
  58 X-BeenThere: notmuch@notmuchmail.org\r
  59 X-Mailman-Version: 2.1.13\r
  60 Precedence: list\r
  61 List-Id: "Use and development of the notmuch mail system."\r
  62         <notmuch.notmuchmail.org>\r
  63 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  64         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  65 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  66 List-Post: <mailto:notmuch@notmuchmail.org>\r
  67 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  68 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  69         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  70 X-List-Received-Date: Sun, 30 Jun 2013 08:46:17 -0000\r
  71 \r
  72 \r
  73 Actually I will post a new version as this version doesn't apply anymore\r
  74 (the context which overlaps an unrelated function has changed)\r
  75 \r
  76 Best wishes\r
  77 \r
  78 Mark\r
  79 \r
  80 \r
  81 On Sun, 30 Jun 2013, Mark Walters <markwalters1009@gmail.com> wrote:\r
  82 > Would anyone be able to review (or just test) this pair of patches: they\r
  83 > are smaller and simpler than the diffstat suggests: the second patch is\r
  84 > just 3 extra lines of code (with some whitespace change and commments).\r
  85 >\r
  86 > This pair of patches does remove the worst piece of code in pick: a\r
  87 > sleep loop waiting for the correct message to arrive.\r
  88 >\r
  89 > I should have said that it applies on top of\r
  90 > id:1369550458-30562-1-git-send-email-markwalters1009@gmail.com\r
  91 >\r
  92 > (which is also very simple)\r
  93 >\r
  94 > Many thanks\r
  95 >\r
  96 > Mark\r
  97 >\r
  98 >\r
  99 >\r
 100 >\r
 101 > On Sun, 26 May 2013, Mark Walters <markwalters1009@gmail.com> wrote:\r
 102 >> Pick used to have one horrible hack: if the user asked it to open the\r
 103 >> first matching message it had to check whether that had arrived (as\r
 104 >> the search is asynchronous) and if not wait and try again. Now the\r
 105 >> opening of the first matching message is called via the pick process\r
 106 >> filter this hack can be removed.\r
 107 >>\r
 108 >> This did reveal the followibg small bug.  Pick shows the subject line\r
 109 >> in the output but if it is the same as the previous line (ignoring re:\r
 110 >> etc) it shows ... If a single message is refreshed (eg for a tag\r
 111 >> update) this was got wrong. The change above triggered this and made\r
 112 >> the test fail as the unread tag was removed from the first matching\r
 113 >> message when it was displayed.\r
 114 >>\r
 115 >> Patch 2/2 fixes this by storing the previous subject with the search result.\r
 116 >>\r
 117 >> Best wishes\r
 118 >>\r
 119 >> Mark\r
 120 >>  \r
 121 >>\r
 122 >> Mark Walters (2):\r
 123 >>   contrib: pick: remove hack notmuch-pick-show-match-message-with-wait\r
 124 >>   contrib: pick: fix refresh result\r
 125 >>\r
 126 >>  contrib/notmuch-pick/notmuch-pick.el |   60 ++++++++++++++++-----------------\r
 127 >>  1 files changed, 29 insertions(+), 31 deletions(-)\r
 128 >>\r
 129 >> -- \r
 130 >> 1.7.9.1\r