Title: Improve error messages for assignment
Type: enhancement Stage: patch review
Components: Interpreter Core Versions: Python 3.8
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: benjamin.peterson, brett.cannon, serhiy.storchaka, yselivanov
Priority: normal Keywords: patch

Created on 2018-11-05 19:17 by serhiy.storchaka, last changed 2018-11-06 18:42 by gvanrossum.

Pull Requests
URL Status Linked Edit
PR 10342 open serhiy.storchaka, 2018-11-05 19:21
Messages (1)
msg329313 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2018-11-05 19:17
This is a follow up of issue34641.

>>> f(lambda x: x = 1)
  File "<stdin>", line 1
SyntaxError: lambda cannot contain assignment
>>> f(x.y = 1)
  File "<stdin>", line 1
SyntaxError: keyword can't be an expression

The error message "keyword can't be an expression" still looks confusing to me. This is because the term "keyword" is ambiguous. Usually it means reserved identifier like "if" or "def". Some keywords, like "None" and "True" can be expressions. Perhaps "keyword name can't be an expression" would be better.

But I think that in these cases it is most likely that "=" was used instead of "==". And it would be better to generalize the error message for lambdas and point on a possible typo.

>>> f(x.y = 1)
  File "<stdin>", line 1
SyntaxError: expression cannot contain assignment, perhaps you meant "=="?

The proposed PR changes this error message. It makes also an error message for forbidden assignment more concrete: "cannot assign to __debug__" instead of "assignment to keyword" (the latter is actually incorrect, because __debug__ is not true keyword in tokenizer). This restores Python 2 error messages. Improved also other error messages for forbidden assigning: dict and set displays, f-string expressions are not literals.
Date User Action Args
2018-11-06 18:42:14gvanrossumsetnosy: - gvanrossum
2018-11-05 19:21:18serhiy.storchakasetkeywords: + patch
stage: patch review
pull_requests: + pull_request9645
2018-11-05 19:17:32serhiy.storchakacreate