Clearer 3.3 "yeild_from" semantic handling

This commit is contained in:
rocky
2019-07-01 12:54:38 -04:00
parent e55a0410c9
commit 68c5b2338f

View File

@@ -26,27 +26,31 @@ from uncompyle6.semantics.customize36 import customize_for_version36
from uncompyle6.semantics.customize37 import customize_for_version37 from uncompyle6.semantics.customize37 import customize_for_version37
from uncompyle6.semantics.customize38 import customize_for_version38 from uncompyle6.semantics.customize38 import customize_for_version38
def customize_for_version3(self, version):
TABLE_DIRECT.update({
'comp_for' : ( ' for %c in %c',
(2, 'store') , (0, 'expr') ),
'conditionalnot' : ( '%c if not %c else %c',
(2, 'expr') , (0, 'expr'), (4, 'expr') ),
'except_cond2' : ( '%|except %c as %c:\n', 1, 5 ),
'function_def_annotate': ( '\n\n%|def %c%c\n', -1, 0),
def customize_for_version3(self, version):
TABLE_DIRECT.update(
{
"comp_for": (" for %c in %c", (2, "store"), (0, "expr")),
"conditionalnot": (
"%c if not %c else %c",
(2, "expr"),
(0, "expr"),
(4, "expr"),
),
"except_cond2": ("%|except %c as %c:\n", 1, 5),
"function_def_annotate": ("\n\n%|def %c%c\n", -1, 0),
# When a generator is a single parameter of a function, # When a generator is a single parameter of a function,
# it doesn't need the surrounding parenethesis. # it doesn't need the surrounding parenethesis.
'call_generator' : ('%c%P', 0, (1, -1, ', ', 100)), "call_generator": ("%c%P", 0, (1, -1, ", ", 100)),
"importmultiple": ("%|import %c%c\n", 2, 3),
'importmultiple' : ( '%|import %c%c\n', 2, 3 ), "import_cont": (", %c", 2),
'import_cont' : ( ', %c', 2 ), "kwarg": ("%[0]{attr}=%c", 1),
'kwarg' : ( '%[0]{attr}=%c', 1), "raise_stmt2": ("%|raise %c from %c\n", 0, 1),
'raise_stmt2' : ( '%|raise %c from %c\n', 0, 1), "store_locals": ("%|# inspect.currentframe().f_locals = __locals__\n",),
'store_locals' : ( '%|# inspect.currentframe().f_locals = __locals__\n', ), "withstmt": ("%|with %c:\n%+%c%-", 0, 3),
'withstmt' : ( '%|with %c:\n%+%c%-', 0, 3), "withasstmt": ("%|with %c as (%c):\n%+%c%-", 0, 2, 3),
'withasstmt' : ( '%|with %c as (%c):\n%+%c%-', 0, 2, 3), }
}) )
assert version >= 3.0 assert version >= 3.0
@@ -61,7 +65,7 @@ def customize_for_version3(self, version):
# ---------- # ----------
# * subclass_code - the code for the subclass body # * subclass_code - the code for the subclass body
subclass_info = None subclass_info = None
if node == 'classdefdeco2': if node == "classdefdeco2":
if self.version >= 3.6: if self.version >= 3.6:
class_name = node[1][1].attr class_name = node[1][1].attr
elif self.version <= 3.3: elif self.version <= 3.3:
@@ -72,17 +76,17 @@ def customize_for_version3(self, version):
else: else:
build_class = node[0] build_class = node[0]
if self.version >= 3.6: if self.version >= 3.6:
if build_class == 'build_class_kw': if build_class == "build_class_kw":
mkfunc = build_class[1] mkfunc = build_class[1]
assert mkfunc == 'mkfunc' assert mkfunc == "mkfunc"
subclass_info = build_class subclass_info = build_class
if hasattr(mkfunc[0], 'attr') and iscode(mkfunc[0].attr): if hasattr(mkfunc[0], "attr") and iscode(mkfunc[0].attr):
subclass_code = mkfunc[0].attr subclass_code = mkfunc[0].attr
else: else:
assert mkfunc[0] == 'load_closure' assert mkfunc[0] == "load_closure"
subclass_code = mkfunc[1].attr subclass_code = mkfunc[1].attr
assert iscode(subclass_code) assert iscode(subclass_code)
if build_class[1][0] == 'load_closure': if build_class[1][0] == "load_closure":
code_node = build_class[1][1] code_node = build_class[1][1]
else: else:
code_node = build_class[1][0] code_node = build_class[1][0]
@@ -91,72 +95,72 @@ def customize_for_version3(self, version):
class_name = node[1][0].attr class_name = node[1][0].attr
build_class = node[0] build_class = node[0]
assert 'mkfunc' == build_class[1] assert "mkfunc" == build_class[1]
mkfunc = build_class[1] mkfunc = build_class[1]
if mkfunc[0] in ('kwargs', 'no_kwargs'): if mkfunc[0] in ("kwargs", "no_kwargs"):
if 3.0 <= self.version <= 3.2: if 3.0 <= self.version <= 3.2:
for n in mkfunc: for n in mkfunc:
if hasattr(n, 'attr') and iscode(n.attr): if hasattr(n, "attr") and iscode(n.attr):
subclass_code = n.attr subclass_code = n.attr
break break
elif n == 'expr': elif n == "expr":
subclass_code = n[0].attr subclass_code = n[0].attr
pass pass
pass pass
else: else:
for n in mkfunc: for n in mkfunc:
if hasattr(n, 'attr') and iscode(n.attr): if hasattr(n, "attr") and iscode(n.attr):
subclass_code = n.attr subclass_code = n.attr
break break
pass pass
pass pass
if node == 'classdefdeco2': if node == "classdefdeco2":
subclass_info = node subclass_info = node
else: else:
subclass_info = node[0] subclass_info = node[0]
elif build_class[1][0] == 'load_closure': elif build_class[1][0] == "load_closure":
# Python 3 with closures not functions # Python 3 with closures not functions
load_closure = build_class[1] load_closure = build_class[1]
if hasattr(load_closure[-3], 'attr'): if hasattr(load_closure[-3], "attr"):
# Python 3.3 classes with closures work like this. # Python 3.3 classes with closures work like this.
# Note have to test before 3.2 case because # Note have to test before 3.2 case because
# index -2 also has an attr. # index -2 also has an attr.
subclass_code = load_closure[-3].attr subclass_code = load_closure[-3].attr
elif hasattr(load_closure[-2], 'attr'): elif hasattr(load_closure[-2], "attr"):
# Python 3.2 works like this # Python 3.2 works like this
subclass_code = load_closure[-2].attr subclass_code = load_closure[-2].attr
else: else:
raise 'Internal Error n_classdef: cannot find class body' raise "Internal Error n_classdef: cannot find class body"
if hasattr(build_class[3], '__len__'): if hasattr(build_class[3], "__len__"):
if not subclass_info: if not subclass_info:
subclass_info = build_class[3] subclass_info = build_class[3]
elif hasattr(build_class[2], '__len__'): elif hasattr(build_class[2], "__len__"):
subclass_info = build_class[2] subclass_info = build_class[2]
else: else:
raise 'Internal Error n_classdef: cannot superclass name' raise "Internal Error n_classdef: cannot superclass name"
elif self.version >= 3.6 and node == 'classdefdeco2': elif self.version >= 3.6 and node == "classdefdeco2":
subclass_info = node subclass_info = node
subclass_code = build_class[1][0].attr subclass_code = build_class[1][0].attr
elif not subclass_info: elif not subclass_info:
if mkfunc[0] in ('no_kwargs', 'kwargs'): if mkfunc[0] in ("no_kwargs", "kwargs"):
subclass_code = mkfunc[1].attr subclass_code = mkfunc[1].attr
else: else:
subclass_code = mkfunc[0].attr subclass_code = mkfunc[0].attr
if node == 'classdefdeco2': if node == "classdefdeco2":
subclass_info = node subclass_info = node
else: else:
subclass_info = node[0] subclass_info = node[0]
if (node == 'classdefdeco2'): if node == "classdefdeco2":
self.write('\n') self.write("\n")
else: else:
self.write('\n\n') self.write("\n\n")
self.currentclass = str(class_name) self.currentclass = str(class_name)
self.write(self.indent, 'class ', self.currentclass) self.write(self.indent, "class ", self.currentclass)
self.print_super_classes3(subclass_info) self.print_super_classes3(subclass_info)
self.println(':') self.println(":")
# class body # class body
self.indent_more() self.indent_more()
@@ -165,11 +169,12 @@ def customize_for_version3(self, version):
self.currentclass = cclass self.currentclass = cclass
if len(self.param_stack) > 1: if len(self.param_stack) > 1:
self.write('\n\n') self.write("\n\n")
else: else:
self.write('\n\n\n') self.write("\n\n\n")
self.prune() self.prune()
self.n_classdef3 = n_classdef3 self.n_classdef3 = n_classdef3
if version == 3.0: if version == 3.0:
@@ -178,28 +183,28 @@ def customize_for_version3(self, version):
# since we pick up the iteration variable some other way and # since we pick up the iteration variable some other way and
# we definitely don't include in the source _[dd]. # we definitely don't include in the source _[dd].
def n_comp_iter(node): def n_comp_iter(node):
if node[0] == 'expr': if node[0] == "expr":
n = node[0][0] n = node[0][0]
if (n == 'LOAD_FAST' and if n == "LOAD_FAST" and n.pattr[0:2] == "_[":
n.pattr[0:2] == '_['):
self.prune() self.prune()
pass pass
pass pass
# Not this special case, procede as normal... # Not this special case, proceed as normal...
self.default(node) self.default(node)
self.n_comp_iter = n_comp_iter self.n_comp_iter = n_comp_iter
if version >= 3.3: elif version == 3.3:
# FIXME: perhaps this can be folded into the 3.4+ case?
def n_yield_from(node): def n_yield_from(node):
self.write('yield from') assert node[0] == "expr"
self.write(' ') assert node[0][0] == "get_iter"
if 3.3 == self.version: # Skip over yield_from.expr.get_iter which adds an
self.preorder(node[0][0][0][0]) # extra iter(). Maybe we can do in tranformation phase instead?
elif self.version >= 3.4: template = ("yield from %c", (0, "expr"))
self.preorder(node[0]) self.template_engine(template, node[0][0])
else: self.prune()
assert False, "Don't know how this Python handles 'yield from'."
self.prune() # stop recursing
self.n_yield_from = n_yield_from self.n_yield_from = n_yield_from
if 3.2 <= version <= 3.4: if 3.2 <= version <= 3.4:
@@ -211,11 +216,11 @@ def customize_for_version3(self, version):
for i in mapping[1:]: for i in mapping[1:]:
key = key[i] key = key[i]
pass pass
if key.kind.startswith('CALL_FUNCTION_VAR_KW'): if key.kind.startswith("CALL_FUNCTION_VAR_KW"):
# We may want to fill this in... # We may want to fill this in...
# But it is distinct from CALL_FUNCTION_VAR below # But it is distinct from CALL_FUNCTION_VAR below
pass pass
elif key.kind.startswith('CALL_FUNCTION_VAR'): elif key.kind.startswith("CALL_FUNCTION_VAR"):
# CALL_FUNCTION_VAR's top element of the stack contains # CALL_FUNCTION_VAR's top element of the stack contains
# the variable argument list, then comes # the variable argument list, then comes
# annotation args, then keyword args. # annotation args, then keyword args.
@@ -229,12 +234,15 @@ def customize_for_version3(self, version):
# kwargs == 0 is handled by the table entry # kwargs == 0 is handled by the table entry
# Should probably handle it here though. # Should probably handle it here though.
if nargs == 0: if nargs == 0:
template = ('%c(*%c, %C)', template = ("%c(*%c, %C)", 0, -2, (1, kwargs + 1, ", "))
0, -2, (1, kwargs+1, ', '))
else: else:
template = ('%c(%C, *%c, %C)', template = (
0, (1, nargs+1, ', '), "%c(%C, *%c, %C)",
-2, (-2-kwargs, -2, ', ')) 0,
(1, nargs + 1, ", "),
-2,
(-2 - kwargs, -2, ", "),
)
self.template_engine(template, node) self.template_engine(template, node)
self.prune() self.prune()
else: else:
@@ -243,6 +251,7 @@ def customize_for_version3(self, version):
self.n_call = n_call self.n_call = n_call
elif version < 3.2: elif version < 3.2:
def n_call(node): def n_call(node):
mapping = self._get_mapping(node) mapping = self._get_mapping(node)
key = node key = node
@@ -251,6 +260,7 @@ def customize_for_version3(self, version):
pass pass
gen_function_parens_adjust(key, node) gen_function_parens_adjust(key, node)
self.default(node) self.default(node)
self.n_call = n_call self.n_call = n_call
def n_mkfunc_annotate(node): def n_mkfunc_annotate(node):
@@ -260,13 +270,13 @@ def customize_for_version3(self, version):
if self.version <= 3.2: if self.version <= 3.2:
code = node[-2 + i] code = node[-2 + i]
elif self.version >= 3.3 or node[-2] == 'kwargs': elif self.version >= 3.3 or node[-2] == "kwargs":
# LOAD_CONST code object .. # LOAD_CONST code object ..
# LOAD_CONST 'x0' if >= 3.3 # LOAD_CONST 'x0' if >= 3.3
# EXTENDED_ARG # EXTENDED_ARG
# MAKE_FUNCTION .. # MAKE_FUNCTION ..
code = node[-3 + i] code = node[-3 + i]
elif node[-3] == 'expr': elif node[-3] == "expr":
code = node[-3][0] code = node[-3][0]
else: else:
# LOAD_CONST code object .. # LOAD_CONST code object ..
@@ -275,41 +285,50 @@ def customize_for_version3(self, version):
self.indent_more() self.indent_more()
for annotate_last in range(len(node) - 1, -1, -1): for annotate_last in range(len(node) - 1, -1, -1):
if node[annotate_last] == 'annotate_tuple': if node[annotate_last] == "annotate_tuple":
break break
# FIXME: the real situation is that when derived from # FIXME: the real situation is that when derived from
# function_def_annotate we the name has been filled in. # function_def_annotate we the name has been filled in.
# But when derived from funcdefdeco it hasn't Would like a better # But when derived from funcdefdeco it hasn't Would like a better
# way to distinquish. # way to distinquish.
if self.f.getvalue()[-4:] == 'def ': if self.f.getvalue()[-4:] == "def ":
self.write(code.attr.co_name) self.write(code.attr.co_name)
# FIXME: handle and pass full annotate args # FIXME: handle and pass full annotate args
make_function3_annotate(self, node, is_lambda=False, make_function3_annotate(
code_node=code, annotate_last=annotate_last) self, node, is_lambda=False, code_node=code, annotate_last=annotate_last
)
if len(self.param_stack) > 1: if len(self.param_stack) > 1:
self.write('\n\n') self.write("\n\n")
else: else:
self.write('\n\n\n') self.write("\n\n\n")
self.indent_less() self.indent_less()
self.prune() # stop recursing self.prune() # stop recursing
self.n_mkfunc_annotate = n_mkfunc_annotate self.n_mkfunc_annotate = n_mkfunc_annotate
TABLE_DIRECT.update({ TABLE_DIRECT.update(
'tryelsestmtl3': ( '%|try:\n%+%c%-%c%|else:\n%+%c%-', {
(1, 'suite_stmts_opt'), "tryelsestmtl3": (
(3, 'except_handler'), "%|try:\n%+%c%-%c%|else:\n%+%c%-",
(5, 'else_suitel') ), (1, "suite_stmts_opt"),
}) (3, "except_handler"),
(5, "else_suitel"),
)
}
)
if version >= 3.4: if version >= 3.4:
####################### #######################
# Python 3.4+ Changes # # Python 3.4+ Changes #
####################### #######################
TABLE_DIRECT.update({ TABLE_DIRECT.update(
'LOAD_CLASSDEREF': ( '%{pattr}', ), {
}) "LOAD_CLASSDEREF": ("%{pattr}",),
"yield_from": ("yield from %c", (0, "expr")),
}
)
if version >= 3.5: if version >= 3.5:
customize_for_version35(self, version) customize_for_version35(self, version)
if version >= 3.6: if version >= 3.6: