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

[Tommy Pettersson]

On Sat, Jan 20, 2007 at 04:36:09PM -0800, David Roundy wrote:
> On Sat, Jan 20, 2007 at 09:27:34PM +0100, Eric Y. Kow wrote:
> > I commented out coalesce and the things which depended on it.  The
> > casualties:
> 
> Thanks for checking this out!
> 
> >   commute_split
> >   unravel
> >   merger_equivalent
> >   reorder_and_coalesce (used in Repository.get_unrecorded)
> >   canonize             (used in AmendRecord and Diff)
> >   try_to_shrink        (used in DarcsRepo.sift_for_pending)
> > 
> > Are you sure this is ok? (Sorry for the delays, Malebria)
> 
> Yeah, that's okay.  I'm sure.

Maybe I'm a bit late to report, but I've come to the same
conclusion. I was unsure about 'unravel' and
'merger_equivalent'. I have checked that they are only called
for mergers, although I don't understand why mergers is ok.
Well, now it's sort of triple-checked. :-)



-- 
Tommy Pettersson <ptp at lysator.liu.se>