Mercurial > public > mercurial-scm > hg-stable
comparison mercurial/filemerge.py @ 52105:a021da4ec509
merge: add a config to allow conflict-free merge of changes on adjacent lines
This change adds a config to make it no longer a conflict to merge changes
made on adjacent lines.
The reason these changes are considered a conflict is that there's no
region of text at the relevant position (sync region) that's kept unchanged
by both sides of the merge.
The problem can be solved by making the sync regions being a bit more
powerful: we can keep a 0-length sync region if we find that
a block unchanged by one side is ajacent to a block unchanged by the
other side.
Since these 0-length sync regions are emitted using the ~same algorithm
as the normal non-empty sync regions, this change involves no arbitrary
decisions and I expect it to work pretty well.
0-length sync regions do create an ambiguity in a special case where two
pairs of adjacent regions "meet" at the same point. This corresponds to
an insertion made at the same place by the two sides of the merge, and
this still results in a conflict.
author | Arseniy Alekseyev <aalekseyev@janestreet.com |
---|---|
date | Thu, 24 Oct 2024 17:35:53 +0200 |
parents | f4733654f144 |
children |
comparison
equal
deleted
inserted
replaced
52104:513b413702e8 | 52105:a021da4ec509 |
---|---|
479 the partially merged file. Markers will have two sections, one for each | 479 the partially merged file. Markers will have two sections, one for each |
480 side of merge, unless mode equals 'union' or 'union-other-first' which | 480 side of merge, unless mode equals 'union' or 'union-other-first' which |
481 suppresses the markers.""" | 481 suppresses the markers.""" |
482 ui = repo.ui | 482 ui = repo.ui |
483 | 483 |
484 relaxed_sync = ui.configbool(b'experimental', b'relaxed-block-sync-merge') | |
485 | |
484 try: | 486 try: |
485 _verifytext(local, ui) | 487 _verifytext(local, ui) |
486 _verifytext(base, ui) | 488 _verifytext(base, ui) |
487 _verifytext(other, ui) | 489 _verifytext(other, ui) |
488 except error.Abort: | 490 except error.Abort: |
489 return True, True, False | 491 return True, True, False |
490 else: | 492 else: |
491 merged_text, conflicts = simplemerge.simplemerge( | 493 merged_text, conflicts = simplemerge.simplemerge( |
492 local, base, other, mode=mode | 494 local, |
495 base, | |
496 other, | |
497 mode=mode, | |
498 relaxed_sync=relaxed_sync, | |
493 ) | 499 ) |
494 # fcd.flags() already has the merged flags (done in | 500 # fcd.flags() already has the merged flags (done in |
495 # mergestate.resolve()) | 501 # mergestate.resolve()) |
496 local.fctx.write(merged_text, local.fctx.flags()) | 502 local.fctx.write(merged_text, local.fctx.flags()) |
497 return True, conflicts, False | 503 return True, conflicts, False |