Re: [PATCH v2 0/6] batch count for notmuch-hello speedup

Subject: Re: [PATCH v2 0/6] batch count for notmuch-hello speedup

Date: Sun, 31 Mar 2013 10:39:24 +0300

To: Jani Nikula, notmuch@notmuchmail.org

Cc:

From: Tomi Ollila


On Sat, Mar 09 2013, Jani Nikula <jani@nikula.org> wrote:

> Rebased v2 of id:cover.1358273133.git.jani@nikula.org
>
> Jani Nikula (5):
>   cli: remove useless strdup
>   cli: extract count printing to a separate function in notmuch count
>   cli: add --batch option to notmuch count
>   man: document notmuch count --batch and --input options
>   test: notmuch count --batch and --input options
>
> Mark Walters (1):
>   emacs: hello: use batch count

LGTM. comments in id:m2a9s0hvgy.fsf@guru.guru-group.fi and
id:m27gn4htrg.fsf@guru.guru-group.fi apply.

Tomi

>
>  emacs/notmuch-hello.el   |   52 +++++++++++++---------
>  man/man1/notmuch-count.1 |   20 +++++++++
>  notmuch-count.c          |  111 +++++++++++++++++++++++++++++++++++-----------
>  test/count               |   46 +++++++++++++++++++
>  4 files changed, 182 insertions(+), 47 deletions(-)
>
> -- 
> 1.7.10.4

Thread: