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
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

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 19410

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