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: Tue, 01 Nov 2011 00:07:59 +0200

To: Jameson Graef Rollins, notmuch@notmuchmail.org

Cc: amdragon@mit.edu

From: Jani Nikula


On Mon, 31 Oct 2011 14:44:29 -0700, Jameson Graef Rollins <jrollins@finestructure.net> wrote:
> Hi, Jani.  Thanks for working on this.  This should also be valuable for
> vim users.

Thanks for your interest! :)

> In order to push forward with this, though, I think we really need to
> have a complete unit test for this new functionality.  We usually like
> to see units tests that describe and then test for the new functionality
> you wish to add, followed by the patches that provide the new
> functionality.  Lots of good tests for new functionality being proposed
> here shouldn't be too difficult to work out ahead of time.

Right. I'd just like to make sure the approach I've taken (particularly
patch 1 in the set as it touches the lib) is acceptable before spending
time on testing and documentation etc. Indeed patches 1 and 2 changed
fundamentally between v1 and v2 after some chats on IRC. If the comments
now are favourable, I'll write the tests and documentation. (Though I
guess I have to admit the tests would've been beneficial to me already
now...)

BR,
Jani.

Thread: