This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author serhiy.storchaka
Recipients benjamin.peterson, brett.cannon, gvanrossum, serhiy.storchaka, yselivanov
Date 2018-11-05.19:17:32
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1541445452.32.0.788709270274.issue35169@psf.upfronthosting.co.za>
In-reply-to
Content
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.
History
Date User Action Args
2018-11-05 19:17:32serhiy.storchakasetrecipients: + serhiy.storchaka, gvanrossum, brett.cannon, benjamin.peterson, yselivanov
2018-11-05 19:17:32serhiy.storchakasetmessageid: <1541445452.32.0.788709270274.issue35169@psf.upfronthosting.co.za>
2018-11-05 19:17:32serhiy.storchakalinkissue35169 messages
2018-11-05 19:17:32serhiy.storchakacreate