Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

What's the purpose for last_bound? #12

Open
VivekManglani opened this issue Jan 4, 2017 · 2 comments
Open

What's the purpose for last_bound? #12

VivekManglani opened this issue Jan 4, 2017 · 2 comments
Assignees
Labels

Comments

@VivekManglani
Copy link

Hi guys,

I am curious to understand what is the purpose of last_bound? It seems to just store the last bound data for some reason.

Also, if the next function returns an empty value, then it's replaced with the last bound value. Which is very weird.

@Nomafin
Copy link
Contributor

Nomafin commented Jan 4, 2017

It is meant to make bind() function work with function cacheing. The logic isn't very good though now when I look at it, because it has that side effect you mentioned and also it would work only with exactly one bind() in a function.

We need to rethink the approach to this issue, we'll look into it and hopefully come up with a bug fix soon.

@VivekManglani
Copy link
Author

@Nomafin thanks for the hot fix, we just updated our codebase and it seems to be working fine now!

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

No branches or pull requests

3 participants