dd/c8e9a1025f196e79262b81f04cf1c8a6bf2b67

   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 0E48B431FBD\r
   6         for <notmuch@notmuchmail.org>; Wed, 24 Sep 2014 11:09: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: 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 MHp51RKLrlpb for <notmuch@notmuchmail.org>;\r
  16         Wed, 24 Sep 2014 11:09:40 -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 3893B431FAF\r
  21         for <notmuch@notmuchmail.org>; Wed, 24 Sep 2014 11:09:40 -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 1XWr0P-0001WY-1m; Wed, 24 Sep 2014 15:09:33 -0300\r
  25 Received: (nullmailer pid 12507 invoked by uid 1000); Wed, 24 Sep 2014\r
  26         18:09:27 -0000\r
  27 From: David Bremner <david@tethera.net>\r
  28 To: "W. Trevor King" <wking@tremily.us>, Austin Clements <amdragon@mit.edu>\r
  29 Subject: Re: [PATCH] lib: Simplify close and codify aborting atomic section\r
  30 In-Reply-To: <20140922190022.GO20827@odin.tremily.us>\r
  31 References: <1411400615-20934-1-git-send-email-amdragon@mit.edu>\r
  32         <20140922165954.GI20827@odin.tremily.us>\r
  33         <20140922185050.GB7970@csail.mit.edu>\r
  34         <20140922190022.GO20827@odin.tremily.us>\r
  35 User-Agent: Notmuch/0.18.1+98~gae27403 (http://notmuchmail.org) Emacs/24.3.1\r
  36         (x86_64-pc-linux-gnu)\r
  37 Date: Wed, 24 Sep 2014 20:09:27 +0200\r
  38 Message-ID: <87iokc28mg.fsf@maritornes.cs.unb.ca>\r
  39 MIME-Version: 1.0\r
  40 Content-Type: text/plain\r
  41 Cc: notmuch@notmuchmail.org\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: Wed, 24 Sep 2014 18:09:48 -0000\r
  55 \r
  56 "W. Trevor King" <wking@tremily.us> writes:\r
  57 \r
  58 \r
  59 > Ah, I thought the implicit flush/commit was just in our code.  Since\r
  60 > it's also in the underlying Xapian close, then this patch looks pretty\r
  61 > good to me.  I'd mention Xapian's explicit close in the notmuch.\r
  62 h\r
  63 > message.  Xapain's docs say [1]:\r
  64 >\r
  65 >   For a WritableDatabase, if a transaction is active it will be\r
  66 >   aborted, while if no transaction is active commit() will be\r
  67 >   implicitly called.\r
  68 \r
  69 I'm not sure what you're asking for here by "explicit close". Isn't what\r
  70 you quote a restatement of \r
  71 \r
  72 + * If the caller is currently in an atomic section (there was a\r
  73 + * notmuch_database_begin_atomic without a matching\r
  74 + * notmuch_database_end_atomic), this will abort the atomic\r
  75 section,\r
  76 + * discarding any modifications made in the atomic section.\r
  77 \r
  78 in terms of underyling Xapian mechanics?\r
  79 \r
  80 \r
  81 P.S. Other than whatever this doc question is, the patch looks ok to me.\r