Message 371384 - Python tracker

➜

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	lys.nikolaou
Recipients	christian.heimes, lys.nikolaou, pablogsal
Date	2020-06-12.15:29:33
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<[email protected]>
In-reply-to

Content
> What happens with: >>> "Ṕýţĥòñ" + Exact same thing. The offset is 16 at the start and gets decreased to 15 in the line I linked to in my previous post. And then col_offset gets converted to col_number which is 9, which seems correct. Although it is correct, it feels correct by accident, since the if-branch on https://github.com/python/cpython/blob/e2fb8a2c42ee60c72a40d93da69e9efc4e359023/Parser/pegen.c#L154 only evaluates to false due to the decrease of size and I don't know if that was intended.

> What happens with:

>>> "Ṕýţĥòñ" +

Exact same thing. The offset is 16 at the start and gets decreased to 15 in the line I linked to in my previous post. And then col_offset gets converted to col_number which is 9, which seems correct.

Although it is correct, it feels correct by accident, since the if-branch on https://github.com/python/cpython/blob/e2fb8a2c42ee60c72a40d93da69e9efc4e359023/Parser/pegen.c#L154 only evaluates to false due to the decrease of size and I don't know if that was intended.

History
Date	User	Action	Args
2020-06-12 15:29:33	lys.nikolaou	set	recipients: + lys.nikolaou, christian.heimes, pablogsal
2020-06-12 15:29:33	lys.nikolaou	set	messageid: <[email protected]>
2020-06-12 15:29:33	lys.nikolaou	link	issue40958 messages
2020-06-12 15:29:33	lys.nikolaou	create