bpo-32905: IDLE - remove unused code in pyparse module (GH-5807)
dump is similar to print but less flexible. lastopenbracketpos is now always initialized in _study2, as was stmt_bracketing, so the class settings are not needed. get_last_open_bracket_pos is never called.
(cherry picked from commit 451d1edaf4d27c4e632d81246d308e8dd6ea945f)
Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
diff --git a/Lib/idlelib/pyparse.py b/Lib/idlelib/pyparse.py
index 72bd9e0..93607dc 100644
--- a/Lib/idlelib/pyparse.py
+++ b/Lib/idlelib/pyparse.py
@@ -18,10 +18,6 @@
(C_NONE, C_BACKSLASH, C_STRING_FIRST_LINE,
C_STRING_NEXT_LINES, C_BRACKET) = range(5)
-if 0: # for throwaway debugging output
- def dump(*stuff):
- sys.__stdout__.write(" ".join(map(str, stuff)) + "\n")
-
# Find what looks like the start of a popular statement.
_synchre = re.compile(r"""
@@ -496,8 +492,7 @@
# end while p < q:
self.lastch = lastch
- if stack:
- self.lastopenbracketpos = stack[-1]
+ self.lastopenbracketpos = stack[-1] if stack else None
self.stmt_bracketing = tuple(bracketing)
def compute_bracket_indent(self):
@@ -620,22 +615,10 @@
self._study2()
return _closere(self.str, self.stmt_start) is not None
- # XXX - is this used?
- lastopenbracketpos = None
-
- def get_last_open_bracket_pos(self):
- "Return index of last open bracket or None."
- self._study2()
- return self.lastopenbracketpos
-
- # XXX - is this used?
- stmt_bracketing = None
-
def get_last_stmt_bracketing(self):
- """Return a tuple of the structure of the bracketing of the last
- interesting statement.
+ """Return bracketing structure of the last interesting statement.
- Tuple is in the format defined in _study2().
+ The returned tuple is in the format defined in _study2().
"""
self._study2()
return self.stmt_bracketing