mdiff: add a config option to use xdiff algorithm
authorJun Wu <quark@fb.com>
Sat, 03 Mar 2018 12:39:14 -0800
changeset 36705 c6a61298ac32
parent 36704 430fdb717549
child 36706 a247a0e82e7d
mdiff: add a config option to use xdiff algorithm The `experimental.xdiff` will affect the default diffopts and make mdiff use the xdiff algorithm for better diff quality. Differential Revision: https://phab.mercurial-scm.org/D2603
mercurial/configitems.py
mercurial/mdiff.py
mercurial/patch.py
--- a/mercurial/configitems.py	Sat Mar 03 12:39:14 2018 -0800
+++ b/mercurial/configitems.py	Sat Mar 03 12:39:14 2018 -0800
@@ -577,6 +577,9 @@
 coreconfigitem('experimental', 'sshpeer.advertise-v2',
     default=False,
 )
+coreconfigitem('experimental', 'xdiff',
+    default=False,
+)
 coreconfigitem('extensions', '.*',
     default=None,
     generic=True,
--- a/mercurial/mdiff.py	Sat Mar 03 12:39:14 2018 -0800
+++ b/mercurial/mdiff.py	Sat Mar 03 12:39:14 2018 -0800
@@ -63,6 +63,7 @@
         'upgrade': False,
         'showsimilarity': False,
         'worddiff': False,
+        'xdiff': False,
         }
 
     def __init__(self, **opts):
@@ -188,6 +189,13 @@
         raise error.Abort(_('line range exceeds file size'))
     return filteredblocks, (lba, uba)
 
+def chooseblocksfunc(opts=None):
+    if (opts is None or not opts.xdiff
+        or not util.safehasattr(bdiff, 'xdiffblocks')):
+        return bdiff.blocks
+    else:
+        return bdiff.xdiffblocks
+
 def allblocks(text1, text2, opts=None, lines1=None, lines2=None):
     """Return (block, type) tuples, where block is an mdiff.blocks
     line entry. type is '=' for blocks matching exactly one another
@@ -201,7 +209,7 @@
     if opts.ignorews or opts.ignorewsamount or opts.ignorewseol:
         text1 = wsclean(opts, text1, False)
         text2 = wsclean(opts, text2, False)
-    diff = bdiff.blocks(text1, text2)
+    diff = chooseblocksfunc(opts)(text1, text2)
     for i, s1 in enumerate(diff):
         # The first match is special.
         # we've either found a match starting at line 0 or a match later
--- a/mercurial/patch.py	Sat Mar 03 12:39:14 2018 -0800
+++ b/mercurial/patch.py	Sat Mar 03 12:39:14 2018 -0800
@@ -2256,6 +2256,7 @@
         'context': get('unified', getter=ui.config),
     }
     buildopts['worddiff'] = ui.configbool('experimental', 'worddiff')
+    buildopts['xdiff'] = ui.configbool('experimental', 'xdiff')
 
     if git:
         buildopts['git'] = get('git')