# HG changeset patch # User Pierre-Yves David # Date 2019-03-06 10:38:54 # Node ID 567892b4306c49dbaff53a6b699d8079650a0345 # Parent e8c4a9f5b9862ff884ae97cc0ff29e4d9c04db41 verify: explicitly return 0 if no error are encountered Relying on the fact None is treated as 0 by other logics seems smarter than we should be. diff --git a/mercurial/verify.py b/mercurial/verify.py --- a/mercurial/verify.py +++ b/mercurial/verify.py @@ -134,7 +134,7 @@ class verifier(object): This method run all verifications, displaying issues as they are found. - return 1 if any error have been encountered""" + return 1 if any error have been encountered, 0 otherwise.""" repo = self.repo ui = repo.ui @@ -171,6 +171,7 @@ class verifier(object): ui.warn(_("(first damaged changeset appears to be %d)\n") % min(self.badrevs)) return 1 + return 0 def _verifychangelog(self): ui = self.ui