]> granicus.if.org Git - python/commitdiff
py-cvs-2001_07_13 (Rev 1.34) merge
authorKurt B. Kaiser <kbk@shore.net>
Sat, 14 Jul 2001 05:10:34 +0000 (05:10 +0000)
committerKurt B. Kaiser <kbk@shore.net>
Sat, 14 Jul 2001 05:10:34 +0000 (05:10 +0000)
"Amazing.  A very subtle change in policy in descr-branch actually
found a bug here.  Here's the deal: Class PyShell derives from class
OutputWindow.  Method PyShell.close()
wants to invoke its parent method, but because PyShell long ago was
inherited from class PyShellEditorWindow, it invokes
PyShelEditorWindow.close(self).  Now, class PyShellEditorWindow itself
derives from class OutputWindow, and inherits the close() method from
there without overriding it.  Under the old rules,
PyShellEditorWindow.close would return an unbound method restricted to
the class that defined the implementation of close(), which was
OutputWindow.close.  Under the new rules, the unbound method is
restricted to the class whose method was requested, that is
PyShellEditorWindow, and this was correctly trapped as an error." --GvR

Lib/idlelib/PyShell.py

index da827b8be16a149e19d86b08dac5b7ba1aadee30..399896af9a0dfd0728167d8b9e6bea60594c36c9 100644 (file)
@@ -440,7 +440,7 @@ class PyShell(OutputWindow):
             if self.reading:
                 self.top.quit()
             return "cancel"
-        return PyShellEditorWindow.close(self)
+        return OutputWindow.close(self)
 
     def _close(self):
         self.close_debugger()