Code

xdl_merge(): fix and simplify conflict handling
authorJohannes Schindelin <Johannes.Schindelin@gmx.de>
Tue, 5 Dec 2006 21:15:35 +0000 (22:15 +0100)
committerJunio C Hamano <junkio@cox.net>
Tue, 5 Dec 2006 21:30:22 +0000 (13:30 -0800)
commit98e6da8a360b77af2924e8056fd951013835699b
tree1748b105970be2c41177c15187e9536e301a56f5
parent710daa83fc76f79b8f2ee9a765d297187c2c1aeb
xdl_merge(): fix and simplify conflict handling

Suppose you have changes in new1 to the original lines 10-20,
and changes in new2 to the original lines 15-25, then the
changes to 10-25 conflict. But it is possible that the next
changes in new1 still overlap with this change to new2.

So, in the next iteration we have to look at the same change
to new2 again.

The old code tried to be a bit too clever. The new code is
shorter and more to the point: do not fiddle with the ranges
at all.

Also, xdl_append_merge() tries harder to combine conflicts.
This is necessary, because with the above simplification,
some conflicts would not be recognized as conflicts otherwise:

In the above scenario, it is possible that there is no other
change to new1. Absent the combine logic, the change in new2
would be recorded _again_, but as a non-conflict.

Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
xdiff/xmerge.c