[PATCH] use notmuch_database_open_verbose

David Bremner david at tethera.net
Sun Mar 13 05:37:40 PDT 2016


Patrick Totzke <patricktotzke at gmail.com> writes:

> As far as I understand the bindings, the returned message is not needed
> because the return
> value of `Database._open` (line 212) contains information about the type of
> error that occurred.
> This is passed on two lines below if the open-call was unsuccessful.
>
> The docstrings for this method are not ideal (the mentioned "status"
> parameter is actually called "mode") but in terms of functionality it seems
> ok.

The error_message parameter (potentially) contains additional
information about an error. In any case, you're passing one too few
arguments to n_d_open_verbose, which doesn't seem like a good idea.

d


More information about the notmuch mailing list