rebase: delete unused p1 argument to _concludenode()
authorMartin von Zweigbergk <martinvonz@google.com>
Fri, 18 Sep 2020 10:15:13 -0700
changeset 45511 e29cd888fd17
parent 45510 1f5c548f15e5
child 45512 b4abfe142ff6
rebase: delete unused p1 argument to _concludenode() Unused since a0192a03216d (rebase: remove now unnecessary logic to allow empty commit when branch changes, 2020-07-09). Differential Revision: https://phab.mercurial-scm.org/D9055
hgext/rebase.py
--- a/hgext/rebase.py	Fri Sep 18 10:04:02 2020 -0700
+++ b/hgext/rebase.py	Fri Sep 18 10:15:13 2020 -0700
@@ -517,7 +517,7 @@
         p.complete()
         ui.note(_(b'rebase merging completed\n'))
 
-    def _concludenode(self, rev, p1, editor, commitmsg=None):
+    def _concludenode(self, rev, editor, commitmsg=None):
         '''Commit the wd changes with parents p1 and p2.
 
         Reuse commit info from rev but also store useful information in extra.
@@ -643,7 +643,7 @@
                 # parents, and we don't want to create a merge commit here (unless
                 # we're rebasing a merge commit).
                 self.wctx.setparents(repo[p1].node(), repo[p2].node())
-                newnode = self._concludenode(rev, p1, editor)
+                newnode = self._concludenode(rev, editor)
             else:
                 # Skip commit if we are collapsing
                 newnode = None
@@ -710,7 +710,7 @@
 
             self.wctx.setparents(repo[p1].node(), repo[self.external].node())
             newnode = self._concludenode(
-                revtoreuse, p1, editor, commitmsg=commitmsg
+                revtoreuse, editor, commitmsg=commitmsg
             )
 
             if newnode is not None: