verify: replace "output parameters" by return values
authorMartin von Zweigbergk <martinvonz@google.com>
Tue, 05 Jan 2016 21:25:51 -0800
changeset 27695 fb0cc863d172
parent 27694 2dc363274702
child 27696 e70c97cc9243
verify: replace "output parameters" by return values _verifychangelog() and _verifymanifest() accept dictionaries that they populate. We pass in empty dictionaries, so it's clearer to create them in the functions and return them.
mercurial/verify.py
--- a/mercurial/verify.py	Tue Jan 05 22:23:27 2016 +0000
+++ b/mercurial/verify.py	Tue Jan 05 21:25:51 2016 -0800
@@ -134,9 +134,6 @@
 
     def verify(self):
         repo = self.repo
-        mflinkrevs = {}
-        filelinkrevs = {}
-        filenodes = {}
 
         ui = repo.ui
 
@@ -150,9 +147,9 @@
             ui.status(_("repository uses revlog format %d\n") %
                            (self.revlogv1 and 1 or 0))
 
-        self._verifychangelog(mflinkrevs, filelinkrevs)
+        mflinkrevs, filelinkrevs = self._verifychangelog()
 
-        self._verifymanifest(mflinkrevs, filenodes)
+        filenodes = self._verifymanifest(mflinkrevs)
 
         self._crosscheckfiles(mflinkrevs, filelinkrevs, filenodes)
 
@@ -172,12 +169,14 @@
                         % min(self.badrevs))
             return 1
 
-    def _verifychangelog(self, mflinkrevs, filelinkrevs):
+    def _verifychangelog(self):
         ui = self.ui
         repo = self.repo
         cl = repo.changelog
 
         ui.status(_("checking changesets\n"))
+        mflinkrevs = {}
+        filelinkrevs = {}
         seen = {}
         self.checklog(cl, "changelog", 0)
         total = len(repo)
@@ -198,13 +197,15 @@
                 self.refersmf = True
                 self.exc(i, _("unpacking changeset %s") % short(n), inst)
         ui.progress(_('checking'), None)
+        return mflinkrevs, filelinkrevs
 
-    def _verifymanifest(self, mflinkrevs, filenodes):
+    def _verifymanifest(self, mflinkrevs):
         repo = self.repo
         ui = self.ui
         mf = self.repo.manifest
 
         ui.status(_("checking manifests\n"))
+        filenodes = {}
         seen = {}
         if self.refersmf:
             # Do not check manifest if there are only changelog entries with
@@ -233,7 +234,7 @@
                 self.exc(lr, _("reading manifest delta %s") % short(n), inst)
         ui.progress(_('checking'), None)
 
-        return mflinkrevs
+        return filenodes
 
     def _crosscheckfiles(self, mflinkrevs, filelinkrevs, filenodes):
         repo = self.repo