comparison mercurial/simplemerge.py @ 48555:c91418480cb0

simplemerge: use 3-way markers if mode=='merge3', ignoring number of labels Before this patch, we use a combination of `mode` and `labels` to decide which marker style to use. That's mostly my own fault because I was the one who added the `mergediff` as a "mode" instead of adding a separate argument for marker style. Still, let's make these arguments more orthogonal by ignoring the number of labels when deciding which marker style to use. This refactoring makes it a little clearer in the simplemerge *extension* that passing more than 2 `--label` arguments changes the marker style. Differential Revision: https://phab.mercurial-scm.org/D11972
author Martin von Zweigbergk <martinvonz@google.com>
date Tue, 14 Dec 2021 13:12:56 -0800
parents 88a45330b290
children ce8c82a5cd65
comparison
equal deleted inserted replaced
48554:0dc698c91ca0 48555:c91418480cb0
504 lines, conflicts = _mergediff(m3, name_a, name_b, name_base) 504 lines, conflicts = _mergediff(m3, name_a, name_b, name_base)
505 else: 505 else:
506 extrakwargs = { 506 extrakwargs = {
507 'minimize': True, 507 'minimize': True,
508 } 508 }
509 if name_base is not None: 509 if mode == b'merge3':
510 extrakwargs['base_marker'] = b'|||||||' 510 extrakwargs['base_marker'] = b'|||||||'
511 extrakwargs['name_base'] = name_base 511 extrakwargs['name_base'] = name_base
512 extrakwargs['minimize'] = False 512 extrakwargs['minimize'] = False
513 lines = list( 513 lines = list(
514 m3.merge_lines(name_a=name_a, name_b=name_b, **extrakwargs) 514 m3.merge_lines(name_a=name_a, name_b=name_b, **extrakwargs)