[darcs-devel] [patch1878] re-unify finished messages in PatchAppliers (and 2 more)

[Ganesh Sittampalam]

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

>  * re-unify finished messages in PatchAppliers
>  * share common start and finish parts in PatchAppliers
>  * issue no-patches message in applyPatchesStart regardless of --reorder

This logic looks pretty clean now, thanks. I wonder if applyPatchesStart
and applyPatchesFinish should be some kind of bracketed operation
applyPatchesWith instead, but that's not important.

>  * cleanup applyCmd
>  * apply: print "reading from stdin" unless --quiet

Both fine.

----------
status: review-in-progress -> accepted-pending-tests

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