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