Skip to content

Commit

Permalink
bpo-38002: Use False/True for IDLE pyshell bools (GH-19203)
Browse files Browse the repository at this point in the history
Change 0/1 assignments to 'executing', 'canceled', 'reading', 'endoffile'.
These are not used outside of pyshell. Other bools already use False/True.
Add comment about int needed for Windows call.
Remove self.more, unused in idlelib and code.InteractiveInterpreter.
The latter uses 'more' as a local.
(cherry picked from commit 34a49aa)

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
  • Loading branch information
miss-islington and terryjreedy committed Mar 28, 2020
1 parent bb85226 commit 8c3ab18
Showing 1 changed file with 12 additions and 13 deletions.
25 changes: 12 additions & 13 deletions Lib/idlelib/pyshell.py
Expand Up @@ -16,7 +16,7 @@
if sys.platform == 'win32':
try:
import ctypes
PROCESS_SYSTEM_DPI_AWARE = 1
PROCESS_SYSTEM_DPI_AWARE = 1 # Int required.
ctypes.OleDLL('shcore').SetProcessDpiAwareness(PROCESS_SYSTEM_DPI_AWARE)
except (ImportError, AttributeError, OSError):
pass
Expand Down Expand Up @@ -676,7 +676,6 @@ def execfile(self, filename, source=None):
def runsource(self, source):
"Extend base class method: Stuff the source in the line cache first"
filename = self.stuffsource(source)
self.more = 0
# at the moment, InteractiveInterpreter expects str
assert isinstance(source, str)
# InteractiveInterpreter.runsource() calls its runcode() method,
Expand Down Expand Up @@ -993,12 +992,12 @@ def open_debugger(self):
def beginexecuting(self):
"Helper for ModifiedInterpreter"
self.resetoutput()
self.executing = 1
self.executing = True

def endexecuting(self):
"Helper for ModifiedInterpreter"
self.executing = 0
self.canceled = 0
self.executing = False
self.canceled = False
self.showprompt()

def close(self):
Expand Down Expand Up @@ -1075,7 +1074,7 @@ def stop_readline(self):
def readline(self):
save = self.reading
try:
self.reading = 1
self.reading = True
self.top.mainloop() # nested mainloop()
finally:
self.reading = save
Expand All @@ -1087,11 +1086,11 @@ def readline(self):
line = "\n"
self.resetoutput()
if self.canceled:
self.canceled = 0
self.canceled = False
if not use_subprocess:
raise KeyboardInterrupt
if self.endoffile:
self.endoffile = 0
self.endoffile = False
line = ""
return line

Expand All @@ -1109,8 +1108,8 @@ def cancel_callback(self, event=None):
self.interp.write("KeyboardInterrupt\n")
self.showprompt()
return "break"
self.endoffile = 0
self.canceled = 1
self.endoffile = False
self.canceled = True
if (self.executing and self.interp.rpcclt):
if self.interp.getdebugger():
self.interp.restart_subprocess()
Expand All @@ -1130,8 +1129,8 @@ def eof_callback(self, event):
self.resetoutput()
self.close()
else:
self.canceled = 0
self.endoffile = 1
self.canceled = False
self.endoffile = True
self.top.quit()
return "break"

Expand Down Expand Up @@ -1303,7 +1302,7 @@ def write(self, s, tags=()):
raise ###pass # ### 11Aug07 KBK if we are expecting exceptions
# let's find out what they are and be specific.
if self.canceled:
self.canceled = 0
self.canceled = False
if not use_subprocess:
raise KeyboardInterrupt
return count
Expand Down

0 comments on commit 8c3ab18

Please sign in to comment.