98/1998e871f86be823407088365bfa965971fc40

   1 Return-Path: <dkg@fifthhorseman.net>\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 arlo.cworth.org (Postfix) with ESMTP id 5EA326DE0A9A\r
   6  for <notmuch@notmuchmail.org>; Fri, 11 Dec 2015 06:36:46 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.017\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.017 tagged_above=-999 required=5\r
  12  tests=[AWL=-0.017] autolearn=disabled\r
  13 Received: from arlo.cworth.org ([127.0.0.1])\r
  14  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15  with ESMTP id lVevpNoNwfwj for <notmuch@notmuchmail.org>;\r
  16  Fri, 11 Dec 2015 06:36:44 -0800 (PST)\r
  17 Received: from che.mayfirst.org (che.mayfirst.org [209.234.253.108])\r
  18  by arlo.cworth.org (Postfix) with ESMTP id 86DF46DE0B4B\r
  19  for <notmuch@notmuchmail.org>; Fri, 11 Dec 2015 06:36:44 -0800 (PST)\r
  20 Received: from fifthhorseman.net (unknown [38.109.115.130])\r
  21  by che.mayfirst.org (Postfix) with ESMTPSA id B004BF984;\r
  22  Fri, 11 Dec 2015 09:36:42 -0500 (EST)\r
  23 Received: by fifthhorseman.net (Postfix, from userid 1000)\r
  24  id B7E8F2006D; Fri, 11 Dec 2015 09:36:42 -0500 (EST)\r
  25 From: Daniel Kahn Gillmor <dkg@fifthhorseman.net>\r
  26 To: David Bremner <david@tethera.net>, Notmuch Mail <notmuch@notmuchmail.org>\r
  27 Subject: Re: [PATCH 2/9] Add a lazily-initialized crypto context to\r
  28  notmuch_database_t\r
  29 In-Reply-To: <871tatf5gm.fsf@zancas.localnet>\r
  30 References: <1449718786-28000-1-git-send-email-dkg@fifthhorseman.net>\r
  31  <1449718786-28000-3-git-send-email-dkg@fifthhorseman.net>\r
  32  <871tatf5gm.fsf@zancas.localnet>\r
  33 User-Agent: Notmuch/0.21 (http://notmuchmail.org) Emacs/24.5.1\r
  34  (x86_64-pc-linux-gnu)\r
  35 Date: Fri, 11 Dec 2015 09:36:42 -0500\r
  36 Message-ID: <8737v9hx1h.fsf@alice.fifthhorseman.net>\r
  37 MIME-Version: 1.0\r
  38 Content-Type: text/plain\r
  39 X-BeenThere: notmuch@notmuchmail.org\r
  40 X-Mailman-Version: 2.1.20\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: <https://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: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  50  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  51 X-List-Received-Date: Fri, 11 Dec 2015 14:36:46 -0000\r
  52 \r
  53 On Fri 2015-12-11 09:03:05 -0500, David Bremner wrote:\r
  54 > Daniel Kahn Gillmor <dkg@fifthhorseman.net> writes:\r
  55 >\r
  56 >> This is in large part a duplicate of parts of crypto.c, but that code\r
  57 >> is in the client (outside the library), and we don't want to entangle\r
  58 >> the libgmime API with the libnotmuch API.\r
  59 >>\r
  60 >> I welcome better proposals for how to share this code explicitly\r
  61 >> between the library and the client.\r
  62 >\r
  63 > Maybe I miss something obvious, but util/libutil.a is exactly there for\r
  64 > sharing code between the library and the client.\r
  65 \r
  66 You didn't miss anything obvious -- i did!  Thanks for pointing that\r
  67 out, i'll take a look at normalizing these bits for my second draft.\r
  68 \r
  69 > perhaps something like "gmime-extra.c" to go with {talloc,zlib}-extra.c\r
  70 \r
  71 right, sounds good.\r
  72 \r
  73 > I didn't look at the code yet, just the commentary.\r
  74 \r
  75 many thanks for the review, that's why i wrote the commentary :)\r
  76 \r
  77 After a couple nights of sleep, i have a proposal to fix one of the\r
  78 open questions too, which i'll follow up on shortly here.\r
  79 \r
  80      --dkg\r