Skip to content

gh-138294: Fix Misleading error for ((a) := 1) #138627

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

gh-138294: Fix Misleading error for ((a) := 1) #138627

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
185ce34
Raise error for parenthesized target in walrus expression
Devansh-b Sep 7, 2025
d9c911e
Raise SyntaxError for parenthesized walrus target (#138294)
Devansh-b Sep 7, 2025
bb95d1f
Update Misc/NEWS.d/next/Core_and_Builtins/2025-09-08-12-30-00.gh-issu…
dbXD320 Sep 7, 2025
84fbba0
Add test for parenthesized walrus target
Devansh-b Sep 8, 2025
7eb3d1e
Move parenthesized walrus target test to docstring in test_syntax
Devansh-b Sep 9, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions Grammar/python.gram
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1261,6 +1261,9 @@ invalid_expression:
RAISE_SYNTAX_ERROR_KNOWN_RANGE(a, b, "t-string: lambda expressions are not allowed without parentheses") }

invalid_named_expression(memo):
| '(' a=NAME ')' ':=' expression {
RAISE_SYNTAX_ERROR_KNOWN_LOCATION(
a, "cannot parenthesize target name in assignment expression") }
| a=expression ':=' expression {
RAISE_SYNTAX_ERROR_KNOWN_LOCATION(
a, "cannot use assignment expressions with %s", _PyPegen_get_expr_name(a)) }
Expand Down
1 change: 1 addition & 0 deletions Misc/NEWS.d/next/Core_and_Builtins/2025-09-08-12-30-00.gh-issue-138294.abcd12.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Raise SyntaxError when using parentheses around a walrus target.
37 changes: 37 additions & 0 deletions Parser/parser.c
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading