"Avery Pennarun" <apenwarr@xxxxxxxxx> writes:
> This uses the low-level mechanism for "ours" and "theirs" autoresolution
> introduced by the previous commit to introduce two additional merge
> strategies, merge-recursive-ours and merge-recursive-theirs.
>
> (Patch originally by Junio Hamano <gitster@xxxxxxxxx>.)
>
> Signed-off-by: Avery Pennarun <apenwarr@xxxxxxxxx>
Two comments.
- The original series was done over a few weeks in 'pu' and this
intermediate step was done before a better alternative of not using
these two extra merge strategies were discovered ("...may have been an
easy way to experiment, but we should bite the bullet", in the next
patch).
As the second round to seriously polish the series for inclusion, it
would make much more sense to squash this with the next patch to erase
this failed approach that has already been shown as clearly inferiour.
- I think we should avoid adding the extra argument to ll_merge_fn() by
combining virtual_ancestor and favor into one "flags" parameter. If
you do so, we do not have to change the callsites again next time we
need to add new optional features that needs only a few bits.
I vaguely recall that I did the counterpart of this patch that way
exactly for the above reason, but it is more than a year ago, so maybe
I didn't do it that way.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html