Merged revisions 84214 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/branches/py3k ........ r84214 | amaury.forgeotdarc | 2010-08-19 23:32:38 +0200 (jeu., 19 août 2010) | 3 lines Add tests for r84209 (crashes in the Ast builder) Also remove one tab, and move a check closer to the possible failure. ........
This commit is contained in:
parent
fc34ac5fdd
commit
a1e5c69d5b
|
@ -476,6 +476,34 @@ Traceback (most recent call last):
|
|||
...
|
||||
SyntaxError: keyword argument repeated
|
||||
|
||||
Corner-cases that used to fail to raise the correct error:
|
||||
|
||||
>>> def f(*, x=lambda __debug__:0): pass
|
||||
Traceback (most recent call last):
|
||||
SyntaxError: assignment to keyword
|
||||
|
||||
>>> def f(*args:(lambda __debug__:0)): pass
|
||||
Traceback (most recent call last):
|
||||
SyntaxError: assignment to keyword
|
||||
|
||||
>>> def f(**kwargs:(lambda __debug__:0)): pass
|
||||
Traceback (most recent call last):
|
||||
SyntaxError: assignment to keyword
|
||||
|
||||
>>> with (lambda *:0): pass
|
||||
Traceback (most recent call last):
|
||||
SyntaxError: named arguments must follow bare *
|
||||
|
||||
Corner-cases that used to crash:
|
||||
|
||||
>>> def f(**__debug__): pass
|
||||
Traceback (most recent call last):
|
||||
SyntaxError: assignment to keyword
|
||||
|
||||
>>> def f(*xx, __debug__): pass
|
||||
Traceback (most recent call last):
|
||||
SyntaxError: assignment to keyword
|
||||
|
||||
"""
|
||||
|
||||
import re
|
||||
|
|
|
@ -674,7 +674,7 @@ handle_keywordonly_args(struct compiling *c, const node *n, int start,
|
|||
if (i + 1 < NCH(n) && TYPE(CHILD(n, i + 1)) == EQUAL) {
|
||||
expression = ast_for_expr(c, CHILD(n, i + 2));
|
||||
if (!expression)
|
||||
goto error;
|
||||
goto error;
|
||||
asdl_seq_SET(kwdefaults, j, expression);
|
||||
i += 2; /* '=' and test */
|
||||
}
|
||||
|
@ -873,14 +873,14 @@ ast_for_arguments(struct compiling *c, const node *n)
|
|||
ch = CHILD(n, i+1); /* tfpdef */
|
||||
assert(TYPE(ch) == tfpdef || TYPE(ch) == vfpdef);
|
||||
kwarg = NEW_IDENTIFIER(CHILD(ch, 0));
|
||||
if (!kwarg)
|
||||
return NULL;
|
||||
if (NCH(ch) > 1) {
|
||||
/* there is an annotation on the kwarg */
|
||||
kwargannotation = ast_for_expr(c, CHILD(ch, 2));
|
||||
if (!kwargannotation)
|
||||
return NULL;
|
||||
}
|
||||
if (!kwarg)
|
||||
return NULL;
|
||||
i += 3;
|
||||
break;
|
||||
default:
|
||||
|
|
Loading…
Reference in New Issue