python bindings: fix return value types with ctypes

python bindings: fix return value types with ctypes

[VA]

[-- Attachment #1: Type: text/plain, Size: 92 bytes --]

There were some typos, where restype was misspelled "resttype" and an 
"argtypes" was lost.

[-- Attachment #2: 0001-python-fix-set_sort-add_tag_exclude-restype-argtypes.patch --]
[-- Type: text/x-patch, Size: 1160 bytes --]

From f291f3d20b4683b468c48b35efc6b5ecc2719a1d Mon Sep 17 00:00:00 2001
From: hydrargyrum <dev@indigo.re>
Date: Sat, 30 Mar 2019 10:31:58 +0100
Subject: [PATCH] python: fix set_sort/add_tag_exclude restype/argtypes typos

---
 bindings/python/notmuch/query.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/bindings/python/notmuch/query.py b/bindings/python/notmuch/query.py
index 06c7b11b..cc70e2aa 100644
--- a/bindings/python/notmuch/query.py
+++ b/bindings/python/notmuch/query.py
@@ -108,7 +108,7 @@ class Query(object):
 
     _set_sort = nmlib.notmuch_query_set_sort
     _set_sort.argtypes = [NotmuchQueryP, c_uint]
-    _set_sort.argtypes = None
+    _set_sort.restype = None
 
     def set_sort(self, sort):
         """Set the sort order future results will be delivered in
@@ -121,7 +121,7 @@ class Query(object):
 
     _exclude_tag = nmlib.notmuch_query_add_tag_exclude
     _exclude_tag.argtypes = [NotmuchQueryP, c_char_p]
-    _exclude_tag.resttype = None
+    _exclude_tag.restype = None
 
     def exclude_tag(self, tagname):
         """Add a tag that will be excluded from the query results by default.
-- 
2.20.1

Re: python bindings: fix return value types with ctypes

[Tomi Ollila]

On Sat, Mar 30 2019, VA wrote:

LGTM (on paper -- checked ctypes doc that '.restype' looks like correctly
spelled name)

> There were some typos, where restype was misspelled "resttype" and an 
> "argtypes" was lost.
> From f291f3d20b4683b468c48b35efc6b5ecc2719a1d Mon Sep 17 00:00:00 2001
> From: hydrargyrum <dev@indigo.re>
> Date: Sat, 30 Mar 2019 10:31:58 +0100
> Subject: [PATCH] python: fix set_sort/add_tag_exclude restype/argtypes typos
>
> ---
>  bindings/python/notmuch/query.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/bindings/python/notmuch/query.py b/bindings/python/notmuch/query.py
> index 06c7b11b..cc70e2aa 100644
> --- a/bindings/python/notmuch/query.py
> +++ b/bindings/python/notmuch/query.py
> @@ -108,7 +108,7 @@ class Query(object):
>  
>      _set_sort = nmlib.notmuch_query_set_sort
>      _set_sort.argtypes = [NotmuchQueryP, c_uint]
> -    _set_sort.argtypes = None
> +    _set_sort.restype = None
>  
>      def set_sort(self, sort):
>          """Set the sort order future results will be delivered in
> @@ -121,7 +121,7 @@ class Query(object):
>  
>      _exclude_tag = nmlib.notmuch_query_add_tag_exclude
>      _exclude_tag.argtypes = [NotmuchQueryP, c_char_p]
> -    _exclude_tag.resttype = None
> +    _exclude_tag.restype = None
>  
>      def exclude_tag(self, tagname):
>          """Add a tag that will be excluded from the query results by default.
> -- 
> 2.20.1
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> https://notmuchmail.org/mailman/listinfo/notmuch

Re: python bindings: fix return value types with ctypes

[David Bremner]

VA <dev+notmuch@indigo.re> writes:

> There were some typos, where restype was misspelled "resttype" and an 
> "argtypes" was lost.

pushed.

d