1 patch for repository http://darcs.net/screened:

patch 287acd410dbf93a84b9ef58bf6ba18bf9617df92
Author: Ben Franksen <ben.franksen@online.de>
Date:   Fri Jun  8 11:33:43 CEST 2018
  * simplify D.R.Hashed.reorderInventory
  
  This gets rid of the last use of tentativelyReplacePatches which has been
  removed. Instead of removing patches from the repo and then adding them
  back, we directly write the new patchset. tentativelyReplacePatches
filtered
  out the rebase patch from the patches it removes. This was brittle and
also
  unnecessary: we can assume the rebase patch is not covered by any tag and
  thus automatically belongs to the untagged tail of patches.
  
  Finding the latest tag and cleaning it is now optimized by using the new
  utility function breakRL. This avoids searching for the latest tag twice
  (first in misplacedPatches and then again in splitOnTag). The fused
  functionality is now in function D.P.Depends.cleanLatestTag.
  
  The test for darcs optimize has been enhanced to actually test that
optimize
  reorder does what it should.

Attachments

• simplify-d_r_hashed_reorderinventory.dpatch

Nice cleanup - the new code is very easy to read.

Did you intend to remove the debug messages? I don't think they were 
very important but I don't think they hurt either and you never know 
when they might be useful.

With the much shorter and simpler definition I guess debug messages just
felt a bit heavy-weight now. Feel free to re-add them, however.