1e/4e9bd785a212c86105360bbee8734d0d443afe

   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 arlo.cworth.org (Postfix) with ESMTP id CD2826DE1868\r
   6  for <notmuch@notmuchmail.org>; Sat, 12 Mar 2016 08:33:54 -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.031\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.031 tagged_above=-999 required=5\r
  12  tests=[AWL=-0.020, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01]\r
  13  autolearn=disabled\r
  14 Received: from arlo.cworth.org ([127.0.0.1])\r
  15  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16  with ESMTP id GKsqVBwQ8VhZ for <notmuch@notmuchmail.org>;\r
  17  Sat, 12 Mar 2016 08:33:53 -0800 (PST)\r
  18 Received: from fethera.tethera.net (fethera.tethera.net [198.245.60.197])\r
  19  by arlo.cworth.org (Postfix) with ESMTPS id D664E6DE1867\r
  20  for <notmuch@notmuchmail.org>; Sat, 12 Mar 2016 08:33:52 -0800 (PST)\r
  21 Received: from remotemail by fethera.tethera.net with local (Exim 4.84)\r
  22  (envelope-from <david@tethera.net>)\r
  23  id 1aemUn-0007wX-AX; Sat, 12 Mar 2016 11:34:29 -0500\r
  24 Received: (nullmailer pid 32316 invoked by uid 1000);\r
  25  Sat, 12 Mar 2016 16:33:50 -0000\r
  26 From: David Bremner <david@tethera.net>\r
  27 To: Patrick Totzke <patricktotzke@gmail.com>, notmuch@notmuchmail.org\r
  28 Subject: Re: [PATCH] use notmuch_database_open_verbose\r
  29 In-Reply-To: <87r3ff64sa.fsf@zancas.localnet>\r
  30 References: <1457795857-12410-1-git-send-email-patricktotzke@gmail.com>\r
  31  <87r3ff64sa.fsf@zancas.localnet>\r
  32 User-Agent: Notmuch/0.21+74~g6c60fb1 (http://notmuchmail.org) Emacs/24.5.1\r
  33  (x86_64-pc-linux-gnu)\r
  34 Date: Sat, 12 Mar 2016 12:33:50 -0400\r
  35 Message-ID: <87oaaj64gh.fsf@zancas.localnet>\r
  36 MIME-Version: 1.0\r
  37 Content-Type: text/plain\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.20\r
  40 Precedence: list\r
  41 List-Id: "Use and development of the notmuch mail system."\r
  42  <notmuch.notmuchmail.org>\r
  43 List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
  44  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  45 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  46 List-Post: <mailto:notmuch@notmuchmail.org>\r
  47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  48 List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Sat, 12 Mar 2016 16:33:55 -0000\r
  51 \r
  52 David Bremner <david@tethera.net> writes:\r
  53 \r
  54 >> -    """notmuch_database_open"""\r
  55 >> -    _open = nmlib.notmuch_database_open\r
  56 >> +    """notmuch_database_open_verbose"""\r
  57 >> +    _open = nmlib.notmuch_database_open_verbose\r
  58 >>      _open.argtypes = [c_char_p, c_uint, POINTER(NotmuchDatabaseP)]\r
  59 >>      _open.restype = c_uint\r
  60 >\r
  61 > I expect you probably want to add the new binding, rather than replacing\r
  62 > the old one. Otherwise updating the bindings will break all bindings\r
  63 > using scripts.\r
  64 \r
  65 Oh silly me. That's not actually exposed to the python API, so that's\r
  66 fine. On the other hand, don't you want to update the argument types and\r
  67 do something useful (throw an exception?) with the returned message?  As\r
  68 it stands you lose information.\r
  69 \r
  70 d\r