Merge pull request #26946 from anntzer/cu

oscargus · web-flow · commit e3a5cee92bf1 · 2023-09-28T11:21:40.000+02:00
Inline Cursor._update into its sole caller.
diff --git a/lib/matplotlib/widgets.py b/lib/matplotlib/widgets.py
@@ -1886,23 +1886,19 @@ def onmove(self, event):
         if not self.ax.contains(event)[0]:
             self.linev.set_visible(False)
             self.lineh.set_visible(False)
-
             if self.needclear:
                 self.canvas.draw()
                 self.needclear = False
             return
         self.needclear = True
-
         xdata, ydata = self._get_data_coords(event)
         self.linev.set_xdata((xdata, xdata))
         self.linev.set_visible(self.visible and self.vertOn)
         self.lineh.set_ydata((ydata, ydata))
         self.lineh.set_visible(self.visible and self.horizOn)
-
-        if self.visible and (self.vertOn or self.horizOn):
-            self._update()
-
-    def _update(self):
+        if not (self.visible and (self.vertOn or self.horizOn)):
+            return
+        # Redraw.
         if self.useblit:
             if self.background is not None:
                 self.canvas.restore_region(self.background)
@@ -1911,7 +1907,6 @@ def _update(self):
             self.canvas.blit(self.ax.bbox)
         else:
             self.canvas.draw_idle()
-        return False
 
 
 class MultiCursor(Widget):
@@ -2026,10 +2021,9 @@ def onmove(self, event):
         for line in self.hlines:
             line.set_ydata((ydata, ydata))
             line.set_visible(self.visible and self.horizOn)
-        if self.visible and (self.vertOn or self.horizOn):
-            self._update()
-
-    def _update(self):
+        if not (self.visible and (self.vertOn or self.horizOn)):
+            return
+        # Redraw.
         if self.useblit:
             for canvas, info in self._canvas_infos.items():
                 if info["background"]:
