Mercurial > hg
changeset 17720:9a1796af4f8d
verify: rename "hasmanifest" variable for source code readability
Before this patch, there are two ambiguous variables: "havemf" and
"hasmanifest".
"havemf" means whether there are any "manifest" entries.
"hasmanifest" means whether there are any "changelog" entries
referring to "manifest" entry.
This patch renames from "hasmanifest" to "refersmf" to clear
difference from "havemf".
author | FUJIWARA Katsunori <foozy@lares.dti.ne.jp> |
---|---|
date | Thu, 04 Oct 2012 01:24:05 +0900 |
parents | 2e3ceb59c312 |
children | cf236e3501c3 |
files | mercurial/verify.py |
diffstat | 1 files changed, 4 insertions(+), 4 deletions(-) [+] |
line wrap: on
line diff
--- a/mercurial/verify.py Thu Oct 04 01:24:05 2012 +0900 +++ b/mercurial/verify.py Thu Oct 04 01:24:05 2012 +0900 @@ -120,7 +120,7 @@ havemf = len(mf) > 0 ui.status(_("checking changesets\n")) - hasmanifest = False + refersmf = False seen = {} checklog(cl, "changelog", 0) total = len(repo) @@ -133,17 +133,17 @@ changes = cl.read(n) if changes[0] != nullid: mflinkrevs.setdefault(changes[0], []).append(i) - hasmanifest = True + refersmf = True for f in changes[3]: filelinkrevs.setdefault(f, []).append(i) except Exception, inst: - hasmanifest = True + refersmf = True exc(i, _("unpacking changeset %s") % short(n), inst) ui.progress(_('checking'), None) ui.status(_("checking manifests\n")) seen = {} - if hasmanifest: + if refersmf: # Do not check manifest if there are only changelog entries with # null manifests. checklog(mf, "manifest", 0)