python: fix error handling

[notmuch] / bindings / python / notmuch / message.py

diff --git a/bindings/python/notmuch/message.py b/bindings/python/notmuch/message.py
index d40a575d92d8587e355935abbb312117e2ec0a0c..883ed233c7e170fbfcab9b58aca2676f31417920 100644 (file)
--- a/bindings/python/notmuch/message.py
+++ b/bindings/python/notmuch/message.py
@@ -117,7 +117,7 @@ class Messages(object):
         :TODO: Make the iterator work more than once and cache the tags in
                the Python object.(?)
         """
         :TODO: Make the iterator work more than once and cache the tags in
                the Python object.(?)
         """

-        if msgs_p is None:
+        if not msgs_p:

             raise NotmuchError(STATUS.NULL_POINTER)
 
         self._msgs = msgs_p
             raise NotmuchError(STATUS.NULL_POINTER)
 
         self._msgs = msgs_p


@@ -349,7 +349,7 @@ class Message(Python3StringMixIn):
               automatically delete the parent object once all derived
               objects are dead.
         """
               automatically delete the parent object once all derived
               objects are dead.
         """

-        if msg_p is None:
+        if not msg_p:

             raise NotmuchError(STATUS.NULL_POINTER)
         self._msg = msg_p
         #keep reference to parent, so we keep it alive
             raise NotmuchError(STATUS.NULL_POINTER)
         self._msg = msg_p
         #keep reference to parent, so we keep it alive


@@ -407,7 +407,7 @@ class Message(Python3StringMixIn):
 
         msgs_p = Message._get_replies(self._msg)
 
 
         msgs_p = Message._get_replies(self._msg)
 

-        if msgs_p is None:
+        if not msgs_p:

             return EmptyMessagesResult(self)
 
         return Messages(msgs_p, self)
             return EmptyMessagesResult(self)
 
         return Messages(msgs_p, self)