1e/96940313d38c2cdb759e5cd2e501d14047bf15

   1 Return-Path: <aclements@csail.mit.edu>\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 6D8B4431FCB\r
   6         for <notmuch@notmuchmail.org>; Sat, 24 Jan 2015 09:29:52 -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.138\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.138 tagged_above=-999 required=5\r
  12         tests=[DNS_FROM_AHBL_RHSBL=2.438, RCVD_IN_DNSWL_MED=-2.3]\r
  13         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 fAZfQNgz2nyh for <notmuch@notmuchmail.org>;\r
  17         Sat, 24 Jan 2015 09:29:49 -0800 (PST)\r
  18 Received: from outgoing.csail.mit.edu (outgoing.csail.mit.edu [128.30.2.149])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 34AA0431FAE\r
  20         for <notmuch@notmuchmail.org>; Sat, 24 Jan 2015 09:29:49 -0800 (PST)\r
  21 Received: from [104.131.20.129] (helo=awakeningjr)\r
  22         by outgoing.csail.mit.edu with esmtpsa (TLS1.0:RSA_AES_128_CBC_SHA1:16)\r
  23         (Exim 4.72) (envelope-from <aclements@csail.mit.edu>)\r
  24         id 1YF4Wq-0004cb-Rq; Sat, 24 Jan 2015 12:29:48 -0500\r
  25 Received: from amthrax by awakeningjr with local (Exim 4.84)\r
  26         (envelope-from <aclements@csail.mit.edu>)\r
  27         id 1YF4Wq-0003bf-Ij; Sat, 24 Jan 2015 12:29:48 -0500\r
  28 From: Austin Clements <amdragon@MIT.EDU>\r
  29 To: Mark Walters <markwalters1009@gmail.com>, notmuch@notmuchmail.org\r
  30 Subject: Re: [PATCH 00/11] Improve charset and cid: handling\r
  31 In-Reply-To: <87tx9gl5yn.fsf@qmul.ac.uk>\r
  32 References: <1398105468-14317-1-git-send-email-amdragon@mit.edu>\r
  33         <87tx9gl5yn.fsf@qmul.ac.uk>\r
  34 User-Agent: Notmuch/0.18.1+86~gef5e66a (http://notmuchmail.org) Emacs/24.4.1\r
  35         (x86_64-pc-linux-gnu)\r
  36 Date: Sat, 24 Jan 2015 12:29:48 -0500\r
  37 Message-ID: <87y4ost7cj.fsf@csail.mit.edu>\r
  38 MIME-Version: 1.0\r
  39 Content-Type: text/plain\r
  40 X-BeenThere: notmuch@notmuchmail.org\r
  41 X-Mailman-Version: 2.1.13\r
  42 Precedence: list\r
  43 List-Id: "Use and development of the notmuch mail system."\r
  44         <notmuch.notmuchmail.org>\r
  45 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  46         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  47 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  48 List-Post: <mailto:notmuch@notmuchmail.org>\r
  49 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  50 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  51         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  52 X-List-Received-Date: Sat, 24 Jan 2015 17:29:52 -0000\r
  53 \r
  54 I added declare-functions for both of these, which should take care of\r
  55 the Emacs 23 warnings and be more robust on Emacs 24.  We can't reach\r
  56 the function that calls these unless shr is actually available, but the\r
  57 byte compiler doesn't know that.\r
  58 \r
  59 On Sat, 26 Apr 2014, Mark Walters <markwalters1009@gmail.com> wrote:\r
  60 > Aside from the minor comments I mentioned in previous emails and one\r
  61 > more comment below this looks good. \r
  62 >\r
  63 > The extra comment is that on emacs23 I get the following when compiling:\r
  64 >\r
  65 > In end of data:\r
  66 > notmuch-show.el:2188:1:Warning: the following functions are not known to be\r
  67 >     defined: libxml-parse-html-region, shr-insert-document\r
  68 >\r
  69 > Finally, I have not really tested it as I mainly use emacs23\r
  70 >\r
  71 > Best wishes\r
  72 >\r
  73 > Mark\r
  74 >\r
  75 >\r
  76 >\r
  77 >\r
  78 > On Mon, 21 Apr 2014, Austin Clements <amdragon@MIT.EDU> wrote:\r
  79 >> I set out to quickly add support for cid: links in the shr renderer\r
  80 >> and wound up making our charset handling more robust and rewriting our\r
  81 >> content-ID handling.  The test introduced in patch 2 passes in all but\r
  82 >> one really obscure case, but only because of many unwritten and\r
  83 >> potentially fragile assumptions that Emacs and the CLI make about each\r
  84 >> other.\r
  85 >>\r
  86 >> The first three patches could reasonably go in to 0.18.  The rest of\r
  87 >> this series is certainly post-0.18, but I didn't want to lose track of\r
  88 >> it.\r
  89 >>\r
  90 >> This series comes in three stages.  Each depends on the earlier ones,\r
  91 >> but each prefix makes sense on its own and could be pushed without the\r
  92 >> later stages.\r
  93 >>\r
  94 >> Patch 1 is a simple clean up patch.\r
  95 >>\r
  96 >> Patches 2 through 7 robust-ify our charset handling in Emacs, mostly\r
  97 >> by splitting the broken `notmuch-get-bodypart-content' API into\r
  98 >> `notmuch-get-bodypart-binary' and `notmuch-get-bodypart-text' so a\r
  99 >> caller can explicitly convey their requirements.\r
 100 >>\r
 101 >> The remaining patches improve our content-ID handling and add support\r
 102 >> for cid: links for shr.\r
 103 >>\r
 104 >> _______________________________________________\r
 105 >> notmuch mailing list\r
 106 >> notmuch@notmuchmail.org\r
 107 >> http://notmuchmail.org/mailman/listinfo/notmuch\r