30/f7df1ce0d14fe6fac17c0e20f51c2d7a8ed5dc

   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 F0533429E20\r
   6         for <notmuch@notmuchmail.org>; Thu, 10 Mar 2011 12:01:48 -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.99\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.99 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, 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 0-ITIDM0Q9iJ; Thu, 10 Mar 2011 12:01:48 -0800 (PST)\r
  16 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  17         by olra.theworths.org (Postfix) with ESMTP id 813E3431FB5;\r
  18         Thu, 10 Mar 2011 12:01:48 -0800 (PST)\r
  19 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  20         id 1C04825401B; Thu, 10 Mar 2011 12:01:48 -0800 (PST)\r
  21 From: Carl Worth <cworth@cworth.org>\r
  22 To: Austin Clements <amdragon@mit.edu>, Karel Zak <kzak@redhat.com>\r
  23 Subject: Re: [PATCH] new: read db_files and db_subdirs if mtime changed\r
  24 In-Reply-To: <AANLkTi=PfbB9Fv6euxuzsjOkSsj-EqQxTFuz9ucfzJHi@mail.gmail.com>\r
  25 References: <1296855871-15702-1-git-send-email-kzak@redhat.com>\r
  26         <AANLkTi=PfbB9Fv6euxuzsjOkSsj-EqQxTFuz9ucfzJHi@mail.gmail.com>\r
  27 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.2.1\r
  28         (i486-pc-linux-gnu)\r
  29 Date: Thu, 10 Mar 2011 12:01:47 -0800\r
  30 Message-ID: <87tyfa9238.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 Cc: notmuch@notmuchmail.org\r
  35 X-BeenThere: notmuch@notmuchmail.org\r
  36 X-Mailman-Version: 2.1.13\r
  37 Precedence: list\r
  38 List-Id: "Use and development of the notmuch mail system."\r
  39         <notmuch.notmuchmail.org>\r
  40 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  41         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  42 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  43 List-Post: <mailto:notmuch@notmuchmail.org>\r
  44 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  45 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  46         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  47 X-List-Received-Date: Thu, 10 Mar 2011 20:01:49 -0000\r
  48 \r
  49 --=-=-=\r
  50 Content-Transfer-Encoding: quoted-printable\r
  51 \r
  52 On Fri,  4 Feb 2011 22:44:31 +0100, Karel Zak <kzak@redhat.com> wrote:\r
  53 > The db_files and db_subdirs are unnecessary for unchanged directories.\r
  54 ...\r
  55 > old version: real    0m0.053s\r
  56 > new version: real    0m0.032s\r
  57 \r
  58 Thanks Karel! What a lovely optimization.\r
  59 \r
  60 On Sun, 27 Feb 2011 03:45:05 -0500, Austin Clements <amdragon@mit.edu> wrot=\r
  61 e:\r
  62 > Looks good (faster than, but provably equivalent to the original code!\r
  63 >  notmuch_directory_get_child_* are side-effect free,\r
  64 > db_files/db_subdirs aren't used between where they were set in the old\r
  65 > code and where they are set in the new code, and db_files/db_subdirs\r
  66 > are initialized to NULL when declared).\r
  67 \r
  68 And thank you, Austin for the careful review. This kind of thing is very\r
  69 helpful for me in reviewing patches. When there's a message like this on\r
  70 the mailing list it makes it very easy for me to trust the patch.\r
  71 \r
  72 I've now pushed the original patch with an updated commit message to\r
  73 include Austin's review comments.\r
  74 \r
  75 I also followed up with a subsequent commit that updates the comments\r
  76 for the add_files_recursive function to match the current\r
  77 implementation, (reflecting both this change and a previous change to a\r
  78 strict equality test for the mtime comparison).\r
  79 \r
  80 =2DCarl\r
  81 \r
  82 =2D-=20\r
  83 carl.d.worth@intel.com\r
  84 \r
  85 --=-=-=\r
  86 Content-Type: application/pgp-signature\r
  87 \r
  88 -----BEGIN PGP SIGNATURE-----\r
  89 Version: GnuPG v1.4.10 (GNU/Linux)\r
  90 \r
  91 iD8DBQFNeS4r6JDdNq8qSWgRArE3AJ4262SMdNGA83ipgIrVQALoe47AKgCfXLKI\r
  92 Xdb0k8/qEkm+rf4+XJmZfFA=\r
  93 =JxSx\r
  94 -----END PGP SIGNATURE-----\r
  95 --=-=-=--\r