Re: [PATCH 0/4] Allow specifying alternate names for addresses in other_email
[notmuch-archives.git] / 2d / 947814bdaaeb67b6b774b1525222bb5fde2224
1 Return-Path: <awg@xvx.ca>\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 54323431FD0\r
6         for <notmuch@notmuchmail.org>; Mon,  9 Jan 2012 17:59:49 -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: -0.7\r
10 X-Spam-Level: \r
11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
12         tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
13 Received: from olra.theworths.org ([127.0.0.1])\r
14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
15         with ESMTP id 0xtiUmtHIcgk for <notmuch@notmuchmail.org>;\r
16         Mon,  9 Jan 2012 17:59:49 -0800 (PST)\r
17 Received: from mail-bk0-f53.google.com (mail-bk0-f53.google.com\r
18         [209.85.214.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
19         (No client certificate requested)\r
20         by olra.theworths.org (Postfix) with ESMTPS id BFCA2431FB6\r
21         for <notmuch@notmuchmail.org>; Mon,  9 Jan 2012 17:59:48 -0800 (PST)\r
22 Received: by bkat2 with SMTP id t2so96988bka.26\r
23         for <notmuch@notmuchmail.org>; Mon, 09 Jan 2012 17:59:47 -0800 (PST)\r
24 MIME-Version: 1.0\r
25 Received: by 10.204.133.207 with SMTP id g15mr7926541bkt.17.1326160787288;\r
26         Mon, 09 Jan 2012 17:59:47 -0800 (PST)\r
27 Sender: awg@xvx.ca\r
28 Received: by 10.205.33.73 with HTTP; Mon, 9 Jan 2012 17:59:47 -0800 (PST)\r
29 X-Originating-IP: [12.50.241.130]\r
30 In-Reply-To: <m2ty45is1j.fsf@gmail.com>\r
31 References: <1326009162-19524-1-git-send-email-awg+notmuch@xvx.ca>\r
32         <1326009162-19524-5-git-send-email-awg+notmuch@xvx.ca>\r
33         <m2ty45is1j.fsf@gmail.com>\r
34 Date: Mon, 9 Jan 2012 18:59:47 -0700\r
35 X-Google-Sender-Auth: GKHrDdRRNkVViLiBIWg3KAtSaN8\r
36 Message-ID:\r
37  <CAMoJFUu=vNcud1HpSDE9tapd8XA+HXQTD9z3_zhH-zA3EqMXqQ@mail.gmail.com>\r
38 Subject: Re: [PATCH 4/4] emacs: Use the new JSON reply format.\r
39 From: Adam Wolfe Gordon <awg+notmuch@xvx.ca>\r
40 To: Aaron Ecay <aaronecay@gmail.com>\r
41 Content-Type: text/plain; charset=windows-1252\r
42 Content-Transfer-Encoding: quoted-printable\r
43 Cc: notmuch@notmuchmail.org\r
44 X-BeenThere: notmuch@notmuchmail.org\r
45 X-Mailman-Version: 2.1.13\r
46 Precedence: list\r
47 List-Id: "Use and development of the notmuch mail system."\r
48         <notmuch.notmuchmail.org>\r
49 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
50         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
51 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
52 List-Post: <mailto:notmuch@notmuchmail.org>\r
53 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
54 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
55         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
56 X-List-Received-Date: Tue, 10 Jan 2012 01:59:49 -0000\r
57 \r
58 On Sun, Jan 8, 2012 at 18:27, Aaron Ecay <aaronecay@gmail.com> wrote:\r
59 >> +(defun w3m-region (start end)) ;; From `w3m.el'.\r
60 >\r
61 > What is the purpose of the above line? =A0If it is to make the compiler\r
62 > aware of the function, you should use =91declare-function=92 instead. =A0=\r
63 Defun\r
64 > will erase the original definition of the w3m-region function.\r
65 \r
66 Indeed, it's to make the compiler aware of the function.  I followed\r
67 the example of notmuch-show.el (defvar w3m-current-buffer, line 391),\r
68 but it sounds like declare-function is a better way to accomplish\r
69 this.  I haven't written a whole lot of emacs lisp, so thanks for the\r
70 tip.\r
71 \r
72 Given David's comments about requiring w3m instead it might be moot,\r
73 but if not I'll make this change for the next version.\r