diff options
author | Pablo Galindo Salgado <Pablogsal@gmail.com> | 2021-09-27 20:59:06 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-09-27 20:59:06 (GMT) |
commit | 20f439b6b9e1032930a31b88694ab9f37a09e6b4 (patch) | |
tree | b8f430c4fa0048bbca33b52770e2cd3c13a6b6aa /Lib/traceback.py | |
parent | adc5d32f474595aa16a90369393440a0bc190777 (diff) | |
download | cpython-20f439b6b9e1032930a31b88694ab9f37a09e6b4.zip cpython-20f439b6b9e1032930a31b88694ab9f37a09e6b4.tar.gz cpython-20f439b6b9e1032930a31b88694ab9f37a09e6b4.tar.bz2 |
bpo-45249: Ensure the traceback module prints correctly syntax errors with ranges (GH-28575)
Diffstat (limited to 'Lib/traceback.py')
-rw-r--r-- | Lib/traceback.py | 27 |
1 files changed, 21 insertions, 6 deletions
diff --git a/Lib/traceback.py b/Lib/traceback.py index 1b537dc..3cb8e57 100644 --- a/Lib/traceback.py +++ b/Lib/traceback.py @@ -622,10 +622,14 @@ class TracebackException: occurred. - :attr:`lineno` For syntax errors - the linenumber where the error occurred. + - :attr:`end_lineno` For syntax errors - the end linenumber where the error + occurred. Can be `None` if not present. - :attr:`text` For syntax errors - the text where the error occurred. - :attr:`offset` For syntax errors - the offset into the text where the error occurred. + - :attr:`end_offset` For syntax errors - the offset into the text where the + error occurred. Can be `None` if not present. - :attr:`msg` For syntax errors - the compiler error message. """ @@ -655,8 +659,11 @@ class TracebackException: self.filename = exc_value.filename lno = exc_value.lineno self.lineno = str(lno) if lno is not None else None + end_lno = exc_value.end_lineno + self.end_lineno = str(end_lno) if end_lno is not None else None self.text = exc_value.text self.offset = exc_value.offset + self.end_offset = exc_value.end_offset self.msg = exc_value.msg if lookup_lines: self._load_lines() @@ -771,12 +778,20 @@ class TracebackException: ltext = rtext.lstrip(' \n\f') spaces = len(rtext) - len(ltext) yield ' {}\n'.format(ltext) - # Convert 1-based column offset to 0-based index into stripped text - caret = (self.offset or 0) - 1 - spaces - if caret >= 0: - # non-space whitespace (likes tabs) must be kept for alignment - caretspace = ((c if c.isspace() else ' ') for c in ltext[:caret]) - yield ' {}^\n'.format(''.join(caretspace)) + + if self.offset is not None: + offset = self.offset + end_offset = self.end_offset if self.end_offset is not None else offset + if offset == end_offset or end_offset == -1: + end_offset = offset + 1 + + # Convert 1-based column offset to 0-based index into stripped text + colno = offset - 1 - spaces + end_colno = end_offset - 1 - spaces + if colno >= 0: + # non-space whitespace (likes tabs) must be kept for alignment + caretspace = ((c if c.isspace() else ' ') for c in ltext[:colno]) + yield ' {}{}'.format("".join(caretspace), ('^' * (end_colno - colno) + "\n")) msg = self.msg or "<no detail available>" yield "{}: {}{}\n".format(stype, msg, filename_suffix) |