[PATCH 1 of 6] ancestor: remove alias of initrevs from _lazyancestorsiter()
Yuya Nishihara
yuya at tcha.org
Tue Sep 11 23:02:06 UTC 2018
# HG changeset patch
# User Yuya Nishihara <yuya at tcha.org>
# Date 1536673112 -32400
# Tue Sep 11 22:38:32 2018 +0900
# Node ID 2989618d644ce4694a3288f8f20107f446703f66
# Parent a1b6b1fcfd29503e8cb9c8cef617571334a69339
ancestor: remove alias of initrevs from _lazyancestorsiter()
It's just redundant and less comprehensible.
diff --git a/mercurial/ancestor.py b/mercurial/ancestor.py
--- a/mercurial/ancestor.py
+++ b/mercurial/ancestor.py
@@ -262,20 +262,18 @@ class incrementalmissingancestors(object
# Extracted from lazyancestors.__iter__ to avoid a reference cycle
def _lazyancestorsiter(parentrevs, initrevs, stoprev, inclusive):
seen = {nullrev}
- revs = initrevs
-
schedule = heapq.heappush
nextitem = heapq.heappop
see = seen.add
if inclusive:
- visit = [-r for r in revs]
- seen.update(revs)
+ visit = [-r for r in initrevs]
+ seen.update(initrevs)
heapq.heapify(visit)
else:
visit = []
heapq.heapify(visit)
- for r in revs:
+ for r in initrevs:
for parent in parentrevs(r):
if parent not in seen:
schedule(visit, -parent)
More information about the Mercurial-devel
mailing list