Skip to content

Commit

Permalink
we dont need result variable, just use handler.default_value directly (
Browse files Browse the repository at this point in the history
  • Loading branch information
aaronayres35 authored Apr 2, 2021
1 parent 58b4fe2 commit c97f322
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions traits/editor_factories.py
Original file line number Diff line number Diff line change
Expand Up @@ -148,12 +148,11 @@ def _instance_handler_factory(handler):
if isinstance(handler, TraitInstance):
return handler.aClass
elif isinstance(handler, BaseInstance):
result = handler.default_value
if handler.default_value_type == DefaultValue.callable_and_args:
default_value_getter, args, kwargs = result
default_value_getter, args, kwargs = handler.default_value
return lambda: default_value_getter(*args, **kwargs)
else:
return result
return handler.default_value
else:
msg = "handler should be TraitInstance or BaseInstance, but got {}"
raise ValueError(msg.format(repr(handler)))
Expand Down

0 comments on commit c97f322

Please sign in to comment.