DomainKey-Signature: a=rsa-sha1; c=nofws; d=SSpaeth.de; h=from:to:subject
	:in-reply-to:references:date:message-id:mime-version:
	content-type; q=dns; s=sspaeth.de; b=QU8B0jbQUKX+1bOQ9YDkRBUpjwi
	WHtJo5sglyEoi9KZSTf1OxLaEDPi/U3cGPQkCNIMuXSBYr3B/gWmEaTQPi5L9S/8
	yUzlAQ5cElWdHRxGsbiyqBDP2mBtKGIzSjEtrL1EzfCTNu1sWW9OQhwFDn9+yXbW
	FHdESGzTtpIim1wo=
From: Sebastian Spaeth <Sebastian@SSpaeth.de>
To: Brian May <brian@microcomaustralia.com.au>,
	notmuch <notmuch@notmuchmail.org>
Subject: Re: [python] get all messages of a thread
In-Reply-To: <BANLkTi=df3+LBWasFic9ZNrKaLGA7Bmdew@mail.gmail.com>
References: <1306588052-sup-9838@brick>
	<BANLkTikEJ9dJ5kLg2QJQ+egHg2sh7S1wiA@mail.gmail.com>
	<87aae07lxi.fsf@SSpaeth.de>
	<BANLkTi=df3+LBWasFic9ZNrKaLGA7Bmdew@mail.gmail.com>
User-Agent: Notmuch/0.5-209-gc8b5718 (http://notmuchmail.org) Emacs/23.2.1
	(x86_64-pc-linux-gnu)
Date: Thu, 02 Jun 2011 16:20:14 +0200
Message-ID: <87r57cuxgh.fsf@SSpaeth.de>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="=-=-=";
	micalg=pgp-sha1; protocol="application/pgp-signature"
Precedence: list

--=-=-=
Content-Transfer-Encoding: quoted-printable

On Thu, 2 Jun 2011 19:43:29 +1000, Brian May wrote:
> On 2 June 2011 17:05, Sebastian Spaeth <Sebastian@sspaeth.de> wrote:
>=20
> > What would be the best way to solve this (besides fixing the C api to
> > allow to reset the iterator ;-) ?)

> * It is not easy to fix the C api to reset the iterator (what about
> repeating the search?)

I am not sure about the difficulty of that, I am not a C-kind of
guy. Repeating the search would be easy but potentially gives you
different results since the db could have changed since then.
=20
> * The only accurate way to get the number of messages is to iterate throu=
gh
> every search result and count them?

There is count_messages() which wraps notmuch_query_count_messages which
invokes some very quick xapian function. But the xapian docs explicitely
state that it's xapians best guess and not guaranteed to be the real
number of messages (although it always was in my attempts). And you
don't want len() to return an approximation of the iterator length....

> If so, then len(...) I think might be very slow if there are a large numb=
er
> of elements.
=20
> Maybe it might be easier/better to implement object.__nonzero__(self)
>  instead of the object.__len__(self) method?
>=20
> http://docs.python.org/reference/datamodel.html
>=20
> object.__nonzero__(self)
> Called to implement truth value testing and the built-in operation bool();
> should return False or True, or their integer equivalents 0 or 1. When th=
is
> method is not defined, __len__() is called, if it is defined, and the obj=
ect
> is considered true if its result is nonzero. If a class defines neither
> __len__() nor __nonzero__(), all its instances are considered true.

Interesting, did not know about this one. I guess that would solve the
example with the:

t =3D query.search_threads()

if t:
  for thread in t:
    print thread

Actually the "if t:" is no longer needed, I just tried a query returning
no Threads and=20

for thread in threads:
  pass

works just fine with an empty Threads() object.

I made the change, and implemented __nonzero__ and removed the len()
method. It just doesn't make sense on 1-time iterators. (I documented
the change in the API docs). Sorry if this breaks existing code.

list(Threads()) works just fine too, it just took a while to create a
list of 13k Thread() objects on this laptop. (and list() will of course
not return until it is finished).


Sebastian

--=-=-=
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)

iEYEARECAAYFAk3nnB4ACgkQVYX1jMgnoGIRMgCeOMJq2ej5K9/WXtxMLMjD02lo
HJoAn1BMBhRczltfWmOI/GQkquUv8WdB
=T4tg
-----END PGP SIGNATURE-----
--=-=-=--