15/03ab08050286ed7fb984b40978a141cb355765

   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 0DE454196F2\r
   6         for <notmuch@notmuchmail.org>; Wed, 14 Apr 2010 07:17:06 -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: -1.841\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-1.841 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, DATE_IN_PAST_12_24=1.049,\r
  13         T_MIME_NO_TEXT=0.01] autolearn=no\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 NJkVeKFnafDE; Wed, 14 Apr 2010 07:17:05 -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 671E4431FC1;\r
  19         Wed, 14 Apr 2010 07:17:05 -0700 (PDT)\r
  20 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  21         id AEB6B5883A6; Tue, 13 Apr 2010 11:43:49 -0700 (PDT)\r
  22 From: Carl Worth <cworth@cworth.org>\r
  23 To: Michal Sojka <sojkam1@fel.cvut.cz>, notmuch@notmuchmail.org\r
  24 Subject: Re: [PATCH 0/4] Mailstore abstraction v4\r
  25 In-Reply-To: <1270737766-29237-1-git-send-email-sojkam1@fel.cvut.cz>\r
  26 References: <1270737766-29237-1-git-send-email-sojkam1@fel.cvut.cz>\r
  27 Date: Tue, 13 Apr 2010 11:43:49 -0700\r
  28 Message-ID: <874ojfkxay.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: Wed, 14 Apr 2010 14:17:06 -0000\r
  45 \r
  46 --=-=-=\r
  47 \r
  48 On Thu,  8 Apr 2010 16:42:42 +0200, Michal Sojka <sojkam1@fel.cvut.cz> wrote:\r
  49 > My biggest question relates to the first patch, which does an\r
  50 > incompatible change to libnotmuch API. After reading RELEASING file, I\r
  51 > found that this change is probably not what Carl wants to merge (and I\r
  52 > understand that) so I'd like to get some feedback on my suggestion in\r
  53 > that patch.\r
  54 \r
  55 [I composed this message a while ago, but failed to get it through the\r
  56 mailing-list moderation until now. It may be largely irrelevant in light\r
  57 of my more recent review of the mailstore abstraction, but here it is.]\r
  58 \r
  59 I haven't looked closely at the implementation here, but a quick glance\r
  60 at the API/ABI change suggests an easy answer:\r
  61 \r
  62 Why not just add a new function that accepts the mailstore type, and\r
  63 preserve the original function, (which would then simply call the new\r
  64 function with an argument of "files" for the mailstore type).\r
  65 \r
  66 -Carl\r
  67 \r
  68 --=-=-=\r
  69 Content-Type: application/pgp-signature\r
  70 \r
  71 -----BEGIN PGP SIGNATURE-----\r
  72 Version: GnuPG v1.4.10 (GNU/Linux)\r
  73 \r
  74 iD8DBQFLxLtl6JDdNq8qSWgRAq/HAKCpoJ0ZmZyX+nfCBP+ac4c4rCjQBwCcDgCB\r
  75 WVWqSX84e7kmKric4oh61bw=\r
  76 =SfO8\r
  77 -----END PGP SIGNATURE-----\r
  78 --=-=-=--\r