[PATCH 2 of 4 V4] extdiff: rename the name of an argument for readability
FUJIWARA Katsunori
foozy at lares.dti.ne.jp
Thu Dec 25 08:46:15 CST 2014
# HG changeset patch
# User FUJIWARA Katsunori <foozy at lares.dti.ne.jp>
# Date 1419518006 -32400
# Thu Dec 25 23:33:26 2014 +0900
# Node ID 52b0d3f6ae99ffee8b87e487a34ae41bbc63821d
# Parent 9daebd666dddc248f69230dca34ff21585e33421
extdiff: rename the name of an argument for readability
To reduce amount of changes for review-ability, previous patch uses
"args" as argument name of "dodiff()", even though "args" includes
also the name of command to be executed (or full-path of it).
This patch replaces "args" by more appropriate name "cmdline".
diff --git a/hgext/extdiff.py b/hgext/extdiff.py
--- a/hgext/extdiff.py
+++ b/hgext/extdiff.py
@@ -109,7 +109,7 @@ def snapshot(ui, repo, files, node, tmpr
os.lstat(dest).st_mtime))
return dirname, fns_and_mtime
-def dodiff(ui, repo, args, pats, opts):
+def dodiff(ui, repo, cmdline, pats, opts):
'''Do the actual diff:
- copy to a temp structure if diffing 2 internal revisions
@@ -120,7 +120,7 @@ def dodiff(ui, repo, args, pats, opts):
revs = opts.get('rev')
change = opts.get('change')
- do3way = '$parent2' in args
+ do3way = '$parent2' in cmdline
if revs and change:
msg = _('cannot specify --rev and --change at the same time')
@@ -219,9 +219,9 @@ def dodiff(ui, repo, args, pats, opts):
# Match parent2 first, so 'parent1?' will match both parent1 and parent
regex = '\$(parent2|parent1?|child|plabel1|plabel2|clabel|root)'
- if not do3way and not re.search(regex, args):
- args += ' $parent1 $child'
- cmdline = re.sub(regex, quote, args)
+ if not do3way and not re.search(regex, cmdline):
+ cmdline += ' $parent1 $child'
+ cmdline = re.sub(regex, quote, cmdline)
ui.debug('running %r in %s\n' % (cmdline, tmproot))
ui.system(cmdline, cwd=tmproot)
More information about the Mercurial-devel
mailing list