Re: [RFC PATCH v2 0/3] lib/cli/emacs: limit number of messages in search results

Subject: Re: [RFC PATCH v2 0/3] lib/cli/emacs: limit number of messages in search results

Date: Wed, 02 Nov 2011 09:27:41 +0000

To: David Bremner, notmuch@notmuchmail.org

Cc:

From: Jani Nikula


On Tue, 01 Nov 2011 09:30:56 -0300, David Bremner <david@tethera.net> wrote:
> Is just because it add a function to the library that you think this
> might be problematic?  I don't think we are super-dogmatic about the
> library never growing.  When notmuch started, there were no bindings, so
> in retrospect maybe more functionality went into the CLI than might
> happen if we started from scratch. If I remember Carl's statement
> correctly, one rule is that stuff in the library should not require
> configuration.

Hi, thanks, that's encouraging. It's mostly that, based on past
experience, I'm hesitant about extending a library interface. Once you
get users, you have to live with it. So you want to get it right.

Having said that, I think in this v2 of the set, the library interface
for notmuch_query_count_threads() is sane and complimentary to the
existing notmuch_query_count_messages(). (Also quoting IRC: "<amdragon>
j4ni: Yes, the interface seems totally reasonable.") Later on, someone
might come up with a better implementation, but for now it's already
much better than having to iterate and construct the threads.

I'll prepare a v3 with some tests and polish.

BR,
Jani.

Thread: