Merge pull request #14632 from anntzer/tkwin

tacaswell · web-flow · commit ce72670a62ab · 2019-07-03T09:06:38.000-04:00
DOC: Remove reference to old Tk/Windows bug.
diff --git a/examples/user_interfaces/embedding_in_tk_sgskip.py b/examples/user_interfaces/embedding_in_tk_sgskip.py
@@ -39,17 +39,7 @@ def on_key_press(event):
 
 canvas.mpl_connect("key_press_event", on_key_press)
 
-
-def _quit():
-    root.quit()  # Stops mainloop.
-    # The following is necessary on Windows to prevent
-    # Fatal Python Error: PyEval_RestoreThread: NULL tstate
-    root.destroy()
-
-
-button = tkinter.Button(master=root, text="Quit", command=_quit)
+button = tkinter.Button(master=root, text="Quit", command=root.quit)
 button.pack(side=tkinter.BOTTOM)
 
 tkinter.mainloop()
-# If you put root.destroy() here, it will cause an error if the window is
-# closed with the window manager.
diff --git a/lib/matplotlib/backends/_backend_tk.py b/lib/matplotlib/backends/_backend_tk.py
@@ -465,10 +465,6 @@ def resize(self, width, height):
             self.toolbar.configure(width=width)
 
     def show(self):
-        """
-        this function doesn't segfault but causes the
-        PyEval_RestoreThread: NULL state bug on win32
-        """
         with _restore_foreground_window_at_end():
             if not self._shown:
                 def destroy(*args):
