a4/1957c81a54137700d74fae26c37aefdf5e8d3d

   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 9C7BF418C36\r
   6         for <notmuch@notmuchmail.org>; Tue, 20 Apr 2010 09:25:37 -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: -2.89\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01]\r
  13         autolearn=ham\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 rh+A065g3-E2; Tue, 20 Apr 2010 09:25:36 -0700 (PDT)\r
  17 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  18         by olra.theworths.org (Postfix) with ESMTP id D4B42418C34;\r
  19         Tue, 20 Apr 2010 09:25:36 -0700 (PDT)\r
  20 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  21         id 8923725400A; Tue, 20 Apr 2010 09:25:36 -0700 (PDT)\r
  22 From: Carl Worth <cworth@cworth.org>\r
  23 To: David Edmondson <dme@dme.org>, notmuch <notmuch@notmuchmail.org>\r
  24 Subject: Re: pull request\r
  25 In-Reply-To: <8739yq8zix.fsf@ut.hh.sledj.net>\r
  26 References: <87sk722sfq.fsf@ut.hh.sledj.net> <87eiibq22s.fsf@ut.hh.sledj.net>\r
  27         <87ljcj2u4z.fsf@yoom.home.cworth.org>\r
  28         <8739yq8zix.fsf@ut.hh.sledj.net>\r
  29 Date: Tue, 20 Apr 2010 09:25:36 -0700\r
  30 Message-ID: <877ho2w0ov.fsf@yoom.home.cworth.org>\r
  31 MIME-Version: 1.0\r
  32 Content-Type: multipart/signed; boundary="=-=-=";\r
  33         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  34 X-BeenThere: notmuch@notmuchmail.org\r
  35 X-Mailman-Version: 2.1.13\r
  36 Precedence: list\r
  37 List-Id: "Use and development of the notmuch mail system."\r
  38         <notmuch.notmuchmail.org>\r
  39 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  40         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  41 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  42 List-Post: <mailto:notmuch@notmuchmail.org>\r
  43 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  44 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  46 X-List-Received-Date: Tue, 20 Apr 2010 16:25:37 -0000\r
  47 \r
  48 --=-=-=\r
  49 \r
  50 On Tue, 20 Apr 2010 06:27:02 +0100, David Edmondson <dme@dme.org> wrote:\r
  51 > The second chunk was intended to cover a similar case (len == 0), but\r
  52 > becomes unnecessary after the first chunk. At least, that's what I\r
  53 > convinced myself after the conversation with Anthony Towns\r
  54 > (id:h2y87b3a4191004060117v5421db8ejbe3030d0626e7440@mail.gmail.com).\r
  55 \r
  56 Thanks for the clarification.\r
  57 \r
  58 And I really appreciated seeing a reference to the original discussion\r
  59 that led to this patch. You'll notice there that Anthony's proposed\r
  60 commit had the same patch content that you had in your tree, but with a\r
  61 more detailed commit message, ("and always return a newly talloced\r
  62 array").\r
  63 \r
  64 That was exactly the kind of explanation I was looking for but couldn't\r
  65 find in the commit I first reviewed. It wasn't really a question of\r
  66 whether the code was correct. The problem was that there was a code\r
  67 change that wasn't described in the commit message. I don't want that\r
  68 even if the change is correct.\r
  69 \r
  70 Anyway, thanks AJ and David.\r
  71 \r
  72 I've now pushed my version of these changes up through this point. I'm\r
  73 currently working on the make-emacs-use-JSON patch, (it's got some\r
  74 confusion about "body visible" vs. "message visible" that I want to fix\r
  75 before pushing).\r
  76 \r
  77 -Carl\r
  78 \r
  79 --=-=-=\r
  80 Content-Type: application/pgp-signature\r
  81 \r
  82 -----BEGIN PGP SIGNATURE-----\r
  83 Version: GnuPG v1.4.10 (GNU/Linux)\r
  84 \r
  85 iD8DBQFLzdWA6JDdNq8qSWgRAsHXAJoCbykIYcx3hPiytv+1HbgD+sBcoQCePU52\r
  86 9i6ngmNfd/RBD93tHrFYk/8=\r
  87 =dgoA\r
  88 -----END PGP SIGNATURE-----\r
  89 --=-=-=--\r