Skip to content
Browse files

add comment to protect against future 'cleanup' of `if ret is False`

  • Loading branch information...
1 parent c5304ee commit 2ac78c7773961a51015e59a92b006594c937e746 @minrk committed
Showing with 2 additions and 0 deletions.
  1. +2 −0 lib/matplotlib/
2 lib/matplotlib/
@@ -1123,6 +1123,8 @@ def _on_timer(self):
for func, args, kwargs in self.callbacks:
ret = func(*args, **kwargs)
+ # docstring above explains why we use `if ret is False` here,
+ # instead of `if not ret`.
if ret is False:
self.callbacks.remove((func, args, kwargs))

0 comments on commit 2ac78c7

Please sign in to comment.
Something went wrong with that request. Please try again.