changeset 38905:379d90327861

changegroup: move changelogdone into cgpacker Looking at what it is used for, it feels like there is a better way to implement all this. So recording a TODO to track that. Differential Revision: https://phab.mercurial-scm.org/D4087
author Gregory Szorc <gregory.szorc@gmail.com>
date Fri, 03 Aug 2018 13:01:53 -0700
parents cdb9bc216771
children 0548f696795b
files mercurial/changegroup.py
diffstat 1 files changed, 7 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/changegroup.py	Fri Aug 03 12:57:11 2018 -0700
+++ b/mercurial/changegroup.py	Fri Aug 03 13:01:53 2018 -0700
@@ -579,13 +579,17 @@
         else:
             self._verbosenote = lambda s: None
 
+        # TODO the functionality keyed off of this should probably be
+        # controlled via arguments to group() that influence behavior.
+        self._changelogdone = False
+
     def _close(self):
         # Ellipses serving mode.
         getattr(self, '_clrev_to_localrev', {}).clear()
         if getattr(self, '_next_clrev_to_localrev', {}):
             self._clrev_to_localrev = self._next_clrev_to_localrev
             del self._next_clrev_to_localrev
-        self._changelog_done = True
+        self._changelogdone = True
 
         return closechunk()
 
@@ -1058,7 +1062,7 @@
     def _revisiondeltanarrow(self, store, rev, prev, linknode):
         # build up some mapping information that's useful later. See
         # the local() nested function below.
-        if not self._changelog_done:
+        if not self._changelogdone:
             self._clnode_to_rev[linknode] = rev
             linkrev = rev
             self._clrev_to_localrev[linkrev] = rev
@@ -1090,7 +1094,7 @@
             if clrev == nullrev:
                 return nullrev
 
-            if not self._changelog_done:
+            if not self._changelogdone:
                 # If we're doing the changelog, it's possible that we
                 # have a parent that is already on the client, and we
                 # need to store some extra mapping information so that
@@ -1403,6 +1407,5 @@
     # Maps changelog nodes to changelog revs. Filled in once
     # during changelog stage and then left unmodified.
     packer._clnode_to_rev = {}
-    packer._changelog_done = False
 
     return packer.generate(common, visitnodes, False, source)