python#111488: Changed error message in case of no 'in' keyword after…

… 'for' in list comprehensions
grigoriev-semyon · Jan 2, 2024 · 7b74e41 · 7b74e41
1 parent acf4cf5
commit 7b74e41
Show file tree

Hide file tree

Showing 5 changed files with 1,718 additions and 1,591 deletions.
diff --git a/Grammar/python.gram b/Grammar/python.gram
@@ -968,6 +968,8 @@ for_if_clause[comprehension_ty]:
         CHECK_VERSION(comprehension_ty, 6, "Async comprehensions are", _PyAST_comprehension(a, b, c, 1, p->arena)) }
     | 'for' a=star_targets 'in' ~ b=disjunction c[asdl_expr_seq*]=('if' z=disjunction { z })* {
         _PyAST_comprehension(a, b, c, 0, p->arena) }
+    | 'async'? 'for' a=star_targets+ !'in' {
+        RAISE_SYNTAX_ERROR("'in' expected after for-loop variables") }
     | invalid_for_target
 
 listcomp[expr_ty]:

diff --git a/Lib/test/test_syntax.py b/Lib/test/test_syntax.py
@@ -259,6 +259,20 @@
 Traceback (most recent call last):
 SyntaxError: invalid syntax
 
+Comprehensions without 'in' keyword:
+
+>>> [x for x if range(1)]
+Traceback (most recent call last):
+SyntaxError: 'in' expected after for-loop variables
+
+>>> tuple(x for x if range(1))
+Traceback (most recent call last):
+SyntaxError: 'in' expected after for-loop variables
+
+>>> [x for x() in a]
+Traceback (most recent call last):
+SyntaxError: cannot assign to function call
+
 Comprehensions creating tuples without parentheses
 should produce a specialized error message:
 

diff --git a/Misc/ACKS b/Misc/ACKS
@@ -665,6 +665,7 @@ Eddy De Greef
 Duane Griffin
 Grant Griffin
 Andrea Griffini
+Semyon Grigoryev
 Duncan Grisby
 Olivier Grisel
 Fabian Groffen

diff --git a/Misc/NEWS.d/next/Core and Builtins/2024-01-02-17-22-57.gh-issue-111488.EJH3Oh.rst b/Misc/NEWS.d/next/Core and Builtins/2024-01-02-17-22-57.gh-issue-111488.EJH3Oh.rst
@@ -0,0 +1,2 @@
+Changed error message in case of no 'in' keyword after 'for' in list
+comprehensions
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		Changed error message in case of no 'in' keyword after 'for' in list
		comprehensions