D6337: tags: avoid double-reversing a list
martinvonz (Martin von Zweigbergk)
phabricator at mercurial-scm.org
Fri May 3 19:12:45 EDT 2019
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG6770df6e4365: tags: avoid double-reversing a list (authored by martinvonz, committed by ).
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST UPDATE
https://phab.mercurial-scm.org/D6337?vs=14994&id=14995
REVISION DETAIL
https://phab.mercurial-scm.org/D6337
AFFECTED FILES
mercurial/tags.py
CHANGE DETAILS
diff --git a/mercurial/tags.py b/mercurial/tags.py
--- a/mercurial/tags.py
+++ b/mercurial/tags.py
@@ -90,7 +90,7 @@
unfi = repo.unfiltered()
tonode = unfi.changelog.node
nodes = [tonode(r) for r in revs]
- fnodes = _getfnodes(ui, repo, nodes[::-1]) # reversed help the cache
+ fnodes = _getfnodes(ui, repo, nodes)
fnodes = _filterfnodes(fnodes, nodes)
return fnodes
@@ -458,7 +458,8 @@
# This is the most expensive part of finding tags, so performance
# depends primarily on the size of newheads. Worst case: no cache
# file, so newheads == repoheads.
- cachefnode = _getfnodes(ui, repo, repoheads)
+ # Reversed order helps the cache ('repoheads' is in descending order)
+ cachefnode = _getfnodes(ui, repo, reversed(repoheads))
# Caller has to iterate over all heads, but can use the filenodes in
# cachefnode to get to each .hgtags revision quickly.
@@ -473,7 +474,7 @@
starttime = util.timer()
fnodescache = hgtagsfnodescache(repo.unfiltered())
cachefnode = {}
- for node in reversed(nodes):
+ for node in nodes:
fnode = fnodescache.getfnode(node)
if fnode != nullid:
cachefnode[node] = fnode
To: martinvonz, #hg-reviewers
Cc: mercurial-devel
More information about the Mercurial-devel
mailing list