# HG changeset patch # User Martin von Zweigbergk # Date 1570772215 25200 # Node ID c320887c8e1c359232f6687dbd70a1dd7ca6fc31 # Parent c5adf480de0f752f12be915e011757fbcfd0730e 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 diff -r c5adf480de0f -r c320887c8e1c 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):