The first patch renames a test to the issue it's testing. The second improves
the error message presented by darcs when invalid patterns are provided to
--match by changing a couple of calls of the Parsec <?> combinator.

2 patches for repository http://darcs.net:

Fri Dec 21 13:46:21 GMT 2012  Owen Stephens <darcs@owenstephens.co.uk>
  * Rename a test

Fri Dec 21 13:46:29 GMT 2012  Owen Stephens <darcs@owenstephens.co.uk>
  * Improve error message for bad match patterns

Attachments

• patch-preview.txt
• rename-a-test.dpatch
• unnamed

Great - the new error message is a definite improvement!

Some thoughts about future improvements that occurred to me while 
experimenting:

I found "expecting valid expressions over:" to be a bit obscure, 
although the pointer to "darcs help patterns" cleared it up quickly. 
Perhaps just "expecting one of:" would be good enough, and people can 
use "darcs help patterns" to find out about the boolean expressions.

It's a bit unfortunate that most of the keywords without any further 
arguments actually work and just match a lot of stuff, as it limits the 
ability of people to discover sensible arguments by experimentation and 
error messages.

Tests pass.