d6/2b3def9eb4fd9e8d71e2ce15139aa7cc71526e

   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 AC8AF4196F0\r
   6         for <notmuch@notmuchmail.org>; Mon, 19 Apr 2010 11:07:48 -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 mWBVPtlrXV+C; Mon, 19 Apr 2010 11:07:46 -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 6F4DD431FC1;\r
  19         Mon, 19 Apr 2010 11:07:46 -0700 (PDT)\r
  20 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  21         id 14286568DE4; Mon, 19 Apr 2010 11:07:46 -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: <87eiibq22s.fsf@ut.hh.sledj.net>\r
  26 References: <87sk722sfq.fsf@ut.hh.sledj.net> <87eiibq22s.fsf@ut.hh.sledj.net>\r
  27 Date: Mon, 19 Apr 2010 11:07:40 -0700\r
  28 Message-ID: <87ljcj2u4z.fsf@yoom.home.cworth.org>\r
  29 MIME-Version: 1.0\r
  30 Content-Type: multipart/signed; boundary="=-=-=";\r
  31         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  32 X-BeenThere: notmuch@notmuchmail.org\r
  33 X-Mailman-Version: 2.1.13\r
  34 Precedence: list\r
  35 List-Id: "Use and development of the notmuch mail system."\r
  36         <notmuch.notmuchmail.org>\r
  37 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  38         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  39 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  40 List-Post: <mailto:notmuch@notmuchmail.org>\r
  41 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  42 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  43         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  44 X-List-Received-Date: Mon, 19 Apr 2010 18:07:48 -0000\r
  45 \r
  46 --=-=-=\r
  47 Content-Transfer-Encoding: quoted-printable\r
  48 \r
  49 On Mon, 19 Apr 2010 09:27:39 +0100, David Edmondson <dme@dme.org> wrote:\r
  50 > This is the same set rebased onto 0.2.\r
  51 \r
  52 Thanks for these, David!\r
  53 \r
  54 >  commit 8586a86b9dd4ed2406a2fbda6c08bdc6a598cfd8\r
  55 >     debian: git should ignore packaging intermediate files\r
  56 \r
  57 I committed an alternate version of this, (with a new debian/.gitignore\r
  58 file). I used more wildcarding too. And I couldn't actually find how to\r
  59 make all these files appear. If you need more ignores, just let me know\r
  60 (and let me know what commands trigger those, since I'm curious).\r
  61 \r
  62 >  commit f5de1bb5b4216a1950f6aa5f471c9964e4d0e521\r
  63 >     emacs/notmuch-show.el: Avoid passing unintended format strings to\r
  64 >     `message'\r
  65 \r
  66 Thanks. I've committed this.\r
  67 \r
  68 > commit 4159baa2166a3410663adc200f91145edf8e0e13\r
  69 >=20\r
  70 >     emacs/notmuch.el: Enable `hl-line-mode' in `notmuch-search-mode'\r
  71 \r
  72 I've got some misgivings about this one. First, notmuch-search-hook is\r
  73 a hook for the user to manipulate, while the hl-line-mode functionality\r
  74 is something that should be on by default. That is, if the user happens\r
  75 to set the search-hook then the hl-line-mode shouldn't magically\r
  76 disappear.\r
  77 \r
  78 Meanwhile, I *am* getting hl-line-mode in notmuch-search-mode\r
  79 already. I'm not sure where that's coming from. (I also don't know\r
  80 what's up with the current "options '(hl-line-mode)" which doesn't seem\r
  81 right.)\r
  82 \r
  83 > commit 53c4e64943d09b07e75c9258fc9f954c87a490d6\r
  84 >     json: Avoid calling strlen(NULL)\r
  85 \r
  86 That commit message describes this hunk of the patch:\r
  87 \r
  88 > +    if (str =3D=3D NULL)\r
  89 > +       str =3D "";\r
  90 > +\r
  91 >     return (json_quote_chararray (ctx, str, strlen (str)));\r
  92 \r
  93 But this other hunk looks independent. What's going on here?\r
  94 \r
  95 > -    if (len =3D=3D 0)\r
  96 > -       return (char *)"\"\"";\r
  97 > -\r
  98 \r
  99 So I haven't committed this piece yet.\r
 100 \r
 101 That brings me up to the big JSON rewrite, which I'll start testing and\r
 102 review in a separate reply.\r
 103 \r
 104 =2DCarl\r
 105 \r
 106 --=-=-=\r
 107 Content-Type: application/pgp-signature\r
 108 \r
 109 -----BEGIN PGP SIGNATURE-----\r
 110 Version: GnuPG v1.4.10 (GNU/Linux)\r
 111 \r
 112 iD8DBQFLzJvs6JDdNq8qSWgRAkb8AJ9p6PYKZ8K/Zw2RJGP3kMtw6xlA4wCfcyes\r
 113 3NP01jDOLRz36fV80gqOPwY=\r
 114 =xHBO\r
 115 -----END PGP SIGNATURE-----\r
 116 --=-=-=--\r