On Fri, 08 Jul 2011 23:23:40 +0200, Michal Sojka <[email protected]> wrote:
> OK. I changed my mind. I thought that the approach with 'notmuch
> search-tags QEURY' followed by 'notmuch count tag:XXX and QUERY' for
> returned tags would be faster then 'notmuch search-tags' followed by
> 'notmuch count tag:XXX and QUERY' for all tags, but it turned out not to
> be the case.

For me, the stronger argument was that the code would have been simpler,
because notmuch-hello refreshes don't happen that often that performance
shouldn't matter much.

> 
> I played with the customization interface a bit and found the following
> problems:
> 
> - It is annoying to insert section titles that includes ": " at the end
>   in order to be nicely "rendered". I think this could be appended
>   automatically.

Agreed.

> 
> - :hide-empty-tags should be renamed in :hide-empty-searches to be
>   effective (see the patch below).

This has been done in v4 of the patch, for which I screwed up the
In-Reply-To header and hence is listed as a separate thread:

id:"[email protected]"

> 
> - The title of custom tags section was not passed correctly to the
>   functions. This is also fixed in the patch below.

I changed title to a mandatory argument for consistency with
notmuch-insert-searches and because a title-less section wouldn't make
much sense anyway.

Cheers,
Daniel

Attachment: pgp2fGjEQHHJa.pgp
Description: PGP signature

_______________________________________________
notmuch mailing list
[email protected]
http://notmuchmail.org/mailman/listinfo/notmuch

Reply via email to