<div dir="ltr"><div><div><div>As far as I understand the bindings, the returned message is not needed because the return<br></div>value of `Database._open` (line 212) contains information about the type of error that occurred.<br></div>This is passed on two lines below if the open-call was unsuccessful.<br><br></div>The
 docstrings for this method are not ideal (the mentioned "status" 
parameter is actually called "mode") but in terms of functionality it 
seems ok.</div><div class="gmail_extra"><br><div class="gmail_quote">On 12 March 2016 at 16:33, David Bremner <span dir="ltr"><<a href="mailto:david@tethera.net" target="_blank">david@tethera.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">David Bremner <<a href="mailto:david@tethera.net">david@tethera.net</a>> writes:<br>
<br>
>> -    """notmuch_database_open"""<br>
>> -    _open = nmlib.notmuch_database_open<br>
>> +    """notmuch_database_open_verbose"""<br>
>> +    _open = nmlib.notmuch_database_open_verbose<br>
>>      _open.argtypes = [c_char_p, c_uint, POINTER(NotmuchDatabaseP)]<br>
>>      _open.restype = c_uint<br>
><br>
> I expect you probably want to add the new binding, rather than replacing<br>
> the old one. Otherwise updating the bindings will break all bindings<br>
> using scripts.<br>
<br>
</span>Oh silly me. That's not actually exposed to the python API, so that's<br>
fine. On the other hand, don't you want to update the argument types and<br>
do something useful (throw an exception?) with the returned message?  As<br>
it stands you lose information.<br>
<span class="HOEnZb"><font color="#888888"><br>
d<br>
</font></span></blockquote></div><br></div>