Skip to content

Commit c29882c

Browse files
Simplify per Pablo's review
1 parent e3d5252 commit c29882c

File tree

2 files changed

+546
-744
lines changed

2 files changed

+546
-744
lines changed

Grammar/python.gram

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1316,7 +1316,7 @@ invalid_parameters:
13161316
RAISE_SYNTAX_ERROR_KNOWN_LOCATION(a, "/ must be ahead of *") }
13171317
| param_maybe_default+ '/' a='*' {
13181318
RAISE_SYNTAX_ERROR_KNOWN_LOCATION(a, "expected comma between / and *") }
1319-
| !(param_maybe_default | '*' | '/' | '**') a=disjunction b=expression_without_invalid {
1319+
| a=NAME b=NAME {
13201320
RAISE_SYNTAX_ERROR_KNOWN_RANGE(a, b, "expected comma between parameters") }
13211321
invalid_default:
13221322
| a='=' &(')'|',') { RAISE_SYNTAX_ERROR_KNOWN_LOCATION(a, "expected default value expression") }
@@ -1346,7 +1346,7 @@ invalid_lambda_parameters:
13461346
RAISE_SYNTAX_ERROR_KNOWN_LOCATION(a, "/ must be ahead of *") }
13471347
| lambda_param_maybe_default+ '/' a='*' {
13481348
RAISE_SYNTAX_ERROR_KNOWN_LOCATION(a, "expected comma between / and *") }
1349-
| !(lambda_param_maybe_default | '*' | '/' | '**') a=disjunction b=expression_without_invalid {
1349+
| a=NAME b=NAME {
13501350
RAISE_SYNTAX_ERROR_KNOWN_RANGE(a, b, "expected comma between parameters") }
13511351
invalid_lambda_parameters_helper:
13521352
| a=lambda_slash_with_default { _PyPegen_singleton_seq(p, a) }

0 commit comments

Comments
 (0)