py/compile: Raise an error on async with/for outside an async function.
A simple reproducer is: async for x in (): x Before this change, it would cause an assertion error in mpy-cross and micropython-coverage.
This commit is contained in:
parent
a60ad33641
commit
f2dbc91022
12
py/compile.c
12
py/compile.c
@ -1970,7 +1970,16 @@ STATIC void compile_async_stmt(compiler_t *comp, mp_parse_node_struct_t *pns) {
|
||||
compile_funcdef(comp, pns0);
|
||||
scope_t *fscope = (scope_t *)pns0->nodes[4];
|
||||
fscope->scope_flags |= MP_SCOPE_FLAG_GENERATOR;
|
||||
} else if (MP_PARSE_NODE_STRUCT_KIND(pns0) == PN_for_stmt) {
|
||||
} else {
|
||||
// async for/with; first verify the scope is a generator
|
||||
int scope_flags = comp->scope_cur->scope_flags;
|
||||
if (!(scope_flags & MP_SCOPE_FLAG_GENERATOR)) {
|
||||
compile_syntax_error(comp, (mp_parse_node_t)pns0,
|
||||
MP_ERROR_TEXT("async for/with outside async function"));
|
||||
return;
|
||||
}
|
||||
|
||||
if (MP_PARSE_NODE_STRUCT_KIND(pns0) == PN_for_stmt) {
|
||||
// async for
|
||||
compile_async_for_stmt(comp, pns0);
|
||||
} else {
|
||||
@ -1979,6 +1988,7 @@ STATIC void compile_async_stmt(compiler_t *comp, mp_parse_node_struct_t *pns) {
|
||||
compile_async_with_stmt(comp, pns0);
|
||||
}
|
||||
}
|
||||
}
|
||||
#endif
|
||||
|
||||
STATIC void compile_expr_stmt(compiler_t *comp, mp_parse_node_struct_t *pns) {
|
||||
|
19
tests/basics/async_syntaxerror.py
Normal file
19
tests/basics/async_syntaxerror.py
Normal file
@ -0,0 +1,19 @@
|
||||
# test syntax errors using async
|
||||
|
||||
try:
|
||||
exec
|
||||
except NameError:
|
||||
print("SKIP")
|
||||
raise SystemExit
|
||||
|
||||
|
||||
def test_syntax(code):
|
||||
try:
|
||||
exec(code)
|
||||
print("no SyntaxError")
|
||||
except SyntaxError:
|
||||
print("SyntaxError")
|
||||
|
||||
|
||||
test_syntax("async for x in (): x")
|
||||
test_syntax("async with x: x")
|
2
tests/basics/async_syntaxerror.py.exp
Normal file
2
tests/basics/async_syntaxerror.py.exp
Normal file
@ -0,0 +1,2 @@
|
||||
SyntaxError
|
||||
SyntaxError
|
Loading…
Reference in New Issue
Block a user