# HG changeset patch # User Pierre-Yves David # Date 1551868734 -3600 # 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 -r e8c4a9f5b986 -r 567892b4306c mercurial/verify.py --- a/mercurial/verify.py Wed Mar 06 11:35:34 2019 +0100 +++ b/mercurial/verify.py Wed Mar 06 11:38:54 2019 +0100 @@ -134,7 +134,7 @@ 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 @@ ui.warn(_("(first damaged changeset appears to be %d)\n") % min(self.badrevs)) return 1 + return 0 def _verifychangelog(self): ui = self.ui