f8/4b4e9c852b36e32a89d82a4f921d914062530b

   1 Return-Path: <bremner@unb.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 A9EF5431FB6\r
   6         for <notmuch@notmuchmail.org>; Wed, 21 Nov 2012 19:51:35 -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\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         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 zG3mPi50TSn6 for <notmuch@notmuchmail.org>;\r
  16         Wed, 21 Nov 2012 19:51:35 -0800 (PST)\r
  17 Received: from tesseract.cs.unb.ca (tesseract.cs.unb.ca [131.202.240.238])\r
  18         (using TLSv1 with cipher AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 3EE5D431FAF\r
  21         for <notmuch@notmuchmail.org>; Wed, 21 Nov 2012 19:51:35 -0800 (PST)\r
  22 Received: from fctnnbsc30w-156034089108.dhcp-dynamic.fibreop.nb.bellaliant.net\r
  23         ([156.34.89.108] helo=zancas.localnet)\r
  24         by tesseract.cs.unb.ca with esmtpsa\r
  25         (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.72)\r
  26         (envelope-from <bremner@unb.ca>)\r
  27         id 1TbNp8-00032f-94; Wed, 21 Nov 2012 23:51:34 -0400\r
  28 Received: from bremner by zancas.localnet with local (Exim 4.80)\r
  29         (envelope-from <bremner@unb.ca>)\r
  30         id 1TbNp2-0000zn-Q1; Wed, 21 Nov 2012 23:51:28 -0400\r
  31 From: David Bremner <david@tethera.net>\r
  32 To: Tomi Ollila <tomi.ollila@iki.fi>, notmuch@notmuchmail.org\r
  33 Subject: Re: [PATCH v3 2/2] notmuch-show.el: handle the case where\r
  34         icalendar-import-buffer returns nil\r
  35 In-Reply-To: <1352376480-29961-3-git-send-email-tomi.ollila@iki.fi>\r
  36 References: <1352376480-29961-1-git-send-email-tomi.ollila@iki.fi>\r
  37         <1352376480-29961-3-git-send-email-tomi.ollila@iki.fi>\r
  38 User-Agent: Notmuch/0.14+75~g984212d (http://notmuchmail.org) Emacs/24.1.1\r
  39         (x86_64-pc-linux-gnu)\r
  40 Date: Wed, 21 Nov 2012 23:51:28 -0400\r
  41 Message-ID: <87sj82wb0f.fsf@zancas.localnet>\r
  42 MIME-Version: 1.0\r
  43 Content-Type: text/plain\r
  44 X-Spam_bar: -\r
  45 Cc: tomi.ollila@iki.fi\r
  46 X-BeenThere: notmuch@notmuchmail.org\r
  47 X-Mailman-Version: 2.1.13\r
  48 Precedence: list\r
  49 List-Id: "Use and development of the notmuch mail system."\r
  50         <notmuch.notmuchmail.org>\r
  51 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  52         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  53 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  54 List-Post: <mailto:notmuch@notmuchmail.org>\r
  55 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  56 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  57         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  58 X-List-Received-Date: Thu, 22 Nov 2012 03:51:35 -0000\r
  59 \r
  60 Tomi Ollila <tomi.ollila@iki.fi> writes:\r
  61 \r
  62 > icalendar-import-buffer can fail by an error signal (which have been\r
  63 > witnessed) but according to its docstring it can also return nil\r
  64 > when failing (it returns t when succeeding).\r
  65 >\r
  66 > Now that the error is caught by the caller of notmuch-show-inset-part-*\r
  67 > functions in case icalendar-import-buffer returns nil an explicit\r
  68 > error is signaled and unwind-protect takes care of deleting the\r
  69 > temporary file (just in case, it is usually not written to the fs yet).\r
  70 \r
  71 This one looks OK to me too, although the API that makes it necessary\r
  72 out to be taken out back...\r
  73 \r
  74 d\r