python: fix Message.get_header
authorJustus Winter <4winter@informatik.uni-hamburg.de>
Thu, 17 May 2012 16:14:58 +0000 (18:14 +0200)
committerJustus Winter <4winter@informatik.uni-hamburg.de>
Thu, 17 May 2012 16:23:11 +0000 (18:23 +0200)
8dc8495010057202b725ac029831c03f4e3ab6bd introduced a bug, if the
requested header is not set the underlying notmuch function returns an
empty string that also made the expression true resulting in an
exception being raised. Partly revert the commit to fix this
issue. Testing for equality with None is correct in this case since
the restype of the function Message._get_header is c_char_p so NULL
pointers are in fact converted to None in this case.

Signed-off-by: Justus Winter <4winter@informatik.uni-hamburg.de>
bindings/python/notmuch/message.py

index d7f029f8789d43faeca117e4febec70ede109506..d1c1b58c4a9a0cc129830fdcec2eceee93821d60 100644 (file)
@@ -229,7 +229,7 @@ class Message(Python3StringMixIn):
 
         #Returns NULL if any error occurs.
         header = Message._get_header(self._msg, _str(header))
-        if not header:
+        if header == None:
             raise NullPointerError()
         return header.decode('UTF-8', 'ignore')