b6/b40e43c0cabd1a114a78be5c5b372cfe3aacc2

   1 Return-Path: <patricktotzke@gmail.com>\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 F0B9E6DE1873\r
   6  for <notmuch@notmuchmail.org>; Sun, 13 Mar 2016 05:03:25 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -1.01\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-1.01 tagged_above=-999 required=5\r
  12  tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13  FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7,\r
  14  RCVD_IN_MSPIKE_H2=-0.211, SPF_PASS=-0.001] autolearn=disabled\r
  15 Received: from arlo.cworth.org ([127.0.0.1])\r
  16  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  17  with ESMTP id nZeFJloXLEOD for <notmuch@notmuchmail.org>;\r
  18  Sun, 13 Mar 2016 05:03:23 -0700 (PDT)\r
  19 Received: from mail-lb0-f176.google.com (mail-lb0-f176.google.com\r
  20  [209.85.217.176])\r
  21  by arlo.cworth.org (Postfix) with ESMTPS id 047366DE186A\r
  22  for <notmuch@notmuchmail.org>; Sun, 13 Mar 2016 05:03:23 -0700 (PDT)\r
  23 Received: by mail-lb0-f176.google.com with SMTP id x1so206873965lbj.3\r
  24  for <notmuch@notmuchmail.org>; Sun, 13 Mar 2016 05:03:22 -0700 (PDT)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;\r
  26  h=mime-version:reply-to:in-reply-to:references:date:message-id\r
  27  :subject:from:to:cc;\r
  28  bh=YCM6UPksz511AsnAT9ete9v5QFCnScKTahzFSsy6KRY=;\r
  29  b=ht/Tx4uJK6C4JCbUdXUXqo9lT8544AegBT3dCTqZbwOlwd60xZwiCcKM8UmtMH1n2g\r
  30  z6AM7BSaM4RpIRvdGjuDGLQllEA3x+2EMqX6s2qFK19D8itl5dce/TwFHAZtQvKCW/2y\r
  31  n4Iqya4BK5n9OKzefraWeyPPA6hTSTWO+Bs8hWP00C+myIU9bgImTI+PYSTzJEtQalh5\r
  32  WG78e9GDVJrsCJ09YF31OltEKwBvi1Dew9dA0R4OelN5g6t5GJgFege2MnzdVD05YZXf\r
  33  JKwZa/FdZbWjYIo1jrfUpy4Yig34ws5d/+DtB/B3VeinM9zHQn7f61ytOmsgm8qAx5Fm\r
  34  312g==\r
  35 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  36  d=1e100.net; s=20130820;\r
  37  h=x-gm-message-state:mime-version:reply-to:in-reply-to:references\r
  38  :date:message-id:subject:from:to:cc;\r
  39  bh=YCM6UPksz511AsnAT9ete9v5QFCnScKTahzFSsy6KRY=;\r
  40  b=kZ1H9blGdTLudBQnkIwAlhSxXYJm0rinLhLTzv5KTvOWtzilbrxt35IjE7pOGeOCRw\r
  41  AvTfNi8mCk/nZID981BMXMe9R4NfmWxBc/1A2N6YJnDWj77IegnavRlNaeCe66iFy1EH\r
  42  YLFX4AOy3kbEJWl/BDDuj93N7vZA/tFm/AM06gKdxVfKL0FDRLhVnrlp5xOqcVc1R9X6\r
  43  hDgkIpuKumfC2GHzmpqOfcwtUlU7gyqC2qZMJH79cz1YJxPgMq+mVZK0Rq/bIiAx17bP\r
  44  nqCNuWH57MOgFxsE4fiFdyK895yvUTlUvkBrnjAJq6KoVdzdtQFIOAXcuddlh3rKJLKC\r
  45  F3jQ==\r
  46 X-Gm-Message-State: AD7BkJJVg/+RENGuxmIszV18oqU4+HS5GZj9vD6lUTklsTBKp4UxcGOadEp8WHssCnjdQ72n9Z6SBN7IlAvmcQ==\r
  47 MIME-Version: 1.0\r
  48 X-Received: by 10.25.169.82 with SMTP id s79mr6432345lfe.47.1457870600694;\r
  49  Sun, 13 Mar 2016 05:03:20 -0700 (PDT)\r
  50 Received: by 10.25.87.145 with HTTP; Sun, 13 Mar 2016 05:03:20 -0700 (PDT)\r
  51 Reply-To: patricktotzke@gmail.com\r
  52 In-Reply-To: <87oaaj64gh.fsf@zancas.localnet>\r
  53 References: <1457795857-12410-1-git-send-email-patricktotzke@gmail.com>\r
  54  <87r3ff64sa.fsf@zancas.localnet> <87oaaj64gh.fsf@zancas.localnet>\r
  55 Date: Sun, 13 Mar 2016 12:03:20 +0000\r
  56 Message-ID:\r
  57  <CAM7JpKkXciAooWMaQ+uuWxTAs_O6DAr0-u-n+uTy2c1w-tkm3g@mail.gmail.com>\r
  58 Subject: Re: [PATCH] use notmuch_database_open_verbose\r
  59 From: Patrick Totzke <patricktotzke@gmail.com>\r
  60 To: David Bremner <david@tethera.net>\r
  61 Cc: "notmuch@notmuchmail.org" <notmuch@notmuchmail.org>\r
  62 Content-Type: multipart/alternative; boundary=001a114106d848727c052decf368\r
  63 X-BeenThere: notmuch@notmuchmail.org\r
  64 X-Mailman-Version: 2.1.20\r
  65 Precedence: list\r
  66 List-Id: "Use and development of the notmuch mail system."\r
  67  <notmuch.notmuchmail.org>\r
  68 List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
  69  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  70 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  71 List-Post: <mailto:notmuch@notmuchmail.org>\r
  72 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  73 List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  74  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  75 X-List-Received-Date: Sun, 13 Mar 2016 12:03:26 -0000\r
  76 \r
  77 --001a114106d848727c052decf368\r
  78 Content-Type: text/plain; charset=UTF-8\r
  79 \r
  80 As far as I understand the bindings, the returned message is not needed\r
  81 because the return\r
  82 value of `Database._open` (line 212) contains information about the type of\r
  83 error that occurred.\r
  84 This is passed on two lines below if the open-call was unsuccessful.\r
  85 \r
  86 The docstrings for this method are not ideal (the mentioned "status"\r
  87 parameter is actually called "mode") but in terms of functionality it seems\r
  88 ok.\r
  89 \r
  90 On 12 March 2016 at 16:33, David Bremner <david@tethera.net> wrote:\r
  91 \r
  92 > David Bremner <david@tethera.net> writes:\r
  93 >\r
  94 > >> -    """notmuch_database_open"""\r
  95 > >> -    _open = nmlib.notmuch_database_open\r
  96 > >> +    """notmuch_database_open_verbose"""\r
  97 > >> +    _open = nmlib.notmuch_database_open_verbose\r
  98 > >>      _open.argtypes = [c_char_p, c_uint, POINTER(NotmuchDatabaseP)]\r
  99 > >>      _open.restype = c_uint\r
 100 > >\r
 101 > > I expect you probably want to add the new binding, rather than replacing\r
 102 > > the old one. Otherwise updating the bindings will break all bindings\r
 103 > > using scripts.\r
 104 >\r
 105 > Oh silly me. That's not actually exposed to the python API, so that's\r
 106 > fine. On the other hand, don't you want to update the argument types and\r
 107 > do something useful (throw an exception?) with the returned message?  As\r
 108 > it stands you lose information.\r
 109 >\r
 110 > d\r
 111 >\r
 112 \r
 113 --001a114106d848727c052decf368\r
 114 Content-Type: text/html; charset=UTF-8\r
 115 Content-Transfer-Encoding: quoted-printable\r
 116 \r
 117 <div dir=3D"ltr"><div><div><div>As far as I understand the bindings, the re=\r
 118 turned message is not needed because the return<br></div>value of `Database=\r
 119 ._open` (line 212) contains information about the type of error that occurr=\r
 120 ed.<br></div>This is passed on two lines below if the open-call was unsucce=\r
 121 ssful.<br><br></div>The\r
 122  docstrings for this method are not ideal (the mentioned &quot;status&quot;=\r
 123 =20\r
 124 parameter is actually called &quot;mode&quot;) but in terms of functionalit=\r
 125 y it=20\r
 126 seems ok.</div><div class=3D"gmail_extra"><br><div class=3D"gmail_quote">On=\r
 127  12 March 2016 at 16:33, David Bremner <span dir=3D"ltr">&lt;<a href=3D"mai=\r
 128 lto:david@tethera.net" target=3D"_blank">david@tethera.net</a>&gt;</span> w=\r
 129 rote:<br><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;borde=\r
 130 r-left:1px #ccc solid;padding-left:1ex"><span class=3D"">David Bremner &lt;=\r
 131 <a href=3D"mailto:david@tethera.net">david@tethera.net</a>&gt; writes:<br>\r
 132 <br>\r
 133 &gt;&gt; -=C2=A0 =C2=A0 &quot;&quot;&quot;notmuch_database_open&quot;&quot;=\r
 134 &quot;<br>\r
 135 &gt;&gt; -=C2=A0 =C2=A0 _open =3D nmlib.notmuch_database_open<br>\r
 136 &gt;&gt; +=C2=A0 =C2=A0 &quot;&quot;&quot;notmuch_database_open_verbose&quo=\r
 137 t;&quot;&quot;<br>\r
 138 &gt;&gt; +=C2=A0 =C2=A0 _open =3D nmlib.notmuch_database_open_verbose<br>\r
 139 &gt;&gt;=C2=A0 =C2=A0 =C2=A0 _open.argtypes =3D [c_char_p, c_uint, POINTER(=\r
 140 NotmuchDatabaseP)]<br>\r
 141 &gt;&gt;=C2=A0 =C2=A0 =C2=A0 _open.restype =3D c_uint<br>\r
 142 &gt;<br>\r
 143 &gt; I expect you probably want to add the new binding, rather than replaci=\r
 144 ng<br>\r
 145 &gt; the old one. Otherwise updating the bindings will break all bindings<b=\r
 146 r>\r
 147 &gt; using scripts.<br>\r
 148 <br>\r
 149 </span>Oh silly me. That&#39;s not actually exposed to the python API, so t=\r
 150 hat&#39;s<br>\r
 151 fine. On the other hand, don&#39;t you want to update the argument types an=\r
 152 d<br>\r
 153 do something useful (throw an exception?) with the returned message?=C2=A0 =\r
 154 As<br>\r
 155 it stands you lose information.<br>\r
 156 <span class=3D"HOEnZb"><font color=3D"#888888"><br>\r
 157 d<br>\r
 158 </font></span></blockquote></div><br></div>\r
 159 \r
 160 --001a114106d848727c052decf368--\r