Austin Clements 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
Quoth David Bremner on Nov 25 at 8:05 pm:
> Austin Clements writes:
> >> +add_email_corpus takes arguments "--small" and "--medium" for when you
> >> +want smaller corpuses to check.
> >
> > "corpora"?
>
> reworded to say
>
> ,
> | add_email_corpus takes arguments "--small" and "--medium"
Austin Clements writes:
>> +subdirs := compat completion emacs lib man parse-time-string
>> +subdirs := $(subdirs) performance-test util test
> += ?
>
Sure.
>> +CORPUS_NAME := notmuch-email-corpus-$(PERFTEST_VERSION).tar.xz
>
> Would it make sense to split out the different size corpora so a
Quoth david at tethera.net on Nov 25 at 11:02 am:
> From: David Bremner
>
> This is not near as fancy as as the unit tests, on the theory that
> the code should typically be crashing when performance tuning.
> Nonetheless, there is plenty of room for improvement. Several more of
> the pieces of
From: David Bremner brem...@debian.org
This is not near as fancy as as the unit tests, on the theory that
the code should typically be crashing when performance tuning.
Nonetheless, there is plenty of room for improvement. Several more of
the pieces of the test infrastructure (e.g. the option
Quoth da...@tethera.net on Nov 25 at 11:02 am:
From: David Bremner brem...@debian.org
This is not near as fancy as as the unit tests, on the theory that
the code should typically be crashing when performance tuning.
Nonetheless, there is plenty of room for improvement. Several more of
the
Quoth David Bremner on Nov 25 at 8:05 pm:
Austin Clements amdra...@mit.edu writes:
+add_email_corpus takes arguments --small and --medium for when you
+want smaller corpuses to check.
corpora?
reworded to say
,
| add_email_corpus takes arguments --small and --medium for when