diff options
author | Justus Winter <4winter@informatik.uni-hamburg.de> | 2012-04-30 19:12:36 +0200 |
---|---|---|
committer | Justus Winter <4winter@informatik.uni-hamburg.de> | 2012-04-30 19:25:16 +0200 |
commit | 7f74a400d197dac5cdf36960f68f63ce3eeff486 (patch) | |
tree | 9dcedef71864d1bd683e3fda85ee13b80a2deed2 /bindings/python/notmuch/tag.py | |
parent | 162687a99e412098729d639ed7bc27f01372cb84 (diff) |
python: cleanup the __nonzero__ implementations
Cleanup the code, reword the docstring and use the same implementation
in the Threads, Tags and Messages classes.
__nonzero__ implements truth value testing. If __nonzero__ is not
implemented, the python runtime would fall back to `len(..) > 0` thus
exhausting the iterator.
Signed-off-by: Justus Winter <4winter@informatik.uni-hamburg.de>
Diffstat (limited to 'bindings/python/notmuch/tag.py')
-rw-r--r-- | bindings/python/notmuch/tag.py | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/bindings/python/notmuch/tag.py b/bindings/python/notmuch/tag.py index e0598139..363c3487 100644 --- a/bindings/python/notmuch/tag.py +++ b/bindings/python/notmuch/tag.py @@ -109,15 +109,15 @@ class Tags(Python3StringMixIn): next = __next__ # python2.x iterator protocol compatibility def __nonzero__(self): - """Implement bool(Tags) check that can be repeatedly used - - If __nonzero__ is not implemented, "if Tags()" - will implicitly call __len__, using up our iterator, so it is - important that this function is defined. - - :returns: True if the Tags() iterator has at least one more Tag - left.""" - return self._valid(self._tags) > 0 + ''' + Implement truth value testing. If __nonzero__ is not + implemented, the python runtime would fall back to `len(..) > + 0` thus exhausting the iterator. + + :returns: True if the wrapped iterator has at least one more object + left. + ''' + return self._tags and self._valid(self._tags) def __unicode__(self): """string representation of :class:`Tags`: a space separated list of tags |