[PATCH 02 of 10] branchmap: factorise access to changelog in updatebranchcache

[Pierre-Yves David]

On Wed, Dec 19, 2012 at 03:11:04PM +0100, Laurens Holst wrote:
> Op 19-12-12 14:53, pierre-yves.david at logilab.fr schreef:
> ># HG changeset patch
> ># User Pierre-Yves David <pierre-yves.david at logilab.fr>
> ># Date 1355916586 -3600
> ># Node ID f8e00dbbb6bb0c871278d61fb979179162d3cdd4
> ># Parent  56c016dea0622854146f52acb9b3d5c9258636b9
> >branchmap: factorise access to changelog in updatebranchcache
> >
> >This prepares merge of `updatebranchcache` and `_branchtags`.
> >
> >diff --git a/mercurial/localrepo.py b/mercurial/localrepo.py
> >--- a/mercurial/localrepo.py
> >+++ b/mercurial/localrepo.py
> >@@ -662,20 +662,21 @@ class localrepository(object):
> >              self._updatebranchcache(partial, ctxgen)
> >          return partial
> >      @unfilteredmethod # Until we get a smarter cache management
> >      def updatebranchcache(self):
> >-        tip = self.changelog.tip()
> >+        cl = self.changelog
> >+        tip = cl.tip()
> >          if self._branchcache is not None and self._branchcachetip == tip:
> >              return
> >          oldtip = self._branchcachetip
> >          self._branchcachetip = tip
> >-        if oldtip is None or oldtip not in self.changelog.nodemap:
> >+        if oldtip is None or oldtip not in cl.nodemap:
> >              partial, last, lrev = self._readbranchcache()
> >          else:
> >-            lrev = self.changelog.rev(oldtip)
> >+            lrev = self.cl.rev(oldtip)
> 
> As cl is a local variable, remove self. ?

oops, this is silently fixed by the next patch and I did not caught it when running the tests.

fixed version pullable from here:

hg pull -r d68f836a210f http://hg-lab.logilab.org/wip/hg/

-- 
Pierre-Yves David

http://www.logilab.fr/

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20121219/e552aa48/attachment.pgp>