Re: [PATCH] add has: query prefix to search for specific properties
[notmuch-archives.git] / ed / 2d1eb91903b80a89ba568715b94539a3d2a202
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 1662B431FC7\r
6         for <notmuch@notmuchmail.org>; Tue, 21 May 2013 08:26:25 -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 TZ0vCd2L3K+a for <notmuch@notmuchmail.org>;\r
17         Tue, 21 May 2013 08:26:17 -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 E7764431FB6\r
22         for <notmuch@notmuchmail.org>; Tue, 21 May 2013 08:26:16 -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 1UeoS6-0000EQ-MD; Tue, 21 May 2013 16:26:15 +0100\r
27 Received: from 93-97-24-31.zone5.bethere.co.uk ([93.97.24.31] helo=localhost)\r
28         by smtp.qmul.ac.uk with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.71)\r
29         (envelope-from <m.walters@qmul.ac.uk>)\r
30         id 1UeoS6-0006B4-EK; Tue, 21 May 2013 16:26:14 +0100\r
31 From: Mark Walters <markwalters1009@gmail.com>\r
32 To: Austin Clements <amdragon@MIT.EDU>, notmuch@notmuchmail.org\r
33 Subject: Re: [PATCH 0/5] Make Emacs search use sexp format\r
34 In-Reply-To: <1368851472-5382-1-git-send-email-amdragon@mit.edu>\r
35 References: <1368851472-5382-1-git-send-email-amdragon@mit.edu>\r
36 User-Agent: Notmuch/0.14+255~gff3cc55 (http://notmuchmail.org) Emacs/23.4.1\r
37         (x86_64-pc-linux-gnu)\r
38 Date: Tue, 21 May 2013 16:26:13 +0100\r
39 Message-ID: <87d2sk2w2y.fsf@qmul.ac.uk>\r
40 MIME-Version: 1.0\r
41 Content-Type: text/plain; charset=us-ascii\r
42 X-Sender-Host-Address: 93.97.24.31\r
43 X-QM-SPAM-Info: Sender has good ham record.  :)\r
44 X-QM-Body-MD5: a1f00f841d29b236a504cfc05f86d630 (of first 20000 bytes)\r
45 X-SpamAssassin-Score: -0.2\r
46 X-SpamAssassin-SpamBar: /\r
47 X-SpamAssassin-Report: The QM spam filters have analysed this message to\r
48         determine if it is\r
49         spam. We require at least 5.0 points to mark a message as spam.\r
50         This message scored -0.2 points.\r
51         Summary of the scoring: \r
52         * 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\r
53         provider *      (markwalters1009[at]gmail.com)\r
54         * -0.2 AWL AWL: From: address is in the auto white-list\r
55 X-QM-Scan-Virus: ClamAV says the message is clean\r
56 X-BeenThere: notmuch@notmuchmail.org\r
57 X-Mailman-Version: 2.1.13\r
58 Precedence: list\r
59 List-Id: "Use and development of the notmuch mail system."\r
60         <notmuch.notmuchmail.org>\r
61 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
62         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
63 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
64 List-Post: <mailto:notmuch@notmuchmail.org>\r
65 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
66 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
67         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
68 X-List-Received-Date: Tue, 21 May 2013 15:26:25 -0000\r
69 \r
70 \r
71 This whole series looks good to me (modulo the minor comments I have\r
72 made). The tests all pass. Additionally it was very simple to modify\r
73 notmuch-pick to use the new parser and that also seemed to work.\r
74 \r
75 Incidentally it seems that notmuch-show still uses the JSON format so we\r
76 may want to switch that over too.\r
77 \r
78 Best wishes\r
79 \r
80 Mark\r
81 \r
82 \r
83 On Sat, 18 May 2013, Austin Clements <amdragon@MIT.EDU> wrote:\r
84 > This series implements an incremental S-expression parser and switches\r
85 > search over to it.  To simplify things, it also implements better\r
86 > handing of stderr for asynchronous processes so we don't have to\r
87 > handle errors embedded in the S-expression stream.\r