[darcs-devel] [patch1791] fix several uses of error where it shoul... (and 7 more)

[Ben Franksen]

Ben Franksen <ben.franksen at online.de> added the comment:

>> hunk ./src/Darcs/Repository/PatchIndex.hs 730
>> -      unless (isInOrder ascTs pids) (error $ "In order test failed! 
> filename: " ++ show fn)
>> [...]>
> How is the user supposed to understand/recover from these errors? Do
> they indicate that the patch index is corrupt and should be recreated,
> for example?

These are in function piTest which is part of the 'show patch-index'
command, which is why I thought they should be treated as normal
failures. I guess 'show patch-index' was added for developers mostly to
see if the patch index works or to investigate things if it doesn't.

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