Mercurial > public > mercurial-scm > hg-stable
comparison mercurial/debugcommands.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 | a8e1ff9ac257 |
children | 9746e618c151 |
comparison
equal
deleted
inserted
replaced
52104:513b413702e8 | 52105:a021da4ec509 |
---|---|
253 | 253 |
254 tags = [] | 254 tags = [] |
255 progress = ui.makeprogress( | 255 progress = ui.makeprogress( |
256 _(b'building'), unit=_(b'revisions'), total=total | 256 _(b'building'), unit=_(b'revisions'), total=total |
257 ) | 257 ) |
258 merge_relaxed_sync = ui.configbool( | |
259 b'experimental', | |
260 b'relaxed-block-sync-merge', | |
261 ) | |
258 with progress, repo.wlock(), repo.lock(), repo.transaction(b"builddag"): | 262 with progress, repo.wlock(), repo.lock(), repo.transaction(b"builddag"): |
259 at = -1 | 263 at = -1 |
260 atbranch = b'default' | 264 atbranch = b'default' |
261 nodeids = [] | 265 nodeids = [] |
262 id = 0 | 266 id = 0 |
277 p2 = repo[ps[1]] | 281 p2 = repo[ps[1]] |
278 pa = p1.ancestor(p2) | 282 pa = p1.ancestor(p2) |
279 base, local, other = [ | 283 base, local, other = [ |
280 x[fn].data() for x in (pa, p1, p2) | 284 x[fn].data() for x in (pa, p1, p2) |
281 ] | 285 ] |
282 m3 = simplemerge.Merge3Text(base, local, other) | 286 m3 = simplemerge.Merge3Text( |
287 base, | |
288 local, | |
289 other, | |
290 relaxed_sync=merge_relaxed_sync, | |
291 ) | |
283 ml = [ | 292 ml = [ |
284 l.strip() | 293 l.strip() |
285 for l in simplemerge.render_minimized(m3)[0] | 294 for l in simplemerge.render_minimized(m3)[0] |
286 ] | 295 ] |
287 ml.append(b"") | 296 ml.append(b"") |