gh-130273: Fix incorrectly colored error locations when wide unicode characters exists#130277
Uh oh!
There was an error while loading. Please reload this page.
Conversation
| foriinrange(start, len(line)): | ||
| ifwidth<=0: | ||
| returni, line[start:i] | ||
| width-=2ifeast_asian_width(line[i]) in_WIDE_CHAR_SPECIFIERSelse1 |
There was a problem hiding this comment.
does this correctly handle zero-width characters? e.g. the '\u0301' from 'cafe\u0301'
There was a problem hiding this comment.
No.
However, _display_width() does not correctly handle zero-width characters for carets positions either.
>>> café = 1/0 Traceback (most recent call last): File "<python-input-0>", line 1, in <module> café =1/0~^~ZeroDivisionError: division by zero >>> They can be fixed in the same way. I wonder how I can find all the zero-width characters.
There was a problem hiding this comment.
It looks like there are many zero-width characters: https://wcwidth.readthedocs.io/en/latest/specs.html#width-of-0
There was a problem hiding this comment.
This is how wcwidth implements wcswidth() and wcwidth(): https://wcwidth.readthedocs.io/en/latest/_modules/wcwidth/wcwidth.html#wcswidth
Maybe it is too complicated for our traceback? Could we have a simple way that can handle most of the cases?
2 participants
Uh oh!
There was an error while loading. Please reload this page.