Merge pull request #2677 from tacaswell/fix_2583

pelson · pelson · commit 8c16c614ecf7 · 2013-12-18T06:00:11.000-08:00
Make sure self._idle is set to `True` in all cases
diff --git a/lib/matplotlib/backends/backend_gtk.py b/lib/matplotlib/backends/backend_gtk.py
@@ -376,8 +376,10 @@ def draw(self):
 
     def draw_idle(self):
         def idle_draw(*args):
-            self.draw()
-            self._idle_draw_id = 0
+            try:
+                self.draw()
+            finally:
+                self._idle_draw_id = 0
             return False
         if self._idle_draw_id == 0:
             self._idle_draw_id = gobject.idle_add(idle_draw)
diff --git a/lib/matplotlib/backends/backend_gtk3.py b/lib/matplotlib/backends/backend_gtk3.py
@@ -322,8 +322,10 @@ def draw(self):
 
     def draw_idle(self):
         def idle_draw(*args):
-            self.draw()
-            self._idle_draw_id = 0
+            try:
+                self.draw()
+            finally:
+                self._idle_draw_id = 0
             return False
         if self._idle_draw_id == 0:
             self._idle_draw_id = GObject.idle_add(idle_draw)
diff --git a/lib/matplotlib/backends/backend_qt4.py b/lib/matplotlib/backends/backend_qt4.py
@@ -415,8 +415,10 @@ def draw_idle(self):
         self._idle = False
 
         def idle_draw(*args):
-            self.draw()
-            self._idle = True
+            try:
+                self.draw()
+            finally:
+                self._idle = True
         if d:
             QtCore.QTimer.singleShot(0, idle_draw)
 
diff --git a/lib/matplotlib/backends/backend_tkagg.py b/lib/matplotlib/backends/backend_tkagg.py
@@ -362,8 +362,10 @@ def draw_idle(self):
         d = self._idle
         self._idle = False
         def idle_draw(*args):
-            self.draw()
-            self._idle = True
+            try:
+                self.draw()
+            finally:
+                self._idle = True
 
         if d:
             self._idle_callback = self._tkcanvas.after_idle(idle_draw)
