fix_map.py 2.99 KB
Newer Older
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23
# Copyright 2007 Google, Inc. All Rights Reserved.
# Licensed to PSF under a Contributor Agreement.

"""Fixer that changes map(F, ...) into list(map(F, ...)) unless there
exists a 'from future_builtins import map' statement in the top-level
namespace.

As a special case, map(None, X) is changed into list(X).  (This is
necessary because the semantics are changed in this case -- the new
map(None, X) is equivalent to [(x,) for x in X].)

We avoid the transformation (except for the special case mentioned
above) if the map() call is directly contained in iter(<>), list(<>),
tuple(<>), sorted(<>), ...join(<>), or for V in <>:.

NOTE: This is still not correct if the original code was depending on
map(F, X, Y, ...) to go on until the longest argument is exhausted,
substituting None for missing values -- like zip(), it now stops as
soon as the shortest argument is exhausted.
"""

# Local imports
from ..pgen2 import token
24 25
from .. import fixer_base
from ..fixer_util import Name, Call, ListComp, in_special_context
26 27
from ..pygram import python_symbols as syms

28
class FixMap(fixer_base.ConditionalFix):
Benjamin Peterson's avatar
Benjamin Peterson committed
29
    BM_compatible = True
30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52

    PATTERN = """
    map_none=power<
        'map'
        trailer< '(' arglist< 'None' ',' arg=any [','] > ')' >
    >
    |
    map_lambda=power<
        'map'
        trailer<
            '('
            arglist<
                lambdef< 'lambda'
                         (fp=NAME | vfpdef< '(' fp=NAME ')'> ) ':' xp=any
                >
                ','
                it=any
            >
            ')'
        >
    >
    |
    power<
53
        'map' trailer< '(' [arglist=any] ')' >
54 55 56
    >
    """

57
    skip_on = 'future_builtins.map'
58 59

    def transform(self, node, results):
60
        if self.should_skip(node):
61
            return
62

63 64 65
        if node.parent.type == syms.simple_stmt:
            self.warning(node, "You should use a for loop here")
            new = node.clone()
66
            new.prefix = ""
67 68
            new = Call(Name("list"), [new])
        elif "map_lambda" in results:
69 70 71
            new = ListComp(results["xp"].clone(),
                           results["fp"].clone(),
                           results["it"].clone())
72 73 74 75
        else:
            if "map_none" in results:
                new = results["arg"].clone()
            else:
76 77 78 79 80 81 82 83 84
                if "arglist" in results:
                    args = results["arglist"]
                    if args.type == syms.arglist and \
                       args.children[0].type == token.NAME and \
                       args.children[0].value == "None":
                        self.warning(node, "cannot convert map(None, ...) "
                                     "with multiple arguments because map() "
                                     "now truncates to the shortest sequence")
                        return
85 86 87
                if in_special_context(node):
                    return None
                new = node.clone()
88
            new.prefix = ""
89
            new = Call(Name("list"), [new])
90
        new.prefix = node.prefix
91
        return new