[PATCH 1/2] python: add classes to wrap all notmuch_*_t types

Justus Winter 4winter at informatik.uni-hamburg.de
Mon Nov 28 04:36:44 PST 2011


Quoting Sebastian Spaeth (2011-11-02 08:25:53)
>On Mon, 10 Oct 2011 00:12:53 +0200, Justus Winter <4winter at informatik.uni-hamburg.de> wrote:
>> Signed-off-by: Justus Winter <4winter at informatik.uni-hamburg.de>
>> ---
>>  bindings/python/notmuch/globals.py |   38 +++++++++++++++++++++++++++++++++++-
>
>Hi there, catching up on email, sorry for the delay.
>I am not opposed to the change, but I fail to see any justification for
>a whole lot more complexity either. What was wrong with the previous
>approach?

Well, quoting my commit message:

> Add type information to the ctypes._FuncPtr wrappers and
> use the wrapper classes instead of c_void_p for pointers
> to notmuch_*_t.
>
> This enables the ctypes library to type check parameters
> being handed to functions from the notmuch library.

Justus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: .signature
Type: application/octet-stream
Size: 17 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20111128/0dbb2960/attachment.obj>


More information about the notmuch mailing list