yuja added a comment.
> if ntiprev > self.tiprev: > self.tiprev = ntiprev > self.tipnode = cl.node(ntiprev) > > + self.filteredhash = scmutil.filteredhash(repo, self.tiprev) > > if not self.validfor(repo): > # cache key are not valid anymore and update `self.filteredhash` later again, which smells. Instead, shouldn't we check the validity first, and take the fast path only if it was valid? if self.validfor(repo): # was valid, can take fast path self.tiprev = ntiprev self.tipnode = cl.node(ntiprev) else: # bad luck, recompute tiprev/tipnode ... self.filteredhash = scmutil.filteredhash(repo, self.tiprev) REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D6312 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