Skip to content

Conversation

thewalkingmonty
Copy link

I initially overlooked @Qix- fix #740 to issue #678 and came up with almost the same solution, with just the difference the enabled property is managed in a slight different way that:

  • caches the status of the enabled property maintaining a stable performance when used in non dynamic context
  • do not change the way the enable function interacts with the enabled property

So I'm proposing my fix besides @Qix-'s one as it may add some little value.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 89.568% when pulling c3646d0 on thewalkingmonty:no-leak into 6087241 on visionmedia:no-leak.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants