yuja added a comment.

  > +        if self._namesreverse is None:
  >  +            self._namesreverse = dict((b, r) for r, b in 
enumerate(self._names))
  > 
  >   if b in self._namesreverse:
  >       branchidx = self._namesreverse[b]
  >   else:
  > 
  > @@ -467,6 +469,8 @@
  > 
  >   def setdata(self, branch, rev, node, close):
  >       """add new data information to the cache"""
  > 
  > +        if self._namesreverse is None:
  >  +            self._namesreverse = dict((b, r) for r, b in 
enumerate(self._names))
  
  Maybe _namesreverse can be a propertycache, instead of duplicating it.
  `del self.__dict__[r'...']` can be used to discard the cache.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D5291

To: pulkit, #hg-reviewers
Cc: yuja, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to