fix_xrange.py 2.63 KB
Newer Older
1 2 3 4 5 6
# Copyright 2007 Google, Inc. All Rights Reserved.
# Licensed to PSF under a Contributor Agreement.

"""Fixer that changes xrange(...) into range(...)."""

# Local imports
7 8
from .. import fixer_base
from ..fixer_util import Name, Call, consuming_calls
9 10
from .. import patcomp

11

12
class FixXrange(fixer_base.BaseFix):
Benjamin Peterson's avatar
Benjamin Peterson committed
13
    BM_compatible = True
14
    PATTERN = """
15 16 17
              power<
                 (name='range'|name='xrange') trailer< '(' args=any ')' >
              rest=any* >
18 19
              """

20 21 22 23 24 25 26
    def start_tree(self, tree, filename):
        super(FixXrange, self).start_tree(tree, filename)
        self.transformed_xranges = set()

    def finish_tree(self, tree, filename):
        self.transformed_xranges = None

27
    def transform(self, node, results):
28 29 30 31 32 33 34 35 36
        name = results["name"]
        if name.value == "xrange":
            return self.transform_xrange(node, results)
        elif name.value == "range":
            return self.transform_range(node, results)
        else:
            raise ValueError(repr(name))

    def transform_xrange(self, node, results):
37
        name = results["name"]
38
        name.replace(Name("range", prefix=name.prefix))
39 40
        # This prevents the new range call from being wrapped in a list later.
        self.transformed_xranges.add(id(node))
41 42

    def transform_range(self, node, results):
43 44
        if (id(node) not in self.transformed_xranges and
            not self.in_special_context(node)):
45 46 47
            range_call = Call(Name("range"), [results["args"].clone()])
            # Encase the range call in list().
            list_call = Call(Name("list"), [range_call],
48
                             prefix=node.prefix)
49 50 51 52
            # Put things that were after the range() call after the list call.
            for n in results["rest"]:
                list_call.append_child(n)
            return list_call
53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73

    P1 = "power< func=NAME trailer< '(' node=any ')' > any* >"
    p1 = patcomp.compile_pattern(P1)

    P2 = """for_stmt< 'for' any 'in' node=any ':' any* >
            | comp_for< 'for' any 'in' node=any any* >
            | comparison< any 'in' node=any any*>
         """
    p2 = patcomp.compile_pattern(P2)

    def in_special_context(self, node):
        if node.parent is None:
            return False
        results = {}
        if (node.parent.parent is not None and
               self.p1.match(node.parent.parent, results) and
               results["node"] is node):
            # list(d.keys()) -> list(d.keys()), etc.
            return results["func"].value in consuming_calls
        # for ... in d.iterkeys() -> for ... in d.keys(), etc.
        return self.p2.match(node.parent, results) and results["node"] is node