diff --git a/Lib/test/test_syntax.py b/Lib/test/test_syntax.py index 11ca4ea88b7..c82787e5486 100644 --- a/Lib/test/test_syntax.py +++ b/Lib/test/test_syntax.py @@ -470,6 +470,12 @@ Make sure that the old "raise X, Y[, Z]" form is gone: ... SyntaxError: invalid syntax + +>>> f(a=23, a=234) +Traceback (most recent call last): + ... +SyntaxError: keyword argument repeated + """ import re diff --git a/Python/ast.c b/Python/ast.c index 79c9403443a..6ec2ef1bc6e 100644 --- a/Python/ast.c +++ b/Python/ast.c @@ -1968,7 +1968,8 @@ ast_for_call(struct compiling *c, const node *n, expr_ty func) } else { keyword_ty kw; - identifier key; + identifier key, tmp; + int k; /* CHILD(ch, 0) is test, but must be an identifier? */ e = ast_for_expr(c, CHILD(ch, 0)); @@ -1989,6 +1990,13 @@ ast_for_call(struct compiling *c, const node *n, expr_ty func) return NULL; } key = e->v.Name.id; + for (k = 0; k < nkeywords; k++) { + tmp = ((keyword_ty)asdl_seq_GET(keywords, k))->arg; + if (!PyUnicode_Compare(tmp, key)) { + ast_error(CHILD(ch, 0), "keyword argument repeated"); + return NULL; + } + } e = ast_for_expr(c, CHILD(ch, 2)); if (!e) return NULL;