widening: remove pointless code for second changegroup
authorMartin von Zweigbergk <martinvonz@google.com>
Thu, 10 Oct 2019 22:36:55 -0700
changeset 43245 c320887c8e1c
parent 43244 c5adf480de0f
child 43246 0fd47eb508c6
widening: remove pointless code for second changegroup The caller passes in common==heads, so we will never find any nodes to visit in this code and there will therefore never be a second changegroup emitted. Differential Revision: https://phab.mercurial-scm.org/D7095
hgext/narrow/narrowbundle2.py
--- a/hgext/narrow/narrowbundle2.py	Thu Oct 10 22:34:16 2019 -0700
+++ b/hgext/narrow/narrowbundle2.py	Thu Oct 10 22:36:55 2019 -0700
@@ -186,28 +186,6 @@
         if b'treemanifest' in repo.requirements:
             part.addparam(b'treemanifest', b'1')
 
-    visitnodes, relevant_nodes, ellipsisroots = exchange._computeellipsis(
-        repo, common, heads, set(), newmatch, depth=depth
-    )
-
-    repo.ui.debug(b'Found %d relevant revs\n' % len(relevant_nodes))
-    if visitnodes:
-        packer = changegroup.getbundler(
-            version,
-            repo,
-            matcher=newmatch,
-            ellipses=True,
-            shallow=depth is not None,
-            ellipsisroots=ellipsisroots,
-            fullnodes=relevant_nodes,
-        )
-        cgdata = packer.generate(common, visitnodes, False, b'narrow_widen')
-
-        part = bundler.newpart(b'changegroup', data=cgdata)
-        part.addparam(b'version', version)
-        if b'treemanifest' in repo.requirements:
-            part.addparam(b'treemanifest', b'1')
-
 
 @bundle2.parthandler(_SPECPART, (_SPECPART_INCLUDE, _SPECPART_EXCLUDE))
 def _handlechangespec_2(op, inpart):