cf/4361a136ad40bf9867d83bf589861ca97e0d5b

   1 Return-Path: <cworth@cworth.org>\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 0F61D431FD0\r
   6         for <notmuch@notmuchmail.org>; Fri,  3 Jun 2011 14:10:56 -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.01\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.01 tagged_above=-999 required=5\r
  12         tests=[T_MIME_NO_TEXT=0.01] 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 CPOdqE3VWSXp for <notmuch@notmuchmail.org>;\r
  16         Fri,  3 Jun 2011 14:10:55 -0700 (PDT)\r
  17 Received: from arlo.cworth.org (arlo.cworth.org [50.43.72.2])\r
  18         by olra.theworths.org (Postfix) with ESMTP id AA220431FB6\r
  19         for <notmuch@notmuchmail.org>; Fri,  3 Jun 2011 14:10:55 -0700 (PDT)\r
  20 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  21         by arlo.cworth.org (Postfix) with ESMTP id 635F429A5CC;\r
  22         Fri,  3 Jun 2011 14:10:54 -0700 (PDT)\r
  23 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  24         id 557B325417E; Fri,  3 Jun 2011 14:10:54 -0700 (PDT)\r
  25 From: Carl Worth <cworth@cworth.org>\r
  26 To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>,\r
  27         Notmuch Mail <notmuch@notmuchmail.org>\r
  28 Subject: Re: [PATCH 1/2] Make `notmuch-show-clean-address'\r
  29  parsing-error-proof.\r
  30 In-Reply-To: <1305208586-24689-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  31 References: <1305208586-24689-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  32 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.3.1\r
  33         (i486-pc-linux-gnu)\r
  34 Date: Fri, 03 Jun 2011 14:10:54 -0700\r
  35 Message-ID: <87ipsmlixt.fsf@yoom.home.cworth.org>\r
  36 MIME-Version: 1.0\r
  37 Content-Type: multipart/signed; boundary="=-=-=";\r
  38         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  39 X-BeenThere: notmuch@notmuchmail.org\r
  40 X-Mailman-Version: 2.1.13\r
  41 Precedence: list\r
  42 List-Id: "Use and development of the notmuch mail system."\r
  43         <notmuch.notmuchmail.org>\r
  44 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  46 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  47 List-Post: <mailto:notmuch@notmuchmail.org>\r
  48 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  49 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  51 X-List-Received-Date: Fri, 03 Jun 2011 21:10:56 -0000\r
  52 \r
  53 --=-=-=\r
  54 \r
  55 On Thu, 12 May 2011 17:56:25 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:\r
  56 > Mail-header-parse-address may fail for an invalid address.\r
  57 > Before the change, this would result in empty notmuch-show buffer\r
  58 > with an error message like: Scan error: "Unbalanced parentheses".\r
  59 > The patch wraps the function in condition-case and returns\r
  60 > unchanged address in case of error.\r
  61 \r
  62 Thanks for the patch, Dmitry. I've just pushed this.\r
  63 \r
  64 And thanks as well for the test case. I actually pushed the test-case\r
  65 commit before the bug-fix commit. I like to do things in that order so\r
  66 that I can:\r
  67 \r
  68         1. Apply the new test case\r
  69         2. Run "make test" and see the failure\r
  70         3. Apply the bug fix\r
  71         4. Run "make test" and see the test pass\r
  72 \r
  73 With a sequence like that, it's very easy for me to feel very\r
  74 comfortable committing the code.\r
  75 \r
  76 -Carl\r
  77 \r
  78 --=-=-=\r
  79 Content-Type: application/pgp-signature\r
  80 \r
  81 -----BEGIN PGP SIGNATURE-----\r
  82 Version: GnuPG v1.4.11 (GNU/Linux)\r
  83 \r
  84 iEYEARECAAYFAk3pTd4ACgkQ6JDdNq8qSWjuxQCgj6A73s9t7h306Nu65U3wmSft\r
  85 Rw0AmwWjSxMdzCCACVV9yfjihZJutbqF\r
  86 =d3/9\r
  87 -----END PGP SIGNATURE-----\r
  88 --=-=-=--\r