# HG changeset patch
# User Martin von Zweigbergk <martinvonz@google.com>
# Date 1569447308 25200
# Node ID b4093d1d3b1861bce5fd3f7afc4cb5c33a00ba2e
# Parent  e3bb2a58af1e19dda1b6a31fd4a80ec87bc81311
update: clarify calculateupdate() call sites by specifying argument names

merge.calculateupdate() takes a lot of parameters and I get confused
all the time which is which.

See also b14fdf1fb615 (update: clarify update() call sites by
specifying argument names, 2017-02-09).

Differential Revision: https://phab.mercurial-scm.org/D6883

diff -r e3bb2a58af1e -r b4093d1d3b18 contrib/perf.py
--- a/contrib/perf.py	Wed Sep 25 17:57:16 2019 -0400
+++ b/contrib/perf.py	Wed Sep 25 14:35:08 2019 -0700
@@ -1030,8 +1030,9 @@
     def d():
         # acceptremote is True because we don't want prompts in the middle of
         # our benchmark
-        merge.calculateupdates(repo, wctx, rctx, [ancestor], False, False,
-                               acceptremote=True, followcopies=True)
+        merge.calculateupdates(repo, wctx, rctx, [ancestor], branchmerge=False,
+                               force=False, acceptremote=True,
+                               followcopies=True)
     timer(d)
     fm.end()
 
diff -r e3bb2a58af1e -r b4093d1d3b18 hgext/convert/hg.py
--- a/hgext/convert/hg.py	Wed Sep 25 17:57:16 2019 -0400
+++ b/hgext/convert/hg.py	Wed Sep 25 14:35:08 2019 -0700
@@ -204,12 +204,8 @@
         anc = [p1ctx.ancestor(p2ctx)]
         # Calculate what files are coming from p2
         actions, diverge, rename = mergemod.calculateupdates(
-            self.repo, p1ctx, p2ctx, anc,
-            True,  # branchmerge
-            True,  # force
-            False, # acceptremote
-            False, # followcopies
-        )
+            self.repo, p1ctx, p2ctx, anc, branchmerge=True,
+            force=True, acceptremote=False, followcopies=False)
 
         for file, (action, info, msg) in actions.iteritems():
             if source.targetfilebelongstosource(file):
diff -r e3bb2a58af1e -r b4093d1d3b18 hgext/histedit.py
--- a/hgext/histedit.py	Wed Sep 25 17:57:16 2019 -0400
+++ b/hgext/histedit.py	Wed Sep 25 14:35:08 2019 -0700
@@ -1986,11 +1986,8 @@
                 repo, wctx, c, ancs,
                 # These parameters were determined by print-debugging
                 # what happens later on inside histedit.
-                False, # branchmerge
-                False, # force
-                False, # acceptremote
-                False, # followcopies
-            )
+                branchmerge=False, force=False, acceptremote=False,
+                followcopies=False)
         except error.Abort:
             raise error.Abort(
        _("untracked files in working directory conflict with files in %s") % (