[PATCH 10 of 10] bundle: inline applybundle1()

Martin von Zweigbergk martinvonz at google.com
Sat Jun 24 11:38:38 EDT 2017


# HG changeset patch
# User Martin von Zweigbergk <martinvonz at google.com>
# Date 1498193132 25200
#      Thu Jun 22 21:45:32 2017 -0700
# Node ID f0b522fb7bdbfa26c7504afb6c2516ddad065878
# Parent  a5adc63a7b001cc4c489e7ea4d574e71adccafe7
bundle: inline applybundle1()

We have now gotten rid of all but one caller, so let's inline it
there.

diff --git a/mercurial/bundle2.py b/mercurial/bundle2.py
--- a/mercurial/bundle2.py
+++ b/mercurial/bundle2.py
@@ -310,12 +310,6 @@
     to be created"""
     raise TransactionUnavailable()
 
-def applybundle1(repo, cg, tr, source, url, **kwargs):
-    # the transactiongetter won't be used, but we might as well set it
-    op = bundleoperation(repo, lambda: tr)
-    _processchangegroup(op, cg, tr, source, url, **kwargs)
-    return op
-
 def applybundle(repo, unbundler, tr, source=None, url=None, **kwargs):
     # transform me into unbundler.apply() as soon as the freeze is lifted
     if isinstance(unbundler, unbundle20):
@@ -326,7 +320,10 @@
             tr.hookargs['url'] = url
         return processbundle(repo, unbundler, lambda: tr)
     else:
-        return applybundle1(repo, unbundler, tr, source, url, **kwargs)
+        # the transactiongetter won't be used, but we might as well set it
+        op = bundleoperation(repo, lambda: tr)
+        _processchangegroup(op, unbundler, tr, source, url, **kwargs)
+        return op
 
 def processbundle(repo, unbundler, transactiongetter=None, op=None):
     """This function process a bundle, apply effect to/from a repo


More information about the Mercurial-devel mailing list