filemerge: give some variables in _xmerge more descriptive names
authorKyle Lippincott <spectral@google.com>
Fri, 19 Jan 2018 18:52:50 -0800
changeset 36976 a4a95bd7158d
parent 36975 795eb53f1d3e
child 36977 a82fc3922446
filemerge: give some variables in _xmerge more descriptive names Differential Revision: https://phab.mercurial-scm.org/D2886
mercurial/filemerge.py
--- a/mercurial/filemerge.py	Thu Mar 15 13:19:55 2018 -0700
+++ b/mercurial/filemerge.py	Fri Jan 19 18:52:50 2018 -0800
@@ -509,10 +509,10 @@
                        'for %s\n') % (tool, fcd.path()))
         return False, 1, None
     unused, unused, unused, back = files
-    a = _workingpath(repo, fcd)
-    b, c = _maketempfiles(repo, fco, fca)
+    localpath = _workingpath(repo, fcd)
+    basepath, otherpath = _maketempfiles(repo, fco, fca)
     try:
-        out = ""
+        outpath = ""
         mylabel, otherlabel = labels[:2]
         if len(labels) >= 3:
             baselabel = labels[2]
@@ -534,11 +534,11 @@
         args = _toolstr(ui, tool, "args")
         if "$output" in args:
             # read input from backup, write to original
-            out = a
-            a = repo.wvfs.join(back.path())
-        replace = {'local': a, 'base': b, 'other': c, 'output': out,
-                   'labellocal': mylabel, 'labelother': otherlabel,
-                   'labelbase': baselabel}
+            outpath = localpath
+            localpath = repo.wvfs.join(back.path())
+        replace = {'local': localpath, 'base': basepath, 'other': otherpath,
+                   'output': outpath, 'labellocal': mylabel,
+                   'labelother': otherlabel, 'labelbase': baselabel}
         args = util.interpolate(br'\$', replace, args,
                                 lambda s: util.shellquote(util.localpath(s)))
         cmd = toolpath + ' ' + args
@@ -550,8 +550,8 @@
         repo.ui.debug('merge tool returned: %d\n' % r)
         return True, r, False
     finally:
-        util.unlink(b)
-        util.unlink(c)
+        util.unlink(basepath)
+        util.unlink(otherpath)
 
 def _formatconflictmarker(ctx, template, label, pad):
     """Applies the given template to the ctx, prefixed by the label.