Incremental review
> * clarify how TreeMonad gets parameterized
Looks good, one tiny comment:
> This module provides the readily configured 'virtualTreeIO' that never
> writes any changes, but may trigger filesystem reads as appropriate.
If I understand this sentence correctly this should be "ready
configured" (meaning already or previously configured with hashItem and
dumpItem) rather than "readily configured" (meaning easily configured).
> * fix docs for Darcs.Util.Tree.Monad.flushItem
Looks good, again one tiny comment:
> --- | Flush a single item to disk. This is the only procedure that uses
> --- 'dumpItem' and 'hashItem' from the environment.
> +-- | Flush a single item to disk. This is the only procedure that uses the
> +-- Reader part of the environment (the procedure of type @'DumpItem' m@).
Maybe "directly uses"?
|