This is a mix of haddock Darcs.Repository.PatchIndex, weeding (little)
code that is not actually used elsewhere, reordering function
definitions so that they lie close to where they are used.

Also, the two functions used by "darcs show patch-index" are clearly
made read-only (do not attempt to update patch-index if out-of-sync),
which simplifies them.

Attachments

• haddock-and-cleanups-in-patch_index-code.dpatch

Looks good to me. The docs are greatly improved. Several functions that
are used only once have been inlined or made local to the using
function. In a few places unneeded arguments or results have been elided.

Just a remark, since this patch is not responsible for it: IMO the code
layout of src/darcs/clean/src/Darcs/Repository/PatchIndex.hs could be
further improved by breaking the many over-long lines (over 130
characters). They make viewing diffs side by side difficult unless you
have two monitors (and don't use xmonad).

I just saw that this patch depends on 'get rid of almost all flag list
hacking', which is not yet in reviewed. What is the procedure in this
case? Should I revert 'accepted' to 'review-in-progress'?

My mistake: forgot to pull from reviewed.