diff options
author | Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> | 2020-08-05 13:52:29 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-08-05 13:52:29 (GMT) |
commit | a1320989f5350439e0677450f49b36f2c10583d2 (patch) | |
tree | ec5f1413a0abf0bdc224e626320070c1846cb4ef /Lib/ast.py | |
parent | ea680631b478f091a171dc802d861f5014f58c8f (diff) | |
download | cpython-a1320989f5350439e0677450f49b36f2c10583d2.zip cpython-a1320989f5350439e0677450f49b36f2c10583d2.tar.gz cpython-a1320989f5350439e0677450f49b36f2c10583d2.tar.bz2 |
bpo-40726: handle uninitalized end_lineno on ast.increment_lineno (GH-20312)
(cherry picked from commit 8f4380d2f5839a321475104765221a7394a9d649)
Co-authored-by: Batuhan Taskaya <batuhanosmantaskaya@gmail.com>
Diffstat (limited to 'Lib/ast.py')
-rw-r--r-- | Lib/ast.py | 13 |
1 files changed, 10 insertions, 3 deletions
@@ -180,7 +180,11 @@ def copy_location(new_node, old_node): for attr in 'lineno', 'col_offset', 'end_lineno', 'end_col_offset': if attr in old_node._attributes and attr in new_node._attributes: value = getattr(old_node, attr, None) - if value is not None: + # end_lineno and end_col_offset are optional attributes, and they + # should be copied whether the value is None or not. + if value is not None or ( + hasattr(old_node, attr) and attr.startswith("end_") + ): setattr(new_node, attr, value) return new_node @@ -229,8 +233,11 @@ def increment_lineno(node, n=1): for child in walk(node): if 'lineno' in child._attributes: child.lineno = getattr(child, 'lineno', 0) + n - if 'end_lineno' in child._attributes: - child.end_lineno = getattr(child, 'end_lineno', 0) + n + if ( + "end_lineno" in child._attributes + and (end_lineno := getattr(child, "end_lineno", 0)) is not None + ): + child.end_lineno = end_lineno + n return node |