[darcs-devel] [patch1984] finally fix parsing of option arguments (and 2 more)

[Ganesh Sittampalam]

Ganesh Sittampalam <ganesh at earth.li> added the comment:

On 11/02/2020 08:33, Ben Franksen wrote:

>   * finally fix parsing of option arguments

Looks good. I think it'd have been slightly nicer to have had
showIntArg/showIndexRangeArg counterparts to
parseIntArg/prettyIndexRangeArg in D.UI.Options.Utils so that the string
formats are defined in one place, but it's no big deal.

>   * fix some outdated TODO items in D.UI.Options.All
>   * improve haddocks in D.UI.Options.Util
>   * fix extraneous space in error message
>   * add test for log --index=N-M
>   * add test for clean failure with malformed option arguments

All fine.

__________________________________
Darcs bug tracker <bugs at darcs.net>
<http://bugs.darcs.net/patch1984>
__________________________________