# HG changeset patch # User Augie Fackler # Date 1478813683 18000 # Node ID 2ded17b64f09a9cda13cbf7656b83a950386ac03 # Parent bff77a693531126c338afc660eeae1fc01c88df0 revlog: avoid shadowing several variables using list comprehensions diff -r bff77a693531 -r 2ded17b64f09 mercurial/revlog.py --- a/mercurial/revlog.py Thu Nov 10 16:33:41 2016 -0500 +++ b/mercurial/revlog.py Thu Nov 10 16:34:43 2016 -0500 @@ -597,7 +597,7 @@ visit.append(p) missing = list(missing) missing.sort() - return has, [self.node(r) for r in missing] + return has, [self.node(miss) for miss in missing] def incrementalmissingrevs(self, common=None): """Return an object that can be used to incrementally compute the @@ -749,10 +749,10 @@ # include roots that aren't ancestors. # Filter out roots that aren't ancestors of heads - roots = [n for n in roots if n in ancestors] + roots = [root for root in roots if root in ancestors] # Recompute the lowest revision if roots: - lowestrev = min([self.rev(n) for n in roots]) + lowestrev = min([self.rev(root) for root in roots]) else: # No more roots? Return empty list return nonodes @@ -811,7 +811,7 @@ # But, obviously its parents aren't. for p in self.parents(n): heads.pop(p, None) - heads = [n for n, flag in heads.iteritems() if flag] + heads = [head for head, flag in heads.iteritems() if flag] roots = list(roots) assert orderedout assert roots @@ -963,9 +963,9 @@ def _partialmatch(self, id): try: - n = self.index.partialmatch(id) - if n and self.hasnode(n): - return n + partial = self.index.partialmatch(id) + if partial and self.hasnode(partial): + return partial return None except RevlogError: # parsers.c radix tree lookup gave multiple matches