verify: move warn() to a class level function
authorDurham Goode <durham@fb.com>
Fri, 18 Dec 2015 16:42:39 -0800
changeset 27446 6b2c1a1871a6
parent 27445 cc178057ab49
child 27447 d1b91c10ce70
verify: move warn() to a class level function This is part of the effort to make verify more modular so extensions can hook into it more easily.
mercurial/verify.py
--- a/mercurial/verify.py	Fri Dec 18 16:42:39 2015 -0800
+++ b/mercurial/verify.py	Fri Dec 18 16:42:39 2015 -0800
@@ -60,6 +60,10 @@
         self.refersmf = False
         self.fncachewarned = False
 
+    def warn(self, msg):
+        self.ui.warn(msg + "\n")
+        self.warnings[0] += 1
+
     def verify(self):
         repo = self.repo
         mflinkrevs = {}
@@ -96,9 +100,6 @@
                 inst = repr(inst)
             err(linkrev, "%s: %s" % (msg, inst), filename)
 
-        def warn(msg):
-            ui.warn(msg + "\n")
-            warnings[0] += 1
 
         def checklog(obj, name, linkrev):
             if not len(obj) and (havecl or havemf):
@@ -113,9 +114,9 @@
 
             if obj.version != revlog.REVLOGV0:
                 if not revlogv1:
-                    warn(_("warning: `%s' uses revlog format 1") % name)
+                    self.warn(_("warning: `%s' uses revlog format 1") % name)
             elif revlogv1:
-                warn(_("warning: `%s' uses revlog format 0") % name)
+                self.warn(_("warning: `%s' uses revlog format 0") % name)
 
         def checkentry(obj, i, node, seen, linkrevs, f):
             lr = obj.linkrev(obj.rev(node))
@@ -133,7 +134,8 @@
                                         if lrugetctx(l)[f].filenode() == node]
                         except Exception:
                             pass
-                    warn(_(" (expected %s)") % " ".join(map(str, linkrevs)))
+                    self.warn(_(" (expected %s)") %
+                              " ".join(map(str, linkrevs)))
                 lr = None # can't be trusted
 
             try:
@@ -282,7 +284,7 @@
                 try:
                     storefiles.remove(ff)
                 except KeyError:
-                    warn(_(" warning: revlog '%s' not in fncache!") % ff)
+                    self.warn(_(" warning: revlog '%s' not in fncache!") % ff)
                     self.fncachewarned = True
 
             checklog(fl, f, lr)
@@ -324,8 +326,8 @@
                                     found = True
                                     break
                             if not found:
-                                warn(_("warning: copy source of '%s' not"
-                                       " in parents of %s") % (f, ctx))
+                                self.warn(_("warning: copy source of '%s' not"
+                                            " in parents of %s") % (f, ctx))
                         fl2 = repo.file(rp[0])
                         if not len(fl2):
                             err(lr, _("empty or missing copy source revlog "
@@ -347,7 +349,7 @@
         ui.progress(_('checking'), None)
 
         for f in storefiles:
-            warn(_("warning: orphan revlog '%s'") % f)
+            self.warn(_("warning: orphan revlog '%s'") % f)
 
         ui.status(_("%d files, %d changesets, %d total revisions\n") %
                        (len(files), len(cl), revisions))