This is an archive of the discontinued Mercurial Phabricator instance.

Differential D7494

localrepo: fastpath access to "."
ClosedPublic

Authored by marmoute on Nov 22 2019, 4:21 AM.

Details

Reviewers
pulkit
Group Reviewers
hg-reviewers
Commits
rHGd86dede17392: localrepo: fastpath access to "."
Summary

"." is just an aliast for p1(wdir(), let us handle it that way.

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

marmoute created this revision.Nov 22 2019, 4:21 AM
marmoute updated this revision to Diff 18359.Nov 23 2019, 8:04 AM
marmoute updated this revision to Diff 18383.Nov 24 2019, 6:26 AM
Alphare updated this revision to Diff 19362.Jan 16 2020, 9:24 AM
pulkit added a subscriber: pulkit.Jan 17 2020, 9:58 AM

"." is just an aliast for p1(wdir(),

"." is just an alias for p1(wdir()),

in flight.

pulkit accepted this revision.Jan 17 2020, 9:58 AM
This revision is now accepted and ready to land.Jan 17 2020, 9:58 AM
marmoute added a commit: rHGd86dede17392: localrepo: fastpath access to ".".Jan 17 2020, 10:13 AM
Closed by commit rHGd86dede17392: localrepo: fastpath access to "." (authored by marmoute). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mmercurial/localrepo.py (3 lines)

Diff 18383

mercurial/localrepo.py

if rev is None: if rev is None:
# unknown working copy parent case: # unknown working copy parent case:
# #
# skip the fast path and let higher code deal with it # skip the fast path and let higher code deal with it
continue continue
pair = (rev, node) pair = (rev, node)
quick[rev] = pair quick[rev] = pair
quick[node] = pair quick[node] = pair
p1node = self.dirstate.p1()
if p1node != nullid:
quick[b'.'] = quick[p1node]
return quick return quick
@unfilteredmethod @unfilteredmethod
def _quick_access_changeid_invalidate(self): def _quick_access_changeid_invalidate(self):
if '_quick_access_changeid_wc' in vars(self): if '_quick_access_changeid_wc' in vars(self):
del self.__dict__['_quick_access_changeid_wc'] del self.__dict__['_quick_access_changeid_wc']
@property @property