DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma;
	h=mime-version:sender:in-reply-to:references:date
	:x-google-sender-auth:message-id:subject:from:to:cc:content-type
	:content-transfer-encoding;
	b=oeD++TMlB79MEuILy+pZAULVukCNzISZKzS+ui6zt6vpVooeFkXF82y63PMP1tCnDb
	KVuCFq2OY2EpV5KBeur/Iu8md8so2EIzvOHhWVbQTIgzdRcrkgVVxtSlbSnqIjlA/sxj
	3k1Fr2tETz6urtIHRUzTXoJM2kvIys9mUBdcw=
MIME-Version: 1.0
Sender: amdragon@gmail.com
In-Reply-To: <87ei3mcw7h.fsf@gilead.invalid>
References: <87fwoath2s.fsf@gilead.home.box>
	<BANLkTinKS5LbQsZuK8HyCzj+Lo4xqa2Sfg@mail.gmail.com>
	<871uznqeox.fsf@tredergarh.home.box>
	<BANLkTin0+O=mubHF7JNpw4u3DQDQJQ5zbw@mail.gmail.com>
	<874o4iwwp5.fsf@gilead.invalid> <871uzmwtng.fsf@gilead.invalid>
	<BANLkTikAB0mogUuS-nT_-i51LBdUBHCz3g@mail.gmail.com>
	<87ei3mcw7h.fsf@gilead.invalid>
Date: Wed, 25 May 2011 18:42:30 -0400
Message-ID: <BANLkTi=729DWai4q57iBSfz1wDhBXsmndQ@mail.gmail.com>
Subject: Re: [PATCH] emacs: Make the queries used in the all-tags section
From: Austin Clements <amdragon@mit.edu>
To: Daniel Schoepe <daniel.schoepe@googlemail.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Cc: notmuch@notmuchmail.org
Precedence: list

On Wed, May 25, 2011 at 5:21 PM, Daniel Schoepe
<daniel.schoepe@googlemail.com> wrote:
> On Wed, 25 May 2011 15:11:16 -0400, Austin Clements <amdragon@mit.edu> wr=
ote:
>> So, perhaps something like
>>
>> (defcustom notmuch-hello-tag-list-counts nil
>> =A0 "Method for generating counts displayed in the all tags list.
>>
>> This variable controls the query used to generate counts for each
>> tag in the \"all tags\" list. =A0If nil, the tag list will count
>> all messages with each tag. =A0This can be a query string that will
>> filter the messages counted for each tag. =A0Finally, this can be a
>> function that will be called for each tag and should return a
>> filter query for that tag, or nil to hide the tag."
>> =A0 :type '(choice (const :tag "Count all messages" nil)
>> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0(const :tag "Count unread messages" "=
tag:unread")
>> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0(const :tag "Custom filter" string)
>> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0(const :tag "Custom filter function" =
function))
>> =A0 :group 'notmuch)
>
> That is slightly less accurate though, since the query is not only used
> to generate the counts, but also the searches that are performed when
> one presses return on a tag in the list.

'Doh.  That's what I get for not reading the surrounding code.  I
misunderstood what your patch was going for and assumed it was what
*I* wanted notmuch to do, which is to show me useful counts (e.g.,
unread count), but not to change the underlying query.  ]:--8)

So, to me, it seems like this turns the all tags section into another
saved searches section, just with a slight twist, and makes me wonder
if there's a better way to reconcile these.

> Ah, thanks. I guess too much exposure to Haskell (and how smart GHC is)
> makes me write things in a functional style without thinking about
> performance.

Hah, been there.  I have to constantly remind myself that Elisp is
*not* a self-respecting functional programming language, despite how
much it may look like Scheme.