61/871780a9e8a0a74220fcb60a47b74ce1668cec

   1 Return-Path: <david@tethera.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 olra.theworths.org (Postfix) with ESMTP id B7EB3431FB6\r
   6         for <notmuch@notmuchmail.org>; Sat, 23 Aug 2014 20:58:50 -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: 0\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         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 7y6cedmaRur0 for <notmuch@notmuchmail.org>;\r
  16         Sat, 23 Aug 2014 20:58:46 -0700 (PDT)\r
  17 Received: from yantan.tethera.net (yantan.tethera.net [199.188.72.155])\r
  18         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 04C07431FAF\r
  21         for <notmuch@notmuchmail.org>; Sat, 23 Aug 2014 20:58:45 -0700 (PDT)\r
  22 Received: from remotemail by yantan.tethera.net with local (Exim 4.80)\r
  23         (envelope-from <david@tethera.net>)\r
  24         id 1XLOx0-0003OC-Gh; Sun, 24 Aug 2014 00:58:42 -0300\r
  25 Received: (nullmailer pid 14459 invoked by uid 1000); Sun, 24 Aug 2014\r
  26         03:58:38 -0000\r
  27 From: David Bremner <david@tethera.net>\r
  28 To: Austin Clements <amdragon@mit.edu>, notmuch@notmuchmail.org\r
  29 Subject: Re: [PATCH v3 04/13] lib: Database version 3: Introduce fine-grained\r
  30         "features"\r
  31 In-Reply-To: <87fvgmg0tx.fsf@awakening.csail.mit.edu>\r
  32 References: <1406859003-11561-1-git-send-email-amdragon@mit.edu>\r
  33         <1406859003-11561-5-git-send-email-amdragon@mit.edu>\r
  34         <87ppfqsv8s.fsf@maritornes.cs.unb.ca>\r
  35         <87fvgmg0tx.fsf@awakening.csail.mit.edu>\r
  36 User-Agent: Notmuch/0.18.1+72~g028c560 (http://notmuchmail.org) Emacs/24.3.1\r
  37         (x86_64-pc-linux-gnu)\r
  38 Date: Sat, 23 Aug 2014 20:58:38 -0700\r
  39 Message-ID: <87zjeu4jyp.fsf@maritornes.cs.unb.ca>\r
  40 MIME-Version: 1.0\r
  41 Content-Type: text/plain\r
  42 X-BeenThere: notmuch@notmuchmail.org\r
  43 X-Mailman-Version: 2.1.13\r
  44 Precedence: list\r
  45 List-Id: "Use and development of the notmuch mail system."\r
  46         <notmuch.notmuchmail.org>\r
  47 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  49 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  50 List-Post: <mailto:notmuch@notmuchmail.org>\r
  51 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  52 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  54 X-List-Received-Date: Sun, 24 Aug 2014 03:58:50 -0000\r
  55 \r
  56 Austin Clements <amdragon@mit.edu> writes:\r
  57 \r
  58 >>> @@ -1048,7 +1164,8 @@ notmuch_database_get_version (notmuch_database_t *notmuch)\r
  59 >>>  notmuch_bool_t\r
  60 >>>  notmuch_database_needs_upgrade (notmuch_database_t *notmuch)\r
  61 >>>  {\r
  62 >>> -    return notmuch->needs_upgrade;\r
  63 >>> +    return notmuch->mode == NOTMUCH_DATABASE_MODE_READ_WRITE &&\r
  64 >>> +   (NOTMUCH_FEATURES_CURRENT & ~notmuch->features);\r
  65 >>>  }\r
  66 >>\r
  67 >> Maybe I'm not thinking hard enough here, but how does this deal with a\r
  68 >> feature that is needed to open a database in read only mode? Maybe it\r
  69 >> needs a comment for people not as clever as Austin ;).\r
  70 >\r
  71 > I'm not quite sure what you mean.  notmuch_database_needs_upgrade\r
  72 > returns false for read-only databases because you can't upgrade a\r
  73 > read-only database.  This was true before this patch, too, though it was\r
  74 > less obvious.  (Maybe that's not what you're asking?)\r
  75 \r
  76 Yes, that's what I was asking. I guess it's orthogonal to your patch\r
  77 series, but the logic of returning FALSE for read only databases is not\r
  78 very intuitive to me (in the sense that "needs upgrade" is not the\r
  79 opposite of "can't be upgraded".  Your new comment later in the series\r
  80 is better, but it would IMHO be even better if you mentioned the read\r
  81 only case.\r
  82 \r
  83 d\r