diff options
author | Pablo Galindo Salgado <Pablogsal@gmail.com> | 2023-10-27 00:46:20 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-10-27 00:46:20 (GMT) |
commit | 22cde39fbf65df8790d1a93f199affa8a0361c4b (patch) | |
tree | 64a1cb1d67806b4e1192473f1f5e3823296f956f /Python | |
parent | 762aba72eb35a5696048ed89df35385c8d784c52 (diff) | |
download | cpython-22cde39fbf65df8790d1a93f199affa8a0361c4b.zip cpython-22cde39fbf65df8790d1a93f199affa8a0361c4b.tar.gz cpython-22cde39fbf65df8790d1a93f199affa8a0361c4b.tar.bz2 |
[3.11] bpo-43950: handle wide unicode characters in tracebacks (GH-28150) (#111373)
Diffstat (limited to 'Python')
-rw-r--r-- | Python/traceback.c | 35 |
1 files changed, 33 insertions, 2 deletions
diff --git a/Python/traceback.c b/Python/traceback.c index c4f5ec8..130f945 100644 --- a/Python/traceback.c +++ b/Python/traceback.c @@ -907,8 +907,39 @@ tb_displayline(PyTracebackObject* tb, PyObject *f, PyObject *filename, int linen goto done; } - if (print_error_location_carets(f, truncation, start_offset, end_offset, - right_start_offset, left_end_offset, + // Convert all offsets to display offsets (e.g. the space they would take up if printed + // on the screen). + Py_ssize_t dp_start = _PyPegen_calculate_display_width(source_line, start_offset); + if (dp_start < 0) { + err = ignore_source_errors() < 0; + goto done; + } + + Py_ssize_t dp_end = _PyPegen_calculate_display_width(source_line, end_offset); + if (dp_end < 0) { + err = ignore_source_errors() < 0; + goto done; + } + + Py_ssize_t dp_left_end = -1; + Py_ssize_t dp_right_start = -1; + if (has_secondary_ranges) { + dp_left_end = _PyPegen_calculate_display_width(source_line, left_end_offset); + if (dp_left_end < 0) { + err = ignore_source_errors() < 0; + goto done; + } + + dp_right_start = _PyPegen_calculate_display_width(source_line, right_start_offset); + if (dp_right_start < 0) { + err = ignore_source_errors() < 0; + goto done; + } + } + + + if (print_error_location_carets(f, truncation, dp_start, dp_end, + dp_right_start, dp_left_end, primary_error_char, secondary_error_char) < 0) { err = -1; goto done; |