# HG changeset patch
# User Martin von Zweigbergk <martinvonz@google.com>
# Date 2015-04-29 17:34:28
# Node ID 147d8892fc4b5f041d1688cff51ae3808ef54efc
# Parent  89c2bf63a83b0ab6caa0c3a2d448e8f650fb5f1e

changegroup: extract condition for linkrev fastpath

The condition for taking the fastpath (or not) is used in two
places. By extracting it, we also provide a place to document what
it's about.

diff --git a/mercurial/changegroup.py b/mercurial/changegroup.py
--- a/mercurial/changegroup.py
+++ b/mercurial/changegroup.py
@@ -361,7 +361,6 @@ class cg1packer(object):
         repo = self._repo
         cl = self._changelog
         ml = self._manifest
-        reorder = self._reorder
 
         clrevorder = {}
         mfs = {} # needed manifests
@@ -386,12 +385,13 @@ class cg1packer(object):
             yield chunk
         self._verbosenote(_('%8.i (changelog)\n') % size)
 
+        fastpathlinkrev = fastpathlinkrev and not self._reorder
         # Callback for the manifest, used to collect linkrevs for filelog
         # revisions.
         # Returns the linkrev node (collected in lookupcl).
         def lookupmf(x):
             clnode = mfs[x]
-            if not fastpathlinkrev or reorder:
+            if not fastpathlinkrev:
                 mdata = ml.readfast(x)
                 for f, n in mdata.iteritems():
                     if f in changedfiles:
@@ -414,7 +414,7 @@ class cg1packer(object):
         clrevs = set(cl.rev(x) for x in clnodes)
 
         def linknodes(filerevlog, fname):
-            if fastpathlinkrev and not reorder:
+            if fastpathlinkrev:
                 llr = filerevlog.linkrev
                 def genfilenodes():
                     for r in filerevlog: