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