4 patches for repository http://darcs.net/screened:

patch 7aa82147dac5338031790d08a7b10c3062266da4
Author: Ben Franksen <ben.franksen@online.de>
Date:   Mon Sep  9 10:09:13 CEST 2019
  * remove OneIndex from haveNonrangeMatch
  
  While in principle this would be a valid nonrange matcher, in practice we
  can (and do) only use it as a patchSetMatch. This is because the index of a
  patch in the current ordering of a repo cannot be read off the patch itself.

patch 16c01c0ab166876790b5cae7f07eeca1df0cc860
Author: Ben Franksen <ben.franksen@online.de>
Date:   Mon Sep  9 10:23:30 CEST 2019
  * cleanup exports of Darcs.Patch.Match
  
  This removes nonrangeMatcherIsTag which is nowhere used and also removes
  matchParser, nonrangeMatcher, and applyNInv from the exports since they are
  used only internally.

patch be16ab83b8490643b807b45d2e5a764950e0ceab
Author: Ben Franksen <ben.franksen@online.de>
Date:   Mon Sep  9 10:53:06 CEST 2019
  * document patchSetMatches in the haddocks for Darcs.Patch.Match

patch a49f3a254e688204df79d6dba600796bc82f52a0
Author: Ben Franksen <ben.franksen@online.de>
Date:   Mon Sep  9 11:03:24 CEST 2019
  * cleanup definition of MatchFlag

Attachments

• patch-preview.txt
• remove-oneindex-from-havenonrangematch.dpatch
• unnamed

>  * remove OneIndex from haveNonrangeMatch

Fine, assuming the tests pass this looks like an improvement to the
readability of 'haveNonrangeMatch'.

>  * cleanup exports of Darcs.Patch.Match

Fine

>  * document patchSetMatches in the haddocks for Darcs.Patch.Match

Looks useful, thanks.

>   * cleanup definition of MatchFlag

Fine