D7473: localrepo: add some basic comment for block in __getitem__
marmoute (Pierre-Yves David)
phabricator at mercurial-scm.org
Fri Nov 22 09:19:34 UTC 2019
marmoute created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
There are different early processing before getting to the core of the function.
We highlight that fact.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D7473
AFFECTED FILES
mercurial/localrepo.py
CHANGE DETAILS
diff --git a/mercurial/localrepo.py b/mercurial/localrepo.py
--- a/mercurial/localrepo.py
+++ b/mercurial/localrepo.py
@@ -1515,10 +1515,13 @@
self.invalidate(clearfilecache=True)
def __getitem__(self, changeid):
+ # dealing with special cases
if changeid is None:
return context.workingctx(self)
if isinstance(changeid, context.basectx):
return changeid
+
+ # dealing with multiple revisions
if isinstance(changeid, slice):
# wdirrev isn't contiguous so the slice shouldn't include it
return [
@@ -1526,6 +1529,8 @@
for i in pycompat.xrange(*changeid.indices(len(self)))
if i not in self.changelog.filteredrevs
]
+
+ # dealing with arbitrary values
try:
if isinstance(changeid, int):
node = self.changelog.node(changeid)
To: marmoute, #hg-reviewers
Cc: mercurial-devel
More information about the Mercurial-devel
mailing list