bf/f54bb4961127d9978c2928a97044de6f79e5b8

   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 72581431FBD\r
   6         for <notmuch@notmuchmail.org>; Wed,  4 Jul 2012 01:42:02 -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.098\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-1.098 tagged_above=-999 required=5\r
  12         tests=[DKIM_ADSP_CUSTOM_MED=0.001, FREEMAIL_FROM=0.001,\r
  13         NML_ADSP_CUSTOM_MED=1.2, RCVD_IN_DNSWL_MED=-2.3] autolearn=disabled\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 Cefhl3FB3QRg for <notmuch@notmuchmail.org>;\r
  17         Wed,  4 Jul 2012 01:42:02 -0700 (PDT)\r
  18 Received: from mail2.qmul.ac.uk (mail2.qmul.ac.uk [138.37.6.6])\r
  19         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id AB3FD431FB6\r
  22         for <notmuch@notmuchmail.org>; Wed,  4 Jul 2012 01:42:01 -0700 (PDT)\r
  23 Received: from smtp.qmul.ac.uk ([138.37.6.40])\r
  24         by mail2.qmul.ac.uk with esmtp (Exim 4.71)\r
  25         (envelope-from <m.walters@qmul.ac.uk>)\r
  26         id 1SmL9s-0007XZ-1i; Wed, 04 Jul 2012 09:42:00 +0100\r
  27 Received: from 94-192-233-223.zone6.bethere.co.uk ([94.192.233.223]\r
  28         helo=localhost)\r
  29         by smtp.qmul.ac.uk with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.69)\r
  30         (envelope-from <m.walters@qmul.ac.uk>)\r
  31         id 1SmL9r-0003I1-PH; Wed, 04 Jul 2012 09:41:59 +0100\r
  32 From: Mark Walters <markwalters1009@gmail.com>\r
  33 To: Austin Clements <amdragon@MIT.EDU>, notmuch@notmuchmail.org\r
  34 Subject: Re: [PATCH 4/8] emacs: Helper for reporting search parsing errors\r
  35 In-Reply-To: <1341354059-29396-5-git-send-email-amdragon@mit.edu>\r
  36 References: <1341354059-29396-1-git-send-email-amdragon@mit.edu>\r
  37         <1341354059-29396-5-git-send-email-amdragon@mit.edu>\r
  38 User-Agent: Notmuch/0.13.2+70~gb6a56e7 (http://notmuchmail.org) Emacs/23.4.1\r
  39         (x86_64-pc-linux-gnu)\r
  40 Date: Wed, 04 Jul 2012 09:41:56 +0100\r
  41 Message-ID: <87ehorhq97.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: 94.192.233.223\r
  45 X-QM-SPAM-Info: Sender has good ham record.  :)\r
  46 X-QM-Body-MD5: 63fbf2399b833a59425a4ac623a80674 (of first 20000 bytes)\r
  47 X-SpamAssassin-Score: -1.8\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 -1.8 points.\r
  53         Summary of the scoring: \r
  54         * -2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at http://www.dnswl.org/,\r
  55         *      medium trust\r
  56         *      [138.37.6.40 listed in list.dnswl.org]\r
  57         * 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\r
  58         provider *      (markwalters1009[at]gmail.com)\r
  59         * -0.0 T_RP_MATCHES_RCVD Envelope sender domain matches handover relay\r
  60         *      domain\r
  61         *  0.5 AWL AWL: From: address is in the auto white-list\r
  62 X-QM-Scan-Virus: ClamAV says the message is clean\r
  63 Cc: tomi.ollila@iki.fi\r
  64 X-BeenThere: notmuch@notmuchmail.org\r
  65 X-Mailman-Version: 2.1.13\r
  66 Precedence: list\r
  67 List-Id: "Use and development of the notmuch mail system."\r
  68         <notmuch.notmuchmail.org>\r
  69 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  70         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  71 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  72 List-Post: <mailto:notmuch@notmuchmail.org>\r
  73 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  74 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  75         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  76 X-List-Received-Date: Wed, 04 Jul 2012 08:42:02 -0000\r
  77 \r
  78 On Tue, 03 Jul 2012, Austin Clements <amdragon@MIT.EDU> wrote:\r
  79 > This removes the last bit of direct output from the parsing function.\r
  80 > With the parser now responsible solely for parsing, we can swap it out\r
  81 > for another parser.\r
  82 > ---\r
  83 >  emacs/notmuch.el |   10 +++++++++-\r
  84 >  1 file changed, 9 insertions(+), 1 deletion(-)\r
  85 >\r
  86 > diff --git a/emacs/notmuch.el b/emacs/notmuch.el\r
  87 > index 4a6490a..a073367 100644\r
  88 > --- a/emacs/notmuch.el\r
  89 > +++ b/emacs/notmuch.el\r
  90 > @@ -745,6 +745,13 @@ non-authors is found, assume that all of the authors match."\r
  91 >       (setq notmuch-search-target-thread "found")\r
  92 >       (goto-char beg)))))\r
  93 >  \r
  94 > +(defun notmuch-search-show-error (string &rest objects)\r
  95 > +  (save-excursion\r
  96 > +    (goto-char (point-max))\r
  97 > +    (insert "Error: Unexpected output from notmuch search:\n")\r
  98 > +    (insert (apply #'format string objects))\r
  99 > +    (insert "\n")))\r
 100 > +\r
 101 >  (defun notmuch-search-process-filter (proc string)\r
 102 >    "Process and filter the output of \"notmuch search\""\r
 103 >    (let ((buffer (process-buffer proc)))\r
 104 > @@ -767,7 +774,8 @@ non-authors is found, assume that all of the authors match."\r
 105 >                          (tags (match-string 6 string))\r
 106 >                          (tag-list (if tags (save-match-data (split-string tags)))))\r
 107 >                     (if (/= (match-beginning 1) line)\r
 108 > -                       (insert (concat "Error: Unexpected output from notmuch search:\n" (substring string line (match-beginning 1)) "\n")))\r
 109 > +                       (notmuch-search-show-error\r
 110 > +                        (substring string line (match-beginning 1))))\r
 111 >                     (notmuch-search-show-result thread-id date count authors subject tag-list)\r
 112 >                     (set 'line (match-end 0)))\r
 113 >                 (set 'more nil)\r
 114 \r
 115 Ok so I should have read this patch before replying to the previous\r
 116 one. This fixes my one trivial concern with Patch 3 and looks good in\r
 117 itself.\r
 118 \r
 119 I have read Patches 1-4 and they all look fine and make things cleaner\r
 120 and all tests pass. As Austin says, they stand on their own and can go\r
 121 in without the rest of the series.\r
 122 \r
 123 On to patches 5-8!\r
 124 \r
 125 Best wishes\r
 126 \r
 127 Mark\r