07/627ce9cc5b3c2fc3d5b5a167d110408bb0bbda

   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 CB2FB431FBC\r
   6         for <notmuch@notmuchmail.org>; Thu, 17 Oct 2013 01:07:58 -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: 0.148\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.148 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_BL_SPAMCOP_NET=1.246,\r
  14         RCVD_IN_DNSWL_MED=-2.3] 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 1fb6xH1ItOsY for <notmuch@notmuchmail.org>;\r
  18         Thu, 17 Oct 2013 01:07:54 -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 D5601431FAE\r
  23         for <notmuch@notmuchmail.org>; Thu, 17 Oct 2013 01:07:53 -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 1VWicV-0000BQ-K5; Thu, 17 Oct 2013 09:07:50 +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 1VWicV-00017l-1l; Thu, 17 Oct 2013 09:07:47 +0100\r
  32 From: Mark Walters <markwalters1009@gmail.com>\r
  33 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org\r
  34 Subject: Re: [PATCH 2/6] cli: sanitize tabs to spaces in notmuch search\r
  35 In-Reply-To:\r
  36  <af8381b86275c4be77cc8527a19d69efbd2e9423.1381948853.git.jani@nikula.org>\r
  37 References: <cover.1381948853.git.jani@nikula.org>\r
  38         <af8381b86275c4be77cc8527a19d69efbd2e9423.1381948853.git.jani@nikula.org>\r
  39 User-Agent: Notmuch/0.15.2+334~gafd5c6a (http://notmuchmail.org) Emacs/23.4.1\r
  40         (x86_64-pc-linux-gnu)\r
  41 Date: Thu, 17 Oct 2013 09:07:46 +0100\r
  42 Message-ID: <87ob6ob9rh.fsf@qmul.ac.uk>\r
  43 MIME-Version: 1.0\r
  44 Content-Type: text/plain; charset=us-ascii\r
  45 X-Sender-Host-Address: 93.97.24.31\r
  46 X-QM-SPAM-Info: Sender has good ham record.  :)\r
  47 X-QM-Body-MD5: a3d5a750b9c0cfb97c132851950f5200 (of first 20000 bytes)\r
  48 X-SpamAssassin-Score: 0.0\r
  49 X-SpamAssassin-SpamBar: /\r
  50 X-SpamAssassin-Report: The QM spam filters have analysed this message to\r
  51         determine if it is\r
  52         spam. We require at least 5.0 points to mark a message as spam.\r
  53         This message scored 0.0 points. Summary of the scoring: \r
  54         * 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\r
  55         provider *      (markwalters1009[at]gmail.com)\r
  56         *  0.0 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: Thu, 17 Oct 2013 08:07:58 -0000\r
  71 \r
  72 \r
  73 Hi\r
  74 \r
  75 I have looked at the whole series and broadly it looks good. However, I\r
  76 don't know this code so this is not a full review. I do have a few\r
  77 comments: some of these may be plain wrong in which case my apologies!\r
  78 \r
  79 On Wed, 16 Oct 2013, Jani Nikula <jani@nikula.org> wrote:\r
  80 \r
  81 > This is in preparation of switching to gmime header parsing, but\r
  82 > arguably converting tabs to spaces rather than question marks is the\r
  83 > right thing to do anyway.\r
  84 > ---\r
  85 \r
  86 I think it would be worth saying in the commit message that this is only\r
  87 for text summary output.\r
  88 \r
  89 Also why only tabs to spaces but \n to a '?'\r
  90 \r
  91 Best wishes\r
  92 \r
  93 Mark\r
  94 \r
  95 \r
  96 \r
  97 \r
  98 >  notmuch-search.c | 4 +++-\r
  99 >  1 file changed, 3 insertions(+), 1 deletion(-)\r
 100 >\r
 101 > diff --git a/notmuch-search.c b/notmuch-search.c\r
 102 > index d9d39ec..eab314f 100644\r
 103 > --- a/notmuch-search.c\r
 104 > +++ b/notmuch-search.c\r
 105 > @@ -40,7 +40,9 @@ sanitize_string (const void *ctx, const char *str)\r
 106 >      loop = out = talloc_strdup (ctx, str);\r
 107 >  \r
 108 >      for (; *loop; loop++) {\r
 109 > -     if ((unsigned char)(*loop) < 32)\r
 110 > +     if (*loop == '\t')\r
 111 > +         *loop = ' ';\r
 112 > +     else if ((unsigned char)(*loop) < 32)\r
 113 >           *loop = '?';\r
 114 >      }\r
 115 >      return out;\r
 116 > -- \r
 117 > 1.8.4.rc3\r
 118 >\r
 119 > _______________________________________________\r
 120 > notmuch mailing list\r
 121 > notmuch@notmuchmail.org\r
 122 > http://notmuchmail.org/mailman/listinfo/notmuch\r