[PATCH 08 of 10] branchmap: _updatebranchmap does not need to be filtered

pierre-yves.david at logilab.fr pierre-yves.david at logilab.fr
Wed Dec 19 07:53:24 CST 2012


# HG changeset patch
# User Pierre-Yves David <pierre-yves.david at logilab.fr>
# Date 1355924858 -3600
# Node ID 5281ce24eb459523cd86749149d0de662d5b586f
# Parent  2ca3a4c5731b87f7c544bc497c4477e9a572006a
branchmap: _updatebranchmap does not need to be filtered

The `_updatebranchmap` method on repo does not need to be filtered as all
callers are already handling filtering themself.

The fact it is filtered may had even lead to buggy behaviors, but by chances the method
make very sparse use of the repo object.

diff --git a/mercurial/localrepo.py b/mercurial/localrepo.py
--- a/mercurial/localrepo.py
+++ b/mercurial/localrepo.py
@@ -709,11 +709,10 @@ class localrepository(object):
         bt = {}
         for bn, heads in self.branchmap().iteritems():
             bt[bn] = self._branchtip(heads)
         return bt
 
-    @unfilteredmethod # Until we get a smarter cache management
     def _updatebranchcache(self, partial, ctxgen):
         """Given a branchhead cache, partial, that may have extra nodes or be
         missing heads, and a generator of nodes that are at least a superset of
         heads missing, this function updates partial to be correct.
         """


More information about the Mercurial-devel mailing list