[darcs-devel] darcs patch: (add a + mv a b = add b) and (mv a b + remove b = remo...

[David Roundy]

On Thu, Jan 11, 2007 at 08:18:56PM +0100, Juliusz Chroboczek wrote:
> > I think this is probably fine, but it's a little bit scary.  The reason is
> > that I this change does affect the semantics of a patch.  It's probably
> > fine, since I think coalesce is only used on pending changes,
> 
> According to some guy called David, it is ;-)
> 
>   http://darcs.net/pipermail/darcs-devel/2006-January/003849.html
> 
> > but I'd like to see a bit of a review of where coalesce is used,
> 
> So would I.

We'd both like it, but I think we could put this change in before the
review is done.  It only has effects in rare corner cases, so even
if it were a bug (which I doubt), it'd be one with little effect.
-- 
David Roundy
Department of Physics
Oregon State University