[PATCH 5 of 6] changegroup: don't reuse 'mfest' variable for different type

Martin von Zweigbergk martinvonz at google.com
Fri May 1 17:50:43 CDT 2015


# HG changeset patch
# User Martin von Zweigbergk <martinvonz at google.com>
# Date 1430241664 25200
#      Tue Apr 28 10:21:04 2015 -0700
# Node ID 936eff6f83d7b97f7dffe0373980cf26aecbc645
# Parent  bb89e35096d2792b16f102cad8f05a0e3b8534e1
changegroup: don't reuse 'mfest' variable for different type

We have a variable 'mfest' that's first a manifest nodeid and then a
manifest. Let's make it clearer by using separate variables for the
two uses.

diff -r bb89e35096d2 -r 936eff6f83d7 mercurial/changegroup.py
--- a/mercurial/changegroup.py	Tue Apr 28 10:19:42 2015 -0700
+++ b/mercurial/changegroup.py	Tue Apr 28 10:21:04 2015 -0700
@@ -789,8 +789,8 @@
         if repo.ui.configbool('server', 'validate', default=False):
             # validate incoming csets have their manifests
             for cset in xrange(clstart, clend):
-                mfest = repo.changelog.read(repo.changelog.node(cset))[0]
-                mfest = repo.manifest.readdelta(mfest)
+                mfnode = repo.changelog.read(repo.changelog.node(cset))[0]
+                mfest = repo.manifest.readdelta(mfnode)
                 # store file nodes we must see
                 for f, n in mfest.iteritems():
                     needfiles.setdefault(f, set()).add(n)


More information about the Mercurial-devel mailing list