Skip to content

Commit

Permalink
Merge pull request #49 from SylvainCorlay/remove
Browse files Browse the repository at this point in the history
Use remove rather than index + del
  • Loading branch information
minrk committed Jul 15, 2015
2 parents a31012b + 60f63ae commit 75d3b2a
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions traitlets/traitlets.py
Original file line number Diff line number Diff line change
Expand Up @@ -699,13 +699,10 @@ def _add_notifiers(self, handler, name):

def _remove_notifiers(self, handler, name):
if name in self._trait_notifiers:
nlist = self._trait_notifiers[name]
try:
index = nlist.index(handler)
self._trait_notifiers[name].remove(handler)
except ValueError:
pass
else:
del nlist[index]

def on_trait_change(self, handler, name=None, remove=False):
"""Setup a handler to be called when a trait changes.
Expand Down

0 comments on commit 75d3b2a

Please sign in to comment.