D6240: branchcache: only iterate over branches which needs to be verified
This revision was automatically updated to reflect the committed changes. Closed by commit rHG75e8e16ca107: branchcache: only iterate over branches which needs to be verified (authored by pulkit, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D6240?vs=14758&id=14764 REVISION DETAIL https://phab.mercurial-scm.org/D6240 AFFECTED FILES mercurial/branchmap.py CHANGE DETAILS diff --git a/mercurial/branchmap.py b/mercurial/branchmap.py --- a/mercurial/branchmap.py +++ b/mercurial/branchmap.py @@ -200,7 +200,8 @@ def _verifyall(self): """ verifies nodes of all the branches """ -for b in self._entries: +needverification = set(self._entries.keys()) - self._verifiedbranches +for b in needverification: self._verifybranch(b) def __iter__(self): To: pulkit, #hg-reviewers Cc: mercurial-devel ___ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
D6240: branchcache: only iterate over branches which needs to be verified
pulkit created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY Otherwise we loop over all the branches and call _verifybranch() even if not required. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D6240 AFFECTED FILES mercurial/branchmap.py CHANGE DETAILS diff --git a/mercurial/branchmap.py b/mercurial/branchmap.py --- a/mercurial/branchmap.py +++ b/mercurial/branchmap.py @@ -200,7 +200,8 @@ def _verifyall(self): """ verifies nodes of all the branches """ -for b in self._entries: +needverification = set(self._entries.keys()) - self._verifiedbranches +for b in needverification: self._verifybranch(b) def __iter__(self): To: pulkit, #hg-reviewers Cc: mercurial-devel ___ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel