Re: [Patch v4 2/2] test: initial performance testing infrastructure

Subject: Re: [Patch v4 2/2] test: initial performance testing infrastructure

Date: Mon, 26 Nov 2012 08:44:27 -0400

To: Austin Clements

Cc: notmuch@notmuchmail.org

From: David Bremner


Austin Clements <amdragon@MIT.EDU> writes:
>
> The README definitely helps.  Might still be worth a comment in the
> code since it took me some thinking to realize it would do something
> reasonable when given no argument.  Perhaps above the initial
> assignment of arg,

I decided to fix the code, rather than comment it ;).  Now every code
path makes an explicit, non-empty assignment to $arg

I have pushed that slightly updated version of the patches.

d



Thread: